[MPlayer-dev-eng] Re: [PATCH] Allow mf:// to read indirect file list

[Diego Biurrun]

On Sat, Nov 26, 2005 at 10:21:46PM +0100, Alban Bedel wrote:
> On Sat, 26 Nov 2005 21:44:49 +0100
> Diego Biurrun <diego at biurrun.de> wrote:
> 
> > On Sat, Nov 26, 2005 at 06:11:11PM +0100, Alban Bedel wrote:
> > > On Sat, 26 Nov 2005 17:03:21 +0100
> > > Guillaume POIRIER <poirierg at gmail.com> wrote:
> > > > 
> > > > I applied the doc patch with some modifications. I'll not apply the
> > > > code part as my craving for cola is not big enough (and my ability to
> > > > fix code is even worse).
> > > 
> > > Done, I was intending to commit only the code anway :)
> > 
> > Please, if you commit patches, don't leave out the documentation part,
> > it just creates unnecessary work for the documentation maintainers.  If
> > there are problems, we will fix them in CVS.
> 
> If you prefer it like this. It's just that this patch generated some
> warnings on the doc part and i'm uneasy on commiting things i can't
> test at all.

I prefer it this way.  It's very easy for me and the others to fix
things in CVS and the breakage you can do to the documentation is
limited.  Fishing out obscure patches can be time-consuming, though, and
it might easily be forgotten.

Diego