[Python-Dev] [Python-checkins] r84559 - python/branches/py3k/Lib/subprocess.py (original) (raw)
Brian Curtin brian.curtin at gmail.com
Tue Sep 7 15:28:40 CEST 2010
- Previous message: [Python-Dev] [Python-checkins] r84559 - python/branches/py3k/Lib/subprocess.py
- Next message: [Python-Dev] [Python-checkins] r84562 - in python/branches/py3k: Doc/library/io.rst Lib/_pyio.py Lib/test/test_memoryio.py Misc/NEWS Modules/_io/_iomodule.c Modules/_io/_iomodule.h Modules/_io/bytesio.c
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On Tue, Sep 7, 2010 at 08:19, Nick Coghlan <ncoghlan at gmail.com> wrote:
On Tue, Sep 7, 2010 at 11:05 PM, Brian Curtin <brian.curtin at gmail.com> wrote: > Sure, seems reasonable to me. > Does """raise ValueError("Unsupported signal: {}".format(sig))""" look fine, > or is there a more preferred format when displaying bad values in exception > messages?
No, that's about what I was thinking as well. When all we have is an error code (or similar number), it's difficult to make the associated error message particularly pretty. Cheers, Nick.
Made the adjustment in r84582 (py3k) and r84583 (release27-maint). Thanks for the suggestion. -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://mail.python.org/pipermail/python-dev/attachments/20100907/9b2b2e62/attachment-0001.html>
- Previous message: [Python-Dev] [Python-checkins] r84559 - python/branches/py3k/Lib/subprocess.py
- Next message: [Python-Dev] [Python-checkins] r84562 - in python/branches/py3k: Doc/library/io.rst Lib/_pyio.py Lib/test/test_memoryio.py Misc/NEWS Modules/_io/_iomodule.c Modules/_io/_iomodule.h Modules/_io/bytesio.c
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]