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

[Reimar Döffinger]

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.