[Patches] [ python-Patches-1104669 ] new-style exceptions

SourceForge.net noreply at sourceforge.net
Tue Feb 15 18:34:33 CET 2005


Patches item #1104669, was opened at 2005-01-18 13:09
Message generated for change (Comment added) made by jimjjewett
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1104669&group_id=5470

Category: Core (C code)
Group: Python 2.5
Status: Open
Resolution: None
Priority: 5
Submitted By: Michael Hudson (mwh)
Assigned to: Nobody/Anonymous (nobody)
Summary: new-style exceptions

Initial Comment:
This patch allows new-style exceptions and makes Exception a 
new-style class.

The test suite runs, apart from failures in test_tempfile (will dig, 
but doubt this is my fault) and test__locale (known OS X problem).

----------------------------------------------------------------------

Comment By: Jim Jewett (jimjjewett)
Date: 2005-02-15 12:34

Message:
Logged In: YES 
user_id=764593

Several lines near the end of errors.c had no visible 
change.  Was this a whitespace cleanup, or is this a risk of 
tab/space mixing?

----------------------------------------------------------------------

Comment By: Tim Peters (tim_one)
Date: 2005-02-15 11:17

Message:
Logged In: YES 
user_id=31435

FYI, there's nothing special about PicklingError in the 
pickletools doctest, it's just aiming at an example of an 
instance -- that it's also an exception instance is irrelevant 
to what the test is aiming at.

Tell you what:  I'll check in a suitable change to 
pickletools.py, so that the example it uses stops interfering 
with this patch.

----------------------------------------------------------------------

Comment By: Tim Peters (tim_one)
Date: 2005-02-15 11:03

Message:
Logged In: YES 
user_id=31435

FYI, there's nothing special about PicklingError in the 
pickletools doctest, it's just aiming at an example of an 
instance -- that it's also an exception instance is irrelevant 
to what the test is aiming at.

Tell you what:  I'll check in a suitable change to 
pickletools.py, so that the example it uses stops interfering 
with this patch.

----------------------------------------------------------------------

Comment By: Michael Hudson (mwh)
Date: 2005-02-15 10:40

Message:
Logged In: YES 
user_id=6656

I found the final wart (caught by test_tempfile of all things).  I think the 
attached is good to go.

Issues remaining: test_pickletools fails, because PickleError is now new-
style and thus pickles differently.  Docs.

----------------------------------------------------------------------

Comment By: Michael Hudson (mwh)
Date: 2005-02-09 09:18

Message:
Logged In: YES 
user_id=6656

New patch attached.  Did this a while ago, don't actually remember the 
details of what's new :-/

The problem with the previous patch was that one of my exception-
checking macros had the side-effect of clearing any pending exception...

----------------------------------------------------------------------

Comment By: Michael Hudson (mwh)
Date: 2005-01-19 08:25

Message:
Logged In: YES 
user_id=6656

You're right!  Odd.  No time to fix it today, I'm afraid.

----------------------------------------------------------------------

Comment By: Simon Percivall (percivall)
Date: 2005-01-18 14:47

Message:
Logged In: YES 
user_id=329382

One thing: Raising an old-style class/instance doesn't give a traceback or 
populate sys.last_*.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1104669&group_id=5470


More information about the Patches mailing list