[Patches] [ python-Patches-1172711 ] long long support for array module

SourceForge.net noreply at sourceforge.net
Fri Apr 8 01:53:34 CEST 2005


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

Category: Library (Lib)
Group: Python 2.5
Status: Open
Resolution: None
Priority: 5
Submitted By: Oren Tirosh (orenti)
Assigned to: Nobody/Anonymous (nobody)
Summary: long long support for array module

Initial Comment:
This patch adds signed and unsigned long long support
to the array module. These types are already supported
by the struct module and use the same format characters
(q/Q).

Also corrects a minor bug in PyLong_AsUnsignedLongLong
which reports a BadInternalCall for arguments of
inappropriate type rather than a mere TypeError as
reported by the other conversion functions.


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

>Comment By: Oren Tirosh (orenti)
Date: 2005-04-07 23:53

Message:
Logged In: YES 
user_id=562624

My patch was against 2.4... (duh!)

The other bug is already fixed on 2.5.


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

Comment By: Armin Rigo (arigo)
Date: 2005-04-03 14:24

Message:
Logged In: YES 
user_id=4771

No two conversion function in longobject.c seem to have the same rules for what to do about non-long objects :-(  I'm afraid some clean-up would be useful, but also difficult for fear of breaking existing user C code :-(

In fact, your patch doesn't apply with today's CVS because someone already tried to add some magic in PyObject_AsLongLong().  It also fails on test_array.py and applies uncleanly on arraymodule.c.

Also, it needs to update the array module documentation.

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

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


More information about the Patches mailing list