[MPlayer-cvslog] r33375 - trunk/TOOLS/mp-uncrustify-style.cfg

[Clément Bœsch]

On Thu, May 05, 2011 at 06:32:03PM +0200, Reimar Döffinger wrote:
> On Thu, May 05, 2011 at 01:13:34PM +0200, cboesch wrote:
> > Author: cboesch
> > Date: Thu May  5 13:13:33 2011
> > New Revision: 33375
> > 
> > Log:
> > Do not prefer '// A' over '//A' in uncrustify profile.
> > 
> > This tends to break Doxygen comments.
> > 
> > Modified:
> >    trunk/TOOLS/mp-uncrustify-style.cfg
> > 
> > Modified: trunk/TOOLS/mp-uncrustify-style.cfg
> > ==============================================================================
> > --- trunk/TOOLS/mp-uncrustify-style.cfg	Thu May  5 12:29:08 2011	(r33374)
> > +++ trunk/TOOLS/mp-uncrustify-style.cfg	Thu May  5 13:13:33 2011	(r33375)
> > @@ -606,7 +606,7 @@ sp_case_label                           
> >  sp_range                                 = ignore   # ignore/add/remove/force
> >  
> >  # Control the space after the opening of a C++ comment '// A' vs '//A'
> > -sp_cmt_cpp_start                         = force    # ignore/add/remove/force
> > +sp_cmt_cpp_start                         = ignore   # ignore/add/remove/force
> 
> You should maybe add a comment why you set it to this value,
> that it breaks doxygen is a really useful information because
> otherwise we'd really prefer "force".
> And maybe this doxygen issue will be fixed in a future uncrustify version.

OK, done.

-- 
Clément B.