[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Adapt fixcc.py to use Astyle instead of emacs (issue4662074)
From: |
Neil Puttock |
Subject: |
Re: Adapt fixcc.py to use Astyle instead of emacs (issue4662074) |
Date: |
Sun, 3 Jul 2011 13:35:23 +0100 |
On 3 July 2011 13:18, Carl Sorensen <address@hidden> wrote:
> On 7/3/11 5:49 AM, "Graham Percival" <address@hidden> wrote:
>
>> On Sun, Jul 03, 2011 at 01:43:40AM -0700, Keith OHara wrote:
>>> On Sat, 02 Jul 2011 19:41:15 -0700, <address@hidden> wrote:
>>>
>>>> Can you make it possible for us to see the diff caused by applying this
>>>> script to the files you've mentioned?
>>>>
>>>
>>> I will soon (unless Graham beats me to it) for a small sampling of files.
>>> It was awkward to set up branches correctly to create the diff;
>>> I'll get back to it when I'm not tired.
>>
>> Here's a complete run:
>> http://git.savannah.gnu.org/gitweb/?p=lilypond.git;a=commit;h=710b0d99b331a824
>> d0ed3d09f3f5f43d559b71b0
>
> Thanks, Graham!
>
> Nearly every change I saw looked like an improvement.
+1
Looks pretty good to me.
> @@ -123,7 +123,7 @@ Book::mark_smob (SCM s)
> }
>
> int
> -Book::print_smob (SCM, SCM p, scm_print_state*)
> +Book::print_smob (SCM, SCM p, scm_print_state *)
> {
> scm_puts ("#<Book>", p);
> return 1;
>
> Note that scm_print_state* has changed to scm_print_state *
>
> Again, I prefer scm_print_state*
Since we associate the pointer operator with variables rather than
types I'd argue
scm_print_state *
is more `correct', even for unused variables.
Cheers,
Neil
- Adapt fixcc.py to use Astyle instead of emacs (issue4662074), k-ohara5a5a, 2011/07/02
- Re: Adapt fixcc.py to use Astyle instead of emacs (issue4662074), Carl . D . Sorensen, 2011/07/02
- Re: Adapt fixcc.py to use Astyle instead of emacs (issue4662074), percival . music . ca, 2011/07/03
- Re: Adapt fixcc.py to use Astyle instead of emacs (issue4662074), k-ohara5a5a, 2011/07/03
- Re: Adapt fixcc.py to use Astyle instead of emacs (issue4662074), Keith OHara, 2011/07/03
- Re: Adapt fixcc.py to use Astyle instead of emacs (issue4662074), Neil Puttock, 2011/07/03
- Re: Adapt fixcc.py to use Astyle instead of emacs (issue4662074), Graham Percival, 2011/07/03
- Re: Adapt fixcc.py to use Astyle instead of emacs (issue4662074), Neil Puttock, 2011/07/03
- Re: Adapt fixcc.py to use Astyle instead of emacs (issue4662074), Graham Percival, 2011/07/03
- Re: Adapt fixcc.py to use Astyle instead of emacs (issue4662074), Keith OHara, 2011/07/03
Re: Adapt fixcc.py to use Astyle instead of emacs (issue4662074), k-ohara5a5a, 2011/07/03