humdek-unibe-ch / smx-core-rts

The runtime system (RTS) library for the coordination language Streamix.
Mozilla Public License 2.0
0 stars 0 forks source link

smx_read_fifo in smx_tf_read_inputs #1

Closed moiri closed 5 years ago

moiri commented 5 years ago

The function smx_tf_read_inputs makes a call to smx_fifo_read. This is bad for two reasons:

To solve both issues, use smx_channel_read instead of smx_fifo_read.

moiri commented 5 years ago

In hindsight I now understand that this was done on purpose: A TF should not create a backup for each read which is why the standard read function was used instead of the decoupled version.

To make things clearer two new types of FIFO were introduced (see adad93eb97bbc64d00ea3057d864b278fad7a023)