Michael Niedermayer <[EMAIL PROTECTED]> added the comment: On Fri, Nov 16, 2007 at 04:19:09AM -0000, heydowns wrote: > Patch to remove SDT processing as per mru. > I have numerous samples failing this, too, which used to work fine. > > ______________________________________________________ > FFmpeg issue tracker <[EMAIL PROTECTED]> > <https://roundup.mplayerhq.hu/roundup/ffmpeg/issue138> > ______________________________________________________ > Index: tests/regression.sh > =================================================================== > --- tests/regression.sh (revision 11045) > +++ tests/regression.sh (working copy) > @@ -61,8 +61,6 @@ > do_mjpeg=y > do_ljpeg=y > do_jpegls=y > - do_rv10=y > - do_rv20=y > do_mp2=y > do_ac3=y > do_g726=y
huh? also wouldnt this patch remove the channel names? if so this is a serious regression feature wise from a users point of view and i think nico who wrote the AVProgram code should get a chance to comment before this is applied that said, iam not mpegts maintainer, its mans decission if the SDT parsing will be removed (and i dont care about SDT at all just loosing channel/program names, language codes and such would be sad. but then i dont know where they are stored anyway, and iam too tired (6am) to check the spec so maybe SDT really is redundant and can be removed with no feature regression in that case please just ignore me) [...] ______________________________________________________ FFmpeg issue tracker <[EMAIL PROTECTED]> <https://roundup.mplayerhq.hu/roundup/ffmpeg/issue138> ______________________________________________________
