[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Implement framework for post-fix text (de)cresc spanners
From: |
Han-Wen Nienhuys |
Subject: |
Re: Implement framework for post-fix text (de)cresc spanners |
Date: |
Fri, 28 Aug 2009 12:10:16 -0300 |
On Fri, Aug 28, 2009 at 7:29 AM, Reinhold
Kainhofer<address@hidden> wrote:
>> - get_property_setting() should take SCMs rather than const char* , so
>> you save runtime lookups.
>
> I don't exactly understand what you mean by this... The two parameters of
> get_property_setting are passed on the get_property, which call
> internal_get_property (symbol2scm (...)) on them. Of course, I could do the
> symbol2scm before calling get_property_setting and then call
> internal_get_property there, but I don't see how this will save anything (in
> particular, since the context property names will be dynamically built from
> the start_type: (start_type + "Text").c_str ()
>
> The only thing that might be optimized is the fixed-name event properties. But
> that saves one symbol2scm call for hairpins and two for text crescendi. Is
> this really worth the trouble of handling the two property names differently
> to
> save on symbol2scm call?
Sorry - I only took a brief look. It might not be worth the trouble
in this case, but in general it is better for the C++ part to work
with SCMs, so the symbol lookups can be memoized in the caller.
--
Han-Wen Nienhuys - address@hidden - http://www.xs4all.nl/~hanwen
- Re: Implement framework for post-fix text (de)cresc spanners, Han-Wen Nienhuys, 2009/08/22
- Re: Implement framework for post-fix text (de)cresc spanners, Reinhold Kainhofer, 2009/08/22
- Re: Implement framework for post-fix text (de)cresc spanners, Carl Sorensen, 2009/08/22
- Re: Implement framework for post-fix text (de)cresc spanners, Graham Percival, 2009/08/22
- Re: Implement framework for post-fix text (de)cresc spanners, Carl Sorensen, 2009/08/22
- Re: Implement framework for post-fix text (de)cresc spanners, Reinhold Kainhofer, 2009/08/23
- Re: Implement framework for post-fix text (de)cresc spanners, Graham Percival, 2009/08/23
- Re: Implement framework for post-fix text (de)cresc spanners, Reinhold Kainhofer, 2009/08/29
Re: Implement framework for post-fix text (de)cresc spanners, Reinhold Kainhofer, 2009/08/28
- Re: Implement framework for post-fix text (de)cresc spanners,
Han-Wen Nienhuys <=