[Python-checkins] r42501 - python/trunk/Include/code.h

Tim Peters tim.peters at gmail.com
Sun Feb 19 22:29:34 CET 2006


[Neal]
> Hmm, this is only used in Include/pythonrun.h and Python/bltinmodule.c
> in PyCF_MASK_OBSOLETE.  Would it be ok to remove CO_GENERATOR_ALLOWED
> from the mask and disallow the flag?  I'm a little hesitant, though
> the flag hasn't been necessary since 2.2.  It was only necessary in
> 2.1 IIRC.
>
> Should I just restore the comment and not worry about this until 3.0?
> The comment was a bit misleading until I re-read it, so if I restore
> it I will update it (and add when it should be removed).

My only point is that the hack and the comment describing it should
both go or both stay.  It's better if they both go (because cruft
reduction is a Good Thing), but whether they _can_ go away now
requires some thought, which I'm unable to give it now.  Try it!


More information about the Python-checkins mailing list