[MPlayer-dev-eng] [PATCH] Audio balance feature

Diego Biurrun diego at biurrun.de
Tue Jun 5 18:49:21 CEST 2007


On Tue, Jun 05, 2007 at 08:13:26PM +0800, Zuxy Meng wrote:
> 
> 2007/6/5, Diego Biurrun <diego at biurrun.de>:
> > On Tue, Jun 05, 2007 at 05:38:15PM +0800, Zuxy Meng wrote:
> > >
> > > 2007/6/5, Reimar Doeffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>:
> > > >On Tue, Jun 05, 2007 at 09:13:28AM +0800, Zuxy Meng wrote:
> > > >> Will apply this weekend of no one objects.
> > > >
> > > >The indentation seems inconsistent, please check it before applying.
> > >
> > > See if this is better.
> >
> > No, it's horrible, you are mixing tabs and spaces.  Also, not adding
> > trailing whitespace is a plus.
> 
> Tabs are used only if the original file used them already. Doesn't the
> rule say that we shall try to follow existing coding style?

When you are adding complete functions, I don't see why you should be
mixing tabs and spaces.

Diego



More information about the MPlayer-dev-eng mailing list