[Patches] [ python-Patches-668124 ] struct.pack("I", -3) doesn't raise an exception

SourceForge.net noreply@sourceforge.net
Thu, 16 Jan 2003 02:26:36 -0800


Patches item #668124, was opened at 2003-01-14 21:39
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=668124&group_id=5470

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Thomas Heller (theller)
Assigned to: Nobody/Anonymous (nobody)
>Summary: struct.pack("I", -3) doesn't raise an exception

Initial Comment:
Each time I try to 'steal' code from Python, I seem to 
find a bug ;-)

This bug is also in 2.2.

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

>Comment By: Thomas Heller (theller)
Date: 2003-01-16 11:26

Message:
Logged In: YES 
user_id=11105

Don't know if this is a bug or not (and to be honest, I don't 
care). I just have the habit to report these things when I find 
them.

IMO it's at least inconsitent that all the unsigned format 
codes "HLQ" don't accept negative numbers, but "IL" do.


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

Comment By: Neal Norwitz (nnorwitz)
Date: 2003-01-15 23:21

Message:
Logged In: YES 
user_id=33168

This didn't raise an exception in 2.1.1 either.  Is this
really a bug or just the behaviour?  Should the fix be to update
doc rather than code?  Don't you need to fix get_ulonglong()
in a similar way?

I'm not necessarily against the patch, perhaps this should
be brought up on python-dev?

If the patch is accepted, you need to make a NEWS entry, and
probably should update libstruct.tex.

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

Comment By: Thomas Heller (theller)
Date: 2003-01-14 21:42

Message:
Logged In: YES 
user_id=11105

This is an upper-case 'i', meaning 'unsigned int'.

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

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