[MPlayer-cvslog] r37706 - trunk/gui/dialog/fileselect.c

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sat Feb 13 18:57:40 CET 2016


On Sat, Feb 13, 2016 at 06:51:37PM +0100, Ingo Brückl wrote:
> Reimar Döffinger wrote on Sat, 13 Feb 2016 18:33:53 +0100:
> 
> > On Sat, Feb 13, 2016 at 06:27:30PM +0100, ib wrote:
> >> Cosmetic: Format to MPlayer coding style.
> >>
> >> -char * const fsVideoAudioFilterNames[][2] = {
> >> +char *const fsVideoAudioFilterNames[][2] = {
> 
> > I really don't care either way, but I considered the coding style to
> > have the * direct in front of the variable name (because
> > that matches the semantics if you declare multiple variables
> > with comma), but not when it's a const following (because
> > the * does not apply to the const so to say).
> 
> Our uncrustify config has to be changed then. I'll see whether this is
> configurable that way.

Don't worry about it, I'm fine with that way as well,
but probably my fingers won't type like that on their
own :)
But I think it is the sp_before_unnamed_ptr_star option.


More information about the MPlayer-cvslog mailing list