[MPlayer-cvslog] r26411 - trunk/libmpdemux/demuxer.c

Alban Bedel albeu at free.fr
Sat Apr 12 20:06:41 CEST 2008


On Sat, 12 Apr 2008 19:24:16 +0200
Diego Biurrun <diego at biurrun.de> wrote:

> On Sat, Apr 12, 2008 at 06:23:08PM +0200, Reimar Döffinger wrote:
> > On Sat, Apr 12, 2008 at 05:51:08PM +0200, uau wrote:
> > > 
> > > Log:
> > > Reindent demuxer.c
> > 
> > and also you are supposed to ask about cosmetic changes before for
> > code you don't maintain.
> 
> Is that so?  I have reindented libpostproc in FFmpeg without asking
> first.

FFmpeg is not MPlayer. MPlayer rules state that, unless the change
is trivial, one should post a patch. And even more so if one is not the
maintainer of the code, or if the change might be controversial. Both
clearly apply here.

	Albeu



More information about the MPlayer-cvslog mailing list