Re: Adapt fixcc.py to use Astyle and/or emacs (issue4662074)

[Keith OHara]

On Thu, 14 Jul 2011 00:47:58 -0700, Trevor Daniels <address@hidden> wrote:

> I was hoping to recommend going with this, so we could
> wrap up this issue, but then I found
>
> - new_context->event_source ()->
> -   add_listener (GET_LISTENER
> (new_context->create_context_from_event),
> + new_context->event_source ()->
> + add_listener (GET_LISTENER
> (new_context->create_context_from_event),
>
> which looks wrong to me (by any measure of code clarity).

I commented on this in Patch Set 3, but it doesn't bother me too badly.  I put 
in an enhancement request for Astyle to indent multi-line statements.

GNU style, interpreted literally, would put the binary operator -> on the 
second line.  I could easily make fixcc.py enforce this, if you think it helps:

new_context->event_source ()
->add_listener (GET_LISTENER (new_context->create_context_from_event),
                ly_symbol2scm ("CreateContext"));

Emacs by default would indent the continuation lines two columns further.