lilypond-devel
[Top][All Lists]
Advanced

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

Re: * Allow for nested contexts of any depth.


From: Neil Puttock
Subject: Re: * Allow for nested contexts of any depth.
Date: Thu, 13 Nov 2008 23:54:45 +0000

2008/11/8 Han-Wen Nienhuys <address@hidden>:
> On Fri, Nov 7, 2008 at 9:10 PM,  <address@hidden> wrote:
>>> http://codereview.appspot.com/7261/diff/1/2
>>> File lily/context-def.cc (right):
>>>
>>> http://codereview.appspot.com/7261/diff/1/2#newcode215
>>> Line 215: seen = new set<const Context_def *> ();
>>> Can you make a wrapper function that does
>>>
>>>  set<..> seen;
>>>  path_to (.. , &seen)
>>>
>>> instead?
>>
>> I've tried, but I'm completely stumped. :)
>>
>> I barely understand how this works, so I'm a bit out of my depth
>> unfortunately.
>>
>> Are you suggesting a recursive call?
>
> No, this call is already recursive; I suggest creating another
> function which allocates the set<> and does the outer call of the
> recursive function.
>

A new patch set is up with regtest and convert-ly rule.

Cheers,
Neil




reply via email to

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