This PR hopefully implements correctly movement/work tags support. Note that I have no experience / knowledge about saving binary files and I it took a long time to figure out what to do. I didn't try heavy usage on this library, I just tried to set metadata and read it with external m4a tools (like AtomicParsley). Please take extra care when reviewing my changes to make sure I didn't break anything...
A few thoughts about this PR:
Backwards compatibility: I couldn't find a way to implement this without breaking backwards compatibility. My guidelines were to change as little as possible - I only changed the Data::Reserved type - which hopefully is not used much by external users of the library.
API decisions: The show movement (shwm) flag is basically a boolean flag (although implemented as integer). I decided to make set_show_movement function without arguments which means you use remove_show_movement to disable. However If you prefer I can change it to accept a boolean or int as parameter.
Thank you very much for this crate. I was searching for a long time for a crate handle m4a 😄 .
Fixes #3
This PR hopefully implements correctly movement/work tags support. Note that I have no experience / knowledge about saving binary files and I it took a long time to figure out what to do. I didn't try heavy usage on this library, I just tried to set metadata and read it with external m4a tools (like AtomicParsley). Please take extra care when reviewing my changes to make sure I didn't break anything...
A few thoughts about this PR:
Data::Reserved
type - which hopefully is not used much by external users of the library.shwm
) flag is basically a boolean flag (although implemented as integer). I decided to makeset_show_movement
function without arguments which means you useremove_show_movement
to disable. However If you prefer I can change it to accept a boolean or int as parameter.Thank you very much for this crate. I was searching for a long time for a crate handle m4a 😄 .