[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Allows lyrics to slide under TimeSignature when OctaveEight present.

From: mtsolo
Subject: Re: Allows lyrics to slide under TimeSignature when OctaveEight present. (issue 6201068)
Date: Thu, 17 May 2012 06:05:39 +0000

On 2012/05/11 08:11:18, dak wrote:
"address@hidden" <mailto:address@hidden> writes:

> On 11 mai 2012, at 09:09, mailto:address@hidden wrote:
>> File lily/ (right):
>> lily/ in_same_column (Grob *g1,
>> *g2)
>> 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
>> it begin
>>>> with Pure_from_neighbor_engraver:: ?  I don't see it being
>>> It doesn't need to be a class method - it'll never be called
>> 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
> - architecturally, I don't see it as being part of a class.  There
> small helper functions like this all over the code base.

I did not say it should be declared a class function.  I said that it
should be declared static since it is not referenced outside of the
source file.

David Kastrup

Hey David,

I don't completely understand the hows and whys of it, and there are
many functions in the source code that are only used in one file but are
not declared static.  If all of these should be declared static, I'd
recommend opening a tracker issue describing the general problem.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]