On Sun, Feb 15, 2009 at 2:33 PM, Kai Willadsen <[email protected]> wrote:
> 2009/2/15 Vincent Legoll <[email protected]>:
>> Here we should use the lambda variable instead of an outer scope
>> variable, I think...
>
> I think the correct fix here would be to absorb the lambda into the
> one-and-only place that it's used (three lines below). Using a lambda
> here at all seems to me to be a net readability loss.

Agreed, I'll do it and resend an updated patch later

-- 
Vincent Legoll
_______________________________________________
meld-list mailing list
[email protected]
http://mail.gnome.org/mailman/listinfo/meld-list

Reply via email to