rust-id3 icon indicating copy to clipboard operation
rust-id3 copied to clipboard

add serde support

Open vauradkar opened this issue 1 year ago • 2 comments

Sending metadata across systems would be very convenient if the tags support serde. Would you be willing to take a patch like this https://github.com/polyfloyd/rust-id3/compare/main...vauradkar:rust-id3:main ?

vauradkar avatar Dec 31 '24 16:12 vauradkar

I have considered this in the past, but concluded that it would make little sense for a crate that already implements a serialization format.

What is your use case?

polyfloyd avatar Jan 01 '25 11:01 polyfloyd

Client-server based tag processing. Client doesn't play or process audio. So server doesn't need to load or send entire audio file. serde makes it convenient also in the sense that wire format(bincode, json, toml, etc) is independent of the id3 library.

vauradkar avatar Jan 01 '25 15:01 vauradkar

Hey! Going though some open issues and I see this is still awaiting a verdict.

I will not be adding serde support as that would expose too many of the internals. Serializing to binary id3 and back is the most portable way to do this. You should be able to dump just the tag from the file and send that over to the server

polyfloyd avatar Jul 28 '25 20:07 polyfloyd