On Tue, May 29, 2012 at 12:40 PM, Edward K. Ream <[email protected]> wrote:
> On Tue, May 29, 2012 at 11:29 AM, Terry Brown <[email protected]> wrote:
>

> Next, I'll remove the pylint warnings, and all should be right.

This warning appears valid.

E0102:313:NestedSplitterHandle.splitter_menu.cb:
    function already defined line 306

Indeed, here is the context of the definition at line 306:

        for provider in splitter.root.providers:
            if hasattr(provider, 'ns_provides'):
                for title, id_ in provider.ns_provides():
                    def cb(id_=id_):
                        splitter.open_window(action=id_)

As I read the code, the only effect of the code would be to define cb
at the *last* iteration of the for loop.  This is surely not what was
intended.

EKR.

-- 
You received this message because you are subscribed to the Google Groups 
"leo-editor" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/leo-editor?hl=en.

Reply via email to