[MPlayer-dev-eng] [PATCH] Configurable blocksize for of mpeg

[Attila Kinali]

On Sat, Jul 17, 2004 at 07:58:01PM +0300, Jan Knutar wrote:
> On Monday 12 July 2004 14:28, Diego Biurrun wrote:
> 
> > > > > ++ WARN: autopadding requires to insert additional 4996980 zero
> > > > > bytes into MPEG stream (due to 18105 unaligned packets of 18106
> > > 
> > > > if you can fix this feel free to send a patch ;-)
> > > 
> > > More proper-ish than below would be to make it configurable at
> > > runtime. I'll probably do that in the weekend when I've got some
> > > free time, if nobody beats me to it :)
> 
> > > -#define MUXER_MPEG_BLOCKSIZE 2048      // 2048 or 2324 - ?
> > > +#define MUXER_MPEG_BLOCKSIZE 2324      // 2048 or 2324 - ?
> > 
> > Weekend long over, what happened ;-)
> 
> Real life, and need to pause from bashing head against wall trying to understand
> all the (S)VCD issues :)
> 
> Anyhow, here's a patch for review, sent in the spirit of [RFC]
> 
> It adds a new mencoder option, -mpegmuxopts, which has one suboption right now,
> vcdblocksize. -mpegmuxopts vcdblocksize should be used with -of mpeg, if one
> wants to feed the .mpg from mencoder to GNU VCDImager and not have the stream
> padded beyond media size...

Any comments on this patch ?

			Attila Kianli