ID3 2.0 Release

[43081j]

I have been gone a long time and have had very little time to maintain this repo (I still do lack time, in fact). However, I am going to give 2.0 a go.

This won't be a release of any new functionality but rather a major cleanup/refactor and improvement of build process, along with any possible bug fixes brought up since I was last around.

TODO

• Add gulp along with necessary tasks
• Use browserify instead of manually trying commonjs/amd/globals
• Move dependencies to NPM instead (readerjs)
• Move dataview-extras to a separate class rather than extending the native DataView
• Fix any lint issues
• Fix major bugs
  • getUint16 - Index out of range #6, add a guard or figure some nicer way
  • Take a look at the (hacky) decodeURIComponent(escape(...)) calls

Notes

The current branch being used for this is feature/browserify-cleanup, not so appropriately named but it'll do for now.

The browserify output needs testing. I have tried it quickly with an mp3 in my browser and it worked fine.

Please do help if you can, this is not such a simple task and I do not have the capacity to do it alone in an acceptable time frame

J

[MaffooBristol]

Hello! What's the current situation with this release?

[43081j]

We have some issues with the decodeURIComponent calls, I haven't yet had time to go through it.

Also, the fix for #6 seems too much of a workaround to me, rather than a solution. I am not entirely sure why it happens as there seem to be multiple cases. Suppressing it doesn't really tell us why, does it?

[vheuken]

Any progress on this release? Getting #6 fixed is pretty crucial.

[r3wt]

convert readAsBinaryString -> readAsArrayBuffer so we can support IE?

[sc1305]

Is only the dist/id3.min.js updated with new lib files? Can you please checkin updated id3.js also.