[issue3163] module struct support for ssize_t and size_t

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Thanks for the reviews!

--
resolution:  -> fixed
stage: patch review -> committed/rejected
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Roundup Robot]

Roundup Robot  added the comment:

New changeset db3e15017172 by Antoine Pitrou in branch 'default':
Issue #3163: The struct module gets new format characters 'n' and 'N'
http://hg.python.org/cpython/rev/db3e15017172

--
nosy: +python-dev

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Meador Inge]

Meador Inge  added the comment:

No problem.  This last version LGTM.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Mark Dickinson]

Changes by Mark Dickinson :


--
assignee: mark.dickinson -> 

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Thanks for the comments. Here is an updated patch.

--
Added file: http://bugs.python.org/file23318/struct_nn4.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Meador Inge]

Meador Inge  added the comment:

Found a few test case nits.  Comments in rietveld.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Antoine Pitrou]

Antoine Pitrou  added the comment:

New patch with cosmetic doc fix.

--
Added file: http://bugs.python.org/file23312/struct_nn3.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Thanks. I have answered one of your comments, and here is a new patch.

--
Added file: http://bugs.python.org/file23311/struct_nn2.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Meador Inge]

Meador Inge  added the comment:

Mostly LGTM.  I have a few comments in rietveld.

--
nosy: +meador.inge
stage: needs patch -> patch review

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Here is a patch.

--
Added file: http://bugs.python.org/file23310/struct_nn.patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Stefan Krah]

Changes by Stefan Krah :


--
nosy: +skrah

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Ooops, I meant #13098.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Antoine Pitrou]

Antoine Pitrou  added the comment:

#3163 is a duplicate.

--
nosy: +pitrou
priority: low -> normal
stage: patch review -> needs patch

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Antoine Pitrou]

Changes by Antoine Pitrou :


--
versions: +Python 3.3 -Python 3.2

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Mark Dickinson]

Mark Dickinson  added the comment:

Okay;  would you be interested in updating your patch?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Jean Brouwers]

Jean Brouwers  added the comment:

It would be sufficient to support size_t/ssize_t in native mode for my usage.  
The workaround I have been using is in this recipe

  

in the function calcsize.  That maps the 'z' and 'Z' format to either 'l', 'L' 
or 'P' for lack of anything better.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Jean Brouwers]

Changes by Jean Brouwers :


Removed file: http://bugs.python.org/file18006/unnamed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Jean Brouwers]

Jean Brouwers  added the comment:

Yes I am and I will get back to you on this after check my (old) notes.

/Jean

On Wed, Jul 14, 2010 at 12:08 PM, Mark Dickinson wrote:

>
> Mark Dickinson  added the comment:
>
> Jean, if you're still around:
>
> Do you still have interest in pursuing this?
>
> --
>
> ___
> Python tracker 
> 
> ___
>

--
Added file: http://bugs.python.org/file18006/unnamed

___
Python tracker 

___Yes I am and I will get back to you on this after check my (old) 
notes./JeanOn Wed, Jul 14, 
2010 at 12:08 PM, Mark Dickinson rep...@bugs.python.org> 
wrote:

Mark Dickinson dicki...@gmail.com> added the 
comment:

Jean, if you're still around:

Do you still have interest in pursuing this?

--

___
Python tracker rep...@bugs.python.org>
http://bugs.python.org/issue3163>
___

___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Mark Dickinson]

Changes by Mark Dickinson :


--
priority: normal -> low

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Mark Dickinson]

Mark Dickinson  added the comment:

Jean, if you're still around:

Do you still have interest in pursuing this?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Mark Dickinson]

Mark Dickinson  added the comment:

The proposed addition seems reasonable to me, for native packing and unpacking.

For standard mode packing and unpacking, I don't see the point;  we'd have to 
pick a standard size, which would almost certainly be either 4 or 8, and so 
would already be covered by either the 'iI' or the 'qQ' codes.  So I'd suggest 
adding the size_t/ssize_t codes only for native mode, and leaving them 
unsupported for the standard modes.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Mark Dickinson]

Mark Dickinson  added the comment:

To avoid clashing with PEP 3118, we could use 'n' and 'N' instead of 'z' and 
'Z'.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Mark Dickinson]

Changes by Mark Dickinson :


--
versions:  -Python 2.7

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Mark Dickinson]

Changes by Mark Dickinson :


--
assignee:  -> mark.dickinson
nosy: +mark.dickinson

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Daniel Diniz]

Changes by Daniel Diniz :


--
priority:  -> normal
stage:  -> patch review
versions: +Python 3.2

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Jean Brouwers]

Changes by Jean Brouwers <[EMAIL PROTECTED]>:


--
versions: +Python 2.7 -Python 2.5.3

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Jean Brouwers]

Changes by Jean Brouwers <[EMAIL PROTECTED]>:


--
versions: +Python 2.5.3 -Python 2.7

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Jean Brouwers]

Changes by Jean Brouwers <[EMAIL PROTECTED]>:


Removed file: http://bugs.python.org/file10687/struct_size_t.patch

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Jean Brouwers]

Jean Brouwers <[EMAIL PROTECTED]> added the comment:

Please ignore the first patch.  Use the rev2 patch which includes an 
update for test file  Lib/test/test_struct.py as well.

Added file: http://bugs.python.org/file10690/struct_size_t.rev2.patch

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Martin v. Löwis]

Martin v. Löwis <[EMAIL PROTECTED]> added the comment:

Now that Python 2.6 is in beta, new features should be deferred to 2.7.

--
nosy: +loewis
versions: +Python 2.7 -Python 2.6

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue3163] module struct support for ssize_t and size_t

[Jean Brouwers]

New submission from Jean Brouwers <[EMAIL PROTECTED]>:

Attached is a (simplistic) patch for the struct module to support C types 
ssize_t and size_t with format characters 'z' resp. 'Z'.

The patch updates Python 2.6b1 files Modules/_struct.c and 
Doc/library/struct.rst.

Note, format character 'Z' is also proposed in PEP 3118 for C type complex  
and that has to be something else, perhaps 'C'.

--
components: Library (Lib)
files: struct_size_t.patch
keywords: patch
messages: 68528
nosy: MrJean1
severity: normal
status: open
title: module struct support for ssize_t and size_t
type: feature request
versions: Python 2.6
Added file: http://bugs.python.org/file10687/struct_size_t.patch

___
Python tracker <[EMAIL PROTECTED]>

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com