[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Allows lyrics to slide under TimeSignature when OctaveEight present.
Re: Allows lyrics to slide under TimeSignature when OctaveEight present. (issue 6201068)
Fri, 11 May 2012 10:02:12 +0200
On 11 mai 2012, at 09:09, address@hidden wrote:
> File lily/pure-from-neighbor-engraver.cc (right):
> lily/pure-from-neighbor-engraver.cc:56: in_same_column (Grob *g1, Grob
> On 2012/05/11 05:23:27, MikeSol wrote:
>> On 2012/05/10 21:40:48, janek wrote:
>>> That's problably the most stupid question ever, but why this doesn't
> it begin
>>> with Pure_from_neighbor_engraver:: ? I don't see it being special.
>> It doesn't need to be a class method - it'll never be called outside
> of this
>> file. It is just a small helper function.
> Then it should be declared static, right?
Why would it be part of the class? It's just a small helper function -
architecturally, I don't see it as being part of a class. There are small
helper functions like this all over the code base.