[Python-Dev] Re: [Python-checkins] CVS: python/dist/src/Python ceval.c,2.287,2.288

[Barry A. Warsaw]

[Followups redirected to python-dev... -BAW]

>>>>> "MAL" == M  <lemburg@users.sourceforge.net> writes:

    | Modified Files:
    | 	ceval.c 
    | Log Message:
    | Add fast-path for comparing interned (true) string objects. 

    MAL> This patch boosts performance for comparing identical string
    MAL> object by some 20% on my machine while not causing any
    MAL> noticable slow-down for other operations (according to tests
    MAL> done with pybench).

>>>>> "JH" == Jeremy Hylton <jeremy@zope.com> writes:

    JH> Hey!  Don't do that.

I had a similar reaction! :)

    JH> The last time this came up, I thought there was a pretty clear
    JH> conclusion that we did not want to make thise change.

Tim's objection is here:

    http://mail.python.org/pipermail/python-dev/2001-October/018274.html

Since there isn't concensus, I don't think it should go into Python
2.2 at this late date.

Please back this one out MAL!

-Barry