[MPlayer-dev-eng] separate infrastructure patch?

Nico Sabbi Nicola.Sabbi at poste.it
Fri May 30 10:12:23 CEST 2008


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?
>
> I am working on television closed captions.  Interpreting them from
> the MPEG files produced by IVTV and displaying them via OSD.  The
> patch runs into thousands of lines, but mostly in separate modular
> files.  Only a smattering of changes to existing files would be
> needed if they were structured amenably.  I'd prefer to submit
> patches to do that structuring first, but would anyone take them
> that way, since they serve no immediate purpose alone?

they serve the purpose of being self-contained and easily reviewable

>
> 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



More information about the MPlayer-dev-eng mailing list