This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author meador.inge
Recipients asvetlov, gregory.p.smith, jcea, loewis, mark.dickinson, meador.inge, serhiy.storchaka, skrah
Date 2012-07-28.16:35:18
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <CAK1Qooru=90HorCcJO+bUpvfjcBy5FyN_8BvsjLM8-3Gkqyvqw@mail.gmail.com>
In-reply-to <201207281126.42753.storchaka@gmail.com>
Content
On Sat, Jul 28, 2012 at 3:27 AM, Serhiy Storchaka
<report@bugs.python.org> wrote:

> Serhiy Storchaka <storchaka@gmail.com> added the comment:
>
>> issue15456
>> efficiently demonstrates that the current style can detect bugs
>> which testing with object.__sizeof__ can't.
>
> Hmm, I see this as a counterexample. The bug *was not detected* with the
> current style of testing.

I disagree.  It wasn't *directly* detected -- the test broke because of the new
structure field that was added and not because of the new dynamic memory
allocation that was added.  Even so, the fact that the test broke *at all*
should have thrown a warning flag up in the developer's mind to reevaluate
how the size is calculated.  If the test were written using your
object.__sizeof__
method, then the test would not have broken *at all* and therefore it might not
have even crossed the developers mind to verify whether the sizeof calculation
is correct.
History
Date User Action Args
2012-07-28 16:35:19meador.ingesetrecipients: + meador.inge, loewis, gregory.p.smith, jcea, mark.dickinson, asvetlov, skrah, serhiy.storchaka
2012-07-28 16:35:18meador.ingelinkissue15402 messages
2012-07-28 16:35:18meador.ingecreate