[issue16261] Fix bare excepts in various places in std lib

Serhiy Storchaka added the comment: Resolved conflicts, fixed some noted issues in the patch and committed unquestionable changes. Thank you for your patch Ramchandra. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed __

[issue16261] Fix bare excepts in various places in std lib

Roundup Robot added the comment: New changeset b3a7215b9ce4 by Serhiy Storchaka in branch 'default': Issue #16261: Converted some bare except statements to except statements https://hg.python.org/cpython/rev/b3a7215b9ce4 -- ___ Python tracker

[issue16261] Fix bare excepts in various places in std lib

Changes by Serhiy Storchaka : -- assignee: -> serhiy.storchaka stage: needs patch -> patch review versions: +Python 3.5 -Python 3.4 ___ Python tracker ___ __

[issue16261] Fix bare excepts in various places in std lib

Martin Panter added the comment: I reviewed the patch on Reitveld. All of the changes that do not have comments look okay to me, although some will definitely have merge conflicts with the current code. If someone updated the patch based on the comments, I think it may be okay to commit.

[issue16261] Fix bare excepts in various places in std lib

Terry J. Reedy added the comment: See more discussion on duplicate #21259. -- ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue16261] Fix bare excepts in various places in std lib

Terry J. Reedy added the comment: The earlier #15313 is about removing bare excepts in Idle code. These should be handled separately for two reasons. 1. Changes can and in most cases should be backported. 2. As with any other Python-coded application, uncaught exceptions cause Idle to quit. Cur

[issue16261] Fix bare excepts in various places in std lib

Changes by Ezio Melotti : -- nosy: +ezio.melotti ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.p

[issue16261] Fix bare excepts in various places in std lib

Changes by STINNER Victor : -- nosy: +haypo ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python

[issue16261] Fix bare excepts in various places in std lib

Serhiy Storchaka added the comment: Want to make a few warnings. Accepting your confirmation may take a long time. And it is very unlikely that your patch can be applied as a whole. The maintainers of the individual modules can apply it by parts. -- ___

[issue16261] Fix bare excepts in various places in std lib

Serhiy Storchaka added the comment: See my comments on Rietveld. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscri

[issue16261] Fix bare excepts in various places in std lib

Richard Oudkerk added the comment: > try: > _MAXFD = os.sysconf("SC_OPEN_MAX") > -except: > +except ValueError: > _MAXFD = 256 os.sysconf() might raise OSError. I think ValueError is only raised if _SC_OPEN_MAX was undefined when the module was compiled. -- nosy: +sbt ___

[issue16261] Fix bare excepts in various places in std lib

Serhiy Storchaka added the comment: Unhappy Rietveld hasn't recognize your patch. I have regenerated it. -- nosy: +serhiy.storchaka Added file: http://bugs.python.org/file28648/bare_except.patch ___ Python tracker

[issue16261] Fix bare excepts in various places in std lib

Ramchandra Apte added the comment: Sorry, I entered the wrong filename for the patch and so it didn't get attached. Now I have attached it. -- keywords: +patch Added file: http://bugs.python.org/file28647/patch.patch ___ Python tracker

[issue16261] Fix bare excepts in various places in std lib

Ramchandra Apte added the comment: Here's a patch on revision 75fe7f5fda9a (I did it on a trip without an internet connection) There might be some mistakes of this type: I incorrectly changed instances like this ` try: something except: close file raise ` to ` try: something fina

[issue16261] Fix bare excepts in various places in std lib

Andrew Svetlov added the comment: I've fixed bare except: in docs. Tomorrow we will have Python sprint in Kiev. Maybe somebody will take a look on Lib. -- ___ Python tracker ___

[issue16261] Fix bare excepts in various places in std lib

Roundup Robot added the comment: New changeset b2bd62d1644f by Andrew Svetlov in branch 'default': Issue #16261: fix bare excepts in Doc/ http://hg.python.org/cpython/rev/b2bd62d1644f -- nosy: +python-dev ___ Python tracker

[issue16261] Fix bare excepts in various places in std lib

Ramchandra Apte added the comment: Yes. most of the bare excepts can be replaced with a stricter clause On 2 November 2012 01:06, Andrew Svetlov wrote: > > Andrew Svetlov added the comment: > > Sorry, looks like I don't understood you correctly. > Do you want to replace `except:` to `except Ex

[issue16261] Fix bare excepts in various places in std lib

Andrew Svetlov added the comment: Sorry, looks like I don't understood you correctly. Do you want to replace `except:` to `except Exception` or something else? -- ___ Python tracker

[issue16261] Fix bare excepts in various places in std lib

Ramchandra Apte added the comment: On 1 November 2012 17:21, Andrew Svetlov wrote: > > Andrew Svetlov added the comment: > > I'm pretty sure Doc and Lib are already fixed, only Tools left. > > -- > nosy: +asvetlov > > ___ > Python tracker >

[issue16261] Fix bare excepts in various places in std lib

Andrew Svetlov added the comment: I'm pretty sure Doc and Lib are already fixed, only Tools left. -- nosy: +asvetlov ___ Python tracker ___ __

[issue16261] Fix bare excepts in various places in std lib

Ramchandra Apte added the comment: This seems quite difficult for one person;there are 589 lines having "except:" I'll work on the Tools directory. If anybody wants to help, please tell which folder you will do. -- ___ Python tracker

[issue16261] Fix bare excepts in various places in std lib

Changes by Ramchandra Apte : -- type: -> behavior ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.

[issue16261] Fix bare excepts in various places in std lib

Changes by Éric Araujo : -- nosy: +eric.araujo versions: +Python 3.4 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscr

[issue16261] Fix bare excepts in various places in std lib

Brett Cannon added the comment: One thing to be aware of, Ramchandra, is that there might be a slight backwards-compatibility problem from tightening up what exceptions propagate. Now with proper documentation and only applying things to 3.4 this probably won't be a problem. -- nosy:

[issue16261] Fix bare excepts in various places in std lib

New submission from Ramchandra Apte: Will attach patch. -- components: Library (Lib) messages: 173140 nosy: ramchandra.apte priority: normal severity: normal status: open title: Fix bare excepts in various places in std lib ___ Python tracker