Re: GOP-PROP 3: C++ formatting

[Keith OHara]

On Mon, 27 Jun 2011 05:45:54 -0700, Graham Percival <address@hidden> wrote:

> On Sun, Jun 26, 2011 at 06:12:07PM -0700, Keith OHara wrote:
> > On Sun, 26 Jun 2011 10:48:36 -0700, Graham Percival <address@hidden> wrote:
> >
> > >A few specific problems with astyle:
> > >    * it doesn’t indent enum in gnu style
> >  I can't find gnu instructions for indenting enums, but if there are any the 
> > astyle maintainers would probably want their gnu-mode to conform.
>
> Yes, probably.  Unfortunately, the only real specification of "gnu
> style" seems to be "whatever emacs does".
and <http://www.gnu.org/prep/standards/standards.html>

> When I add
>   --pad-paren-out
>   --pad-paren-oper
> (which is what -dp does)
>
> I see changes like this:
> -      int barnum = measure_number (context());
> +      int barnum = measure_number (context() );

In defense of astyle, I should have written astyle -Udp
 --unpad-paren
 --pad-paren-out
 --pad-oper

-int barnum=me->measure_number(context());
-int barnum=me->measure_number(bar(a));
+int barnum = me->measure_number (context());
+int barum = me->measure_number (bar (a));

and if a human puts a space in "context ()" astyle -Udp leaves it in place.