Code

Opened 3 years ago

Closed 3 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:

Description

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 3 years ago.

Download all attachments as: .zip

Change History (3)

Changed 3 years ago by DaNmarner

comment:1 Changed 3 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 3 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

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.