-
Notifications
You must be signed in to change notification settings - Fork 43
Open
Description
Hey,
I tried using this plugin after installing it via NPM, but ran into some odd issues with the src/layer3.js file having some seemingly random typos throw in.
After debugging it, I determined one of the issues was MP3FrameHeader.FLAGS.tables.I_STEREO which should actually be MP3FrameHeader.FLAGS.I_STEREO. It seems this issue was fixed in the prior commit though, just the version on NPM is outdated for some reason.
Once I fixed that problem though, I ran into another very strange looking line of code.
var lsf_scale = IS_Ltables.SF_TABLE[right_ch.scalefac_compress & 0x1];There is no IS_Ltables variable, and I think it is supposed to be:
var lsf_scale = tables.SF_TABLE[right_ch.scalefac_compress & 0x1];This typo has not been fixed yet.
JohnWeisz
Metadata
Metadata
Assignees
Labels
No labels