Opened 4 years ago

Closed 4 years ago

#18799 closed enhancement (fixed)

Python 3 preparation: Change syntax of raise with traceback

Reported by: wluebbe Owned by:
Priority: major Milestone: sage-6.10
Component: misc Keywords: python3
Cc: Merged in:
Authors: Wilfried Luebbe Reviewers: Frédéric Chapoton
Report Upstream: N/A Work issues:
Branch: 249b939 (Commits) Commit: 249b939ee9144e5c2cfb939dda5a1c178cf82960
Dependencies: Stopgaps:

Description

2to3 proposes changes for

raise E, V, T -> raise E(V).with_traceback(T)
raise E, None, T -> raise E.with_traceback(T)

These changes are invalid (see also #18796).

Several options are given in the documentation of python-future (see http://python-future.org/compatible_idioms.html#raising-exceptions).

This ticket is tracked as a dependency of meta-ticket #16052. The simple raise statements are fixed in #15990.

Change History (4)

comment:1 Changed 4 years ago by wluebbe

  • Authors set to Wilfried Luebbe
  • Branch set to u/wluebbe/18799
  • Commit set to 249b939ee9144e5c2cfb939dda5a1c178cf82960
  • Status changed from new to needs_review

New commits:

249b939Trac #18799: Change syntax of raise with traceback

comment:2 Changed 4 years ago by chapoton

I propose to convert the elliptic curve one to a simple raise (no traceback)

comment:3 Changed 4 years ago by chapoton

  • Milestone changed from sage-6.8 to sage-6.10
  • Reviewers set to Frédéric Chapoton
  • Status changed from needs_review to positive_review

ok, let it be.

comment:4 Changed 4 years ago by vbraun

  • Branch changed from u/wluebbe/18799 to 249b939ee9144e5c2cfb939dda5a1c178cf82960
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.