Message165892
| Author |
meador.inge |
| Recipients |
Robert.Elsner, jcea, mark.dickinson, meador.inge, pitrou, r.david.murray, serhiy.storchaka |
| Date |
2012年07月20日.00:27:45 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1342744068.34.0.817081620296.issue14596@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
I just read through all this and see three separate points be discussed:
1. The unbounded caching behavior.
2. A more compact representation for repeat counts.
3. Correct __sizeof__ support for struct.
For issue (1) I think this is unfortunate, but I don't think any code changes are required because there is already a way to get the cached and uncached behavior by using the free function or a Struct object, respectively. I do think adjusting the documentation is appropriate.
As a side note, we do have a private function named 'struct._clearcache' that is used by the regression tests. If others really think the caching is a problem we could make that public.
Issues (2) and (3) are hijacking this issue IMO. However, since they are being discussed... I already implemented something like (2) when reworking the struct data structures for PEP 3118 in issue3132. Hopefully I will get the PEP 3118 patch pushed through one of these days.
I do think issue (3) should be fixed, but a separate issue should be opened for it. This issue should just address the caching behavior. Serhiy, if you open another issue for the __sizeof__ change, then I promise to review ASAP. |
|