[Top][All Lists]
[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
- Adds a warning for non-list fingeringOrientations settings. (issue4650070), mtsolo, 2011/07/04
- Re: Adds a warning for non-list fingeringOrientations settings. (issue4650070), n . puttock, 2011/07/04
- Re: Adds a warning for non-list fingeringOrientations settings. (issue4650070), address@hidden, 2011/07/04
- Re: Adds a warning for non-list fingeringOrientations settings. (issue4650070), address@hidden, 2011/07/05
- Re: Adds a warning for non-list fingeringOrientations settings. (issue4650070), Neil Puttock, 2011/07/05
- Re: Adds a warning for non-list fingeringOrientations settings. (issue4650070), address@hidden, 2011/07/05
- Re: Adds a warning for non-list fingeringOrientations settings. (issue4650070), Neil Puttock, 2011/07/05
Re: Adds a warning for non-list fingeringOrientations settings. (issue4650070), lemniskata . bernoullego, 2011/07/04