[Top][All Lists]

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

Re: Replace internal_get_property with get_property where possible (issu

From: janek . lilypond
Subject: Re: Replace internal_get_property with get_property where possible (issue 137760043 by address@hidden)
Date: Wed, 03 Sep 2014 22:42:25 +0000


On 2014/09/02 13:45:45, dak wrote:
So I went through a review after all and found a few niggles that seem
fixing.  Probably does not require a full countdown and stuff, but
patchy run might be warranted in order to avoid typos.

Thanks for reviewing, and for giving me green light for pushing without
next full countdown (i take this as a sign of trust).  However,
considering how my recent patches tended to have problems, i'll leave it
for a full cycle, especially that i'm not in hurry with this one.
File lily/ (right):
lily/ SCM meta = info.grob ()->get_property
On 2014/09/02 13:33:02, dak wrote:
I'm not sure whether meta lookups would be wanted in the statistics
when we have
the stat-collection enabled.

Hmm.  I don't know either.  I'd really like to get rid of
internal_get_property where possible (so that no one will be confused as
i was), but maybe here it'd be better to err on the safe side...
File lily/ (right):
lily/ SCM min_ext = get_property (min_ext_name);
On 2014/09/02 13:45:44, dak wrote:
This one is bad since it breaks memoization.  Either leave it alone or
(properly formatted)
SCM min_ext = a == X_AXIS ? get_property ("minimum-X-extent") :

File lily/ (right):
lily/ SCM align (me->get_property (sym));
On 2014/09/02 13:45:45, dak wrote:
See above.

lily/ ? me->get_property
On 2014/09/02 13:45:45, dak wrote:
Here you do it properly!

File lily/ (right):
lily/ SCM vis = bars_[0]->get_property
On 2014/09/02 13:45:45, dak wrote:
This causes the symbol to be memoized three times in a row.  However,
only the
first time the code is being run, so I'd say never mind.  The
readability should
be worth it.


reply via email to

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