Saimadhav Heblikar added the comment:
This patch does
1.Remove pep8 violations in PathBrowser.py . Replaces file,dir,sorted by
file_,dir_,sorted_ respectively.
2.Extends test coverage for PathBrowser.py in idle_test/test_PathBrowser.py
New modules now under tests include
1.dirBrowserTreeItem
Changes by Saimadhav Heblikar saimadhavhebli...@gmail.com:
Removed file: http://bugs.python.org/file34242/pathbrowser1.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16226
___
Terry J. Reedy added the comment:
S.H. moved his patch for new tests to new issue #20792.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16226
___
Terry J. Reedy added the comment:
Test has been committed on all three branches as part of #15392
--
nosy: +terry.reedy
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16226
___
Roundup Robot added the comment:
New changeset d369e50677a6 by Ned Deily in branch '3.3':
Issue #16226: Fix IDLE Path Browser crash.
http://hg.python.org/cpython/rev/d369e50677a6
New changeset 58f9523cf407 by Ned Deily in branch 'default':
Issue #16226: Fix IDLE Path Browser crash.
Ned Deily added the comment:
I've committed Roger's fix for release in 3.3.1 and 3.4.0. Thanks, Roger. I
did not apply the proposed test since, as it stands, it would never be run by
test.regrtest without establishing a idlelib test runner infrastructure as
proposed by Issue15392. I'm
New submission from Francisco Gracia:
The menu option *File / Path browser* (as well in the *Shell* window as in the
*Editor* one) shows a new window with a tree structure rooted at *sys.path*.
The available leaf nodes show the *plus* sign that usually implies that they
can be expanded by
Ned Deily added the comment:
Thank you for the report. There does seem to be a regression in the 3.3
version of IDLE. Using the OS X version of bin/idle3.3 and selecting menu item
File - Path Browser results in the following exception:
Exception in Tkinter callback
Traceback (most recent
Roger Serwy added the comment:
Here's the revision that broke it: b81ddaf0db47
The 2.7 branch still uses imp.get_suffixes, whereas the 3.3 and 3.4 uses this
importlib construct.
The imp.get_suffixes returns a list of tuples of 3 items: (suffix, mode, type).
The current code only returns what