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 blaisorblade
Recipients ajaksu2, blaisorblade, collinwinter, jyasskin, pitrou, skip.montanaro
Date 2009-01-17.00:16:43
SpamBayes Score 0.04951356
Marked as misclassified No
Message-id <1232151405.76.0.556519707098.issue4896@psf.upfronthosting.co.za>
In-reply-to
Content
Given a 10% speedup on some systems, and statistically insignificant
changes on other systems, I would still apply the patch, even simply
because the bitmask part simply makes more sense.

I'm not sure about the goto part, but still, it does straighten the
code. Anyway, simply call the label "why_is_WHY_NOT",
"why_set_to_WHY_NOT" or something like that. Verbosity on a use-once
label used with goto should be encouraged - we're not Java programmer,
but we need to pay for using goto by increasing readability in other ways.

@collinwinter: since the differences you report are so low (similar to
the statistical noise I get on my machine), I would expect that you're
just getting statistical noise instead of different results depending on
the GCC version, unless you performed statistical hypothesis testing
(confidence intervals and related stuff). And if I had done the needed
tens/hundreds of repetitions for hypothesis testing, I'd state that
clearly, so I suppose you didn't, and that's fully acceptable since the
result is likely to be statistically insignificant anyway.
History
Date User Action Args
2009-01-17 00:16:45blaisorbladesetrecipients: + blaisorblade, skip.montanaro, collinwinter, pitrou, ajaksu2, jyasskin
2009-01-17 00:16:45blaisorbladesetmessageid: <1232151405.76.0.556519707098.issue4896@psf.upfronthosting.co.za>
2009-01-17 00:16:44blaisorbladelinkissue4896 messages
2009-01-17 00:16:43blaisorbladecreate