[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Run fixcc + astyle2.02.1 (issue 6477062)
From: |
Benkő Pál |
Subject: |
Re: Run fixcc + astyle2.02.1 (issue 6477062) |
Date: |
Mon, 27 Aug 2012 20:47:03 +0200 |
>>> lily/include/skyline.hh:58: list<Building> *const result);
>>> I'd like to see the const removed (top-level const on function parameter
>>> types are ignored), but that may be the target of another patch.
>
> Not exactly correct: they are not part of the function signature (like
> "result" isn't), but they are relevant in the actual function, uh,
> declaration? I mean the thing when the function body follows.
it's the definition. I agree that it's relevant there, but here
I talk about the declaration (the thing in the .h file) only.
>> Good idea! However, that would be a more intelligent
>> modification, which doesn't fit with fixcc's mandate. At the
>> moment I just want to move forward with the automatic formatting.
agreed.
p