midi2-dev / bl-midi2-rs

Ergonomic, versatile, strong types wrapping MIDI 2.0 message data.
https://crates.io/crates/midi2
Apache License 2.0
18 stars 2 forks source link

Review Timestamp Headers Implementation #12

Closed BenLeadbetter closed 1 month ago

BenLeadbetter commented 2 months ago

Currently every (mutable) message type has a header for timestamp information. This is perhaps not the best design choice, since timestamps are officially distinct from the messages to which they apply.

It is perhaps better to allow Utility messages to simply be first class message types.