Opened 5 years ago

Closed 5 years ago

#15609 closed (fixed)

Respect PEP 3109

Reported by: DaNmarner Owned by: nobody
Component: Uncategorized Version: master
Severity: Keywords: PEP
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:


Tho porting to Python 3 has not started yet, there're things to be done without affecting the current support for Python (>2.4).

As mentioned in PEP 3109, one of Python's guiding maxims is "there should be one -- and preferably only one -- obvious way to do it". Specifically, raising exceptions with a message should be done in the ONE TRUE WAY since Python 1.5, as forced by Python 3:

raise E(V)

All this patch does is to respect this principle.

Attachments (1)

respect_pep3109.patch (4.1 KB) - added by DaNmarner 5 years ago.

Download all attachments as: .zip

Change History (3)

Changed 5 years ago by DaNmarner

comment:1 Changed 5 years ago by lrekucki

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Ready for checkin

No need to quote the Zen. This is a PEP 8 recommendation:

When raising an exception, use "raise ValueError('message')" instead of the older form "raise ValueError, 'message'".

comment:2 Changed 5 years ago by adrian

  • Resolution set to fixed
  • Status changed from new to closed

In [15811]:

Fixed #15609 -- Fixed some 'raise' statements to use the newer style syntax. Thanks, DaNmarner

Note: See TracTickets for help on using tickets.
Back to Top