lilypond-devel
[Top][All Lists]
Advanced

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

Re: Adds a warning for non-list fingeringOrientations settings. (issue46


From: Carl Sorensen
Subject: Re: Adds a warning for non-list fingeringOrientations settings. (issue4650070)
Date: Mon, 4 Jul 2011 08:31:12 -0600

On 7/4/11 7:14 AM, "Neil Puttock" <address@hidden> wrote:

> On 4 July 2011 13:53, address@hidden <address@hidden> wrote:
> 
>> I didn't realize this was the real issue :)
>> Any tips as to how one would go about fixing this?  Anything that happens
>> before engravers kick in (dispatchers, parsers, etc.) remains a mystery to
>> me...
> 
> Context_def::add_context_mod () is where the assignment takes place
> (and you can see from set_property () how the type-checking is done).
> I suppose though this is deliberate, otherwise every compilation would
> redundantly type-check settings from default context definitions,
> which we assume are correct (i.e., from
> engraver-init.ly/performer-init.ly).

Would a redundant check of settings from default context definitions be a
problem?  I can't imagine that such a check would take 1% of the processing
time.

Plus, I don't think it's really a redundant check; I think it's a real
check.   Absent such a check, we're trusting on the *-init.ly files being
correct, which admits a potential programming error.

Thanks,

Carl




reply via email to

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