[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