gh-62142: IDLE - rework debugger UI by roseman · Pull Request #22947 · python/cpython (original) (raw)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Special chars are normal inside [] without \, so I think [^\.] and [^.] have the same effect (I prefer the latter) so that [^\\.] is needed to match 3 all 3 chars. Prefix r suppress strings escapes, but there are none, so I think it has no effect, though some prefer to use it for all REs. Outside the brackets, '\.' matches literal period, which is what want, whereas '\\.' matches multiple backslashes? I am not sure why later periods are excluded. CPython now prints addresses with A-F.

In any case, non-trivial REs should be defined at the top of the file so that they can be tested.