Message165936
| Author |
meador.inge |
| Recipients |
jcea, mark.dickinson, meador.inge, serhiy.storchaka |
| Date |
2012年07月20日.15:13:05 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1342797186.36.0.0944796335813.issue15402@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
Module a few cosmetic changes (variable names and doc string tweaks), the patch looks good. Having a correct answer for `sys.getsizeof(struct.Struct('100B'))` is definitely better. Per the documentation for 'sys.getsizeof' [1]:
"""
All built-in objects will return correct results, but this does not have to hold true for third-party extensions as it is implementation specific.
"""
If we consider extension modules as built-in objects, then this is a bug and should be fixed in 2.7, 3.2, and 3.3. If we don't, then the documentation for 'sys.getsizeof' should be adjusted and this patch should be applied after the default branch is unfrozen for enhancements.
[1] http://docs.python.org/library/sys.html |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2012年07月20日 15:13:06 | meador.inge | set | recipients:
+ meador.inge, jcea, mark.dickinson, serhiy.storchaka |
| 2012年07月20日 15:13:06 | meador.inge | set | messageid: <1342797186.36.0.0944796335813.issue15402@psf.upfronthosting.co.za> |
| 2012年07月20日 15:13:05 | meador.inge | link | issue15402 messages |
| 2012年07月20日 15:13:05 | meador.inge | create |
|