tag icon indicating copy to clipboard operation
tag copied to clipboard

Bitrate calculation for mp3

Open xhenner opened this issue 9 years ago • 3 comments

This patch reads the first mp3 frames, with support for the XING header to speed things up It adds some sanity check (for example a flac file with an id3 tag will be rejected) I tested it against 20k mp3. Without the patch, it takes 19s to scan the whole library With the patch, it takes 52s

I still have some checks to do. I want to compare the result with other tool I know are working fine

Next step : adding the support for mp4 and flac files ; should be easier

xhenner avatar May 20 '15 00:05 xhenner

are you still working on that ?

xhenner avatar Jun 09 '15 22:06 xhenner

This needs to be in a sub-package. I started to work on it and then didn't have time to finish. I don't have a use for the code at the moment, but if you want to refactor it out into a sub-package that would be great.

dhowden avatar Jun 09 '15 22:06 dhowden

I don't really get the subpackage idea For mp3, it make sense, but for mp4, the information is in the moov atom. Using a sub-package will make thing more complicated

Same thing for the flac format

xhenner avatar Jun 09 '15 22:06 xhenner