Re: Error in streamed MPEG4 handling - Movie.pm or Audio::Scan error ?

Summary:  I think the PR can be merged.

I did some more testing with a variety of m4a audio podcasts as well as
some more Tidal (m4a) and Accuradio tracks. I'm summarising the results
to record them somewhere,  not to get more changes.

All podcast I tested had an "stco" atom.

A couple of podcasts didn't play on LMS but I think not related to the
proposed fix (e.g. incorrect MIME type)
http://images.anandtech.com/reviews/Podcast/AnandTech_Podcast_048.m4a
https://mbse-podcast.rocks/wp-content/uploads/2021/12/TMPCT_19-24.m4a

One podcast which played OK on LMS, had an "stco" atom but its offset
was *not* used.  This podcast had "moov" after "mdat" and so technically
is not a streaming MPEG-4 file. Great that LMS managed to play it.
https://www.smarterartschool.com/uploads/8/3/1/7/8317777/danny_schwartz_aug30_-_10_9_20_12.04_pm.m4a

I haven't seen any recent comments about non-playing m4a podcasts, so I
think not necessary to make more changes. 

With regard to why there are extra 8 zeros in Accuradio mdat field - my
current theory is related to the encoder implementation.  Without
knowing the total size of audio data when creating the header, the
encoder is allowing for a very large mdat atom (i.e. audio data) which
would require the 64 bit largesize field.


------------------------------------------------------------------------
bpa's Profile: http://forums.slimdevices.com/member.php?userid=1806
View this thread: http://forums.slimdevices.com/showthread.php?t=115678