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

[SourceForge.net]

Patches item #1104669, was opened at 2005-01-18 18:09
Message generated for change (Comment added) made by mwh
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: Michael Hudson (mwh)
Date: 2005-02-15 17:39

Message:
Logged In: YES 
user_id=6656

Jim: Yes.  It looks like I ran delete-trailing-whitespace on the file at 
some point.  I could redo the diff without them, but it would be tedious...

Tim: thanks!

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

Comment By: Jim Jewett (jimjjewett)
Date: 2005-02-15 17: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 16: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 16: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 15: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 14: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 13: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 19: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