[MPlayer-dev-eng] separate infrastructure patch?
Aurelien Jacobs
aurel at gnuage.org
Fri May 30 15:47:44 CEST 2008
On Fri, 30 May 2008 10:12:23 +0200
Nico Sabbi <Nicola.Sabbi at poste.it> wrote:
> On Friday 30 May 2008 04:38:54 Bryan Henderson wrote:
> > I need some advice on submitting a large enhancement. Should
> > infrastructure changes that (supposedly) have no functional effect
> > by themselves be submitted separately? Is it better to submit in
> > pieces or all at once?
Separate in small self-contained patch as much as you can.
> > I've attached an example. This creates the function demux_flush().
> > Five parts of Mplayer have to flush the various demuxer streams (as
> > when the user seeks). They flush them individually, and not all
> > the same way. This patch replaces that with five calls to
> > demux_flush(), which flushes all the streams.
> >
> > This matters to closed captioning, because the closed caption
> > stream is one more thing that has to be flushed; it makes one place
> > instead of five for the closed caption patch to patch.
>
>
> the patch is good to commit and a perfectly structured
Except that it won't compile (I guess)...
demuxer->television_vbi do not exist for now, and shouldn't be
referenced in your patch.
Aurel
More information about the MPlayer-dev-eng
mailing list