>> If this does the job for others, then someone can commit. I won't have 
>> time until late tonight.
>
>I have backported it to branch, and I can commit. I'd like to hear Vincent's
>opinion first, though, since his approach (removing updateMacro calls) was
>quite different.

Well, both should be fixed, the logic in updateLabels is bypassed by this call 
to updateMacros in setParent.

I'm a bit sceptical that someone did the effort to write a FIXME and to change 
updateLabels according to this, just not to implement a for-loop of one line of 
code. Why ? Maybe we miss something and we introduce another bug. 

By the way, doing both like in Richard's patch would always be ok I guess.

Vincent

Reply via email to