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

[Jan Knutar]

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...
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ofmpeg_configurable_blocksize3.patch
Type: text/x-diff
Size: 4171 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040717/fbfbc237/attachment.patch>