On Mon, Nov 05, 2007 at 03:20:51PM -0600, Bo Peng wrote:
> >  ASpell::ASpell(BufferParams const &, string const & lang)
> >
> > by
> >
> >  ASpell::ASpell(BufferParams const & , string const & lang)
> >
> >
> > I think I prefer the original version.
> > So, as I said: Checking patches with it is nice, but as it stands I
> > would not like to see it ruling our style.
> 
> There are 138 options and I have not gone through them one by one.
> Most, if not all, your preferences can be achieved by changing
> uncrustify.cfg.

Possibly, but _I_ won't do that ;-)

Apart from that there's always the "You do not have to stick to rule X
if it makes your code look really ugly". That's something that's not 
easilty taught to any tool...

Andre'

Reply via email to