[Patches] [ python-Patches-403671 ] Allow jython to complete the test_class test

[noreply@sourceforge.net]

Patches item #403671, was opened at 2001-02-07 13:06
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=305470&aid=403671&group_id=5470

Category: None
Group: None
>Status: Closed
Resolution: Accepted
Priority: 5
Submitted By: Finn Bock (bckfnn)
Assigned to: Barry Warsaw (bwarsaw)
Summary: Allow jython to complete the test_class test

Initial Comment:
A breakdown of the changes:

- the __del__ method must be present at class definition time for jython to use it.
- slices pass into __xxxitem__ works, but the default values are not "0, maxint, None" but "None, None, 1". At this time this is a known difference.
- in jython the __int__, __long__ etc methods must return a correctly typed value. The test that just return None is skipped.
- An explicit call to GC is required to force a call to __del__.

The two changes that just adds a print "the expected result" is really ugly, but it is necessary for jython to generate the same output as python.

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

>Comment By: Barry Warsaw (bwarsaw)
Date: 2001-08-20 13:30

Message:
Logged In: YES 
user_id=12800

Applied to the Py2.2 trunk and 2.2a2 branch.

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

Comment By: Finn Bock (bckfnn)
Date: 2001-08-09 11:40

Message:
Logged In: YES 
user_id=4201

Yes, combatibility have not improved in any of these areas. 
Jython still needs helps like this patch to pass the test. 

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

Comment By: Guido van Rossum (gvanrossum)
Date: 2001-08-09 09:23

Message:
Logged In: YES 
user_id=6380

Accepted.  I'll leave it to Barry to check this in.  (Finn,
do you still need this?  I assume so!)

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

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