classification
Title: sre_compile._optimize_unicode() needs a cleanup
Type: behavior Stage: committed/rejected
Components: Library (Lib), Regular Expressions, Unicode Versions: Python 3.3
process
Status: closed Resolution: wont fix
Dependencies: Superseder:
Assigned To: Nosy List: ezio.melotti, haypo, mrabarnett, pitrou
Priority: normal Keywords:

Created on 2011-10-04 17:11 by haypo, last changed 2013-01-03 04:40 by ezio.melotti. This issue is now closed.

Messages (2)
msg144905 - (view) Author: STINNER Victor (haypo) * (Python committer) Date: 2011-10-04 17:11
The following comment is wrong, 

    except IndexError:
        # non-BMP characters; XXX now they should work
        return charset

sys.maxunicode != 65535 is now always true in Python 3.3

        if sys.maxunicode != 65535:
            # XXX: negation does not work with big charsets
            # XXX2: now they should work, but removing this will make the
            # charmap 17 times bigger
            return charset

See the related commit: f39b26ca7f3d (from issue #13054).
msg178896 - (view) Author: STINNER Victor (haypo) * (Python committer) Date: 2013-01-03 02:02
I don't know what to do with this issue. The code looks to work anyway, so I guess that it's safer to not touch it :-)
History
Date User Action Args
2013-01-03 04:40:30ezio.melottisetnosy: + mrabarnett

type: behavior
stage: committed/rejected
2013-01-03 02:02:38hayposetstatus: open -> closed
resolution: wont fix
messages: + msg178896
2011-10-04 17:11:58haypocreate