bpa wrote: > A long time, I was involved in changing standard faad to accept input > from stdin. Most of the change were rewritten (for the better) when > Logitech (Andy) licensed faad. The build was also changed from a > cygwin one to a proper MS Visual C one. > > The issue is probably due to the ADTS header which IIRC only occurs once > with a file but can be many times with a stream and faad has to find the > header to sync the start of stream and get description of audio stream > characteristics.. > > edit: > > The build (e.g. vcproj) files for MS Visual Studio are included in the > sdtandard faad-2.7 zip file in the frontend directory
Thanks - I hadn't spotted that there. Well... It is a genuine bug in the LMS patched faad (for windows)! As part of the machinations dealing with stdin input and looking for a header, the code does a SEEK_END on the input file - which is expected to fail. But it doesn't actually fail on windows - it seeks forward 4096 bytes in the stream. This puts it into the middle of the aac data, missing the first header. The code then searches for a header and for this stream there happens to be something that looks like a header in that data before the next real header, and decoding fails. This means that on the windows platform the faad transcoder will always be missing the start of the stream - I'm surprised this hasn't been noticed before. I've attached a new faad.exe which works. We really need Michael to update the distributions 'officially' I guess. +-------------------------------------------------------------------+ |Filename: faad.zip | |Download: http://forums.slimdevices.com/attachment.php?attachmentid=17757| +-------------------------------------------------------------------+ ------------------------------------------------------------------------ utgg's Profile: http://forums.slimdevices.com/member.php?userid=40900 View this thread: http://forums.slimdevices.com/showthread.php?t=53229 _______________________________________________ plugins mailing list [email protected] http://lists.slimdevices.com/mailman/listinfo/plugins
