Roundup Robot devn...@psf.upfronthosting.co.za 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
Antoine Pitrou pit...@free.fr added the comment:
Thanks for the reviews!
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
Antoine Pitrou pit...@free.fr added the comment:
Thanks for the comments. Here is an updated patch.
--
Added file: http://bugs.python.org/file23318/struct_nn4.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
Changes by Mark Dickinson dicki...@gmail.com:
--
assignee: mark.dickinson -
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
___
___
Meador Inge mead...@gmail.com added the comment:
No problem. This last version LGTM.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
___
Antoine Pitrou pit...@free.fr added the comment:
#3163 is a duplicate.
--
nosy: +pitrou
priority: low - normal
stage: patch review - needs patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
Antoine Pitrou pit...@free.fr added the comment:
Ooops, I meant #13098.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
___
___
Changes by Stefan Krah stefan-use...@bytereef.org:
--
nosy: +skrah
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
___
___
Python-bugs-list
Antoine Pitrou pit...@free.fr added the comment:
Here is a patch.
--
Added file: http://bugs.python.org/file23310/struct_nn.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
___
Meador Inge mead...@gmail.com added the comment:
Mostly LGTM. I have a few comments in rietveld.
--
nosy: +meador.inge
stage: needs patch - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
Antoine Pitrou pit...@free.fr 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 rep...@bugs.python.org
Antoine Pitrou pit...@free.fr added the comment:
New patch with cosmetic doc fix.
--
Added file: http://bugs.python.org/file23312/struct_nn3.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
Changes by Antoine Pitrou pit...@free.fr:
--
versions: +Python 3.3 -Python 3.2
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
___
___
Mark Dickinson dicki...@gmail.com added the comment:
Okay; would you be interested in updating your patch?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
___
Changes by Jean Brouwers mrje...@gmail.com:
Removed file: http://bugs.python.org/file18006/unnamed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
___
Jean Brouwers mrje...@gmail.com 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
http://code.activestate.com/recipes/546530/
in the function calcsize. That maps the 'z' and 'Z' format to
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
___
Changes by Mark Dickinson dicki...@gmail.com:
--
priority: normal - low
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
___
___
Jean Brouwers mrje...@gmail.com 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 rep...@bugs.python.orgwrote:
Mark Dickinson dicki...@gmail.com added the comment:
Jean, if you're still around:
Changes by Mark Dickinson dicki...@gmail.com:
--
assignee: - mark.dickinson
nosy: +mark.dickinson
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
___
Changes by Mark Dickinson dicki...@gmail.com:
--
versions: -Python 2.7
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
___
___
Mark Dickinson dicki...@gmail.com added the comment:
To avoid clashing with PEP 3118, we could use 'n' and 'N' instead of 'z' and
'Z'.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
Mark Dickinson dicki...@gmail.com 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
Changes by Daniel Diniz aja...@gmail.com:
--
priority: - normal
stage: - patch review
versions: +Python 3.2
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3163
___
Changes by Jean Brouwers [EMAIL PROTECTED]:
--
versions: +Python 2.5.3 -Python 2.7
___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue3163
___
___
Changes by Jean Brouwers [EMAIL PROTECTED]:
--
versions: +Python 2.7 -Python 2.5.3
___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue3163
___
___
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
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]
http://bugs.python.org/issue3163
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
Changes by Jean Brouwers [EMAIL PROTECTED]:
Removed file: http://bugs.python.org/file10687/struct_size_t.patch
___
Python tracker [EMAIL PROTECTED]
http://bugs.python.org/issue3163
___
30 matches
Mail list logo