2008/11/8 Han-Wen Nienhuys <[EMAIL PROTECTED]>:
> On Fri, Nov 7, 2008 at 9:10 PM,  <[EMAIL PROTECTED]> 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


_______________________________________________
lilypond-devel mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to