[issue33204] IDLE: remove \b from colorizer string prefix

Change by Terry J. Reedy : -- status: open -> closed ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue33204] IDLE: remove \b from colorizer string prefix

Change by Terry J. Reedy : -- resolution: -> fixed stage: patch review -> resolved ___ Python tracker ___ ___ Python-bugs-list

[issue33204] IDLE: remove \b from colorizer string prefix

Tal Einat added the comment: Should this be closed as resolved? -- nosy: +taleinat ___ Python tracker ___ ___ Python-bugs-list

[issue33204] IDLE: remove \b from colorizer string prefix

miss-islington added the comment: New changeset 306559e6ca15b86eb230609f484f48132b7ca383 by Miss Islington (bot) in branch '3.6': bpo-33204: IDLE - revise and extend colorizer test. (GH-6347)

[issue33204] IDLE: remove \b from colorizer string prefix

miss-islington added the comment: New changeset fa91aff07c16ad48a63580c2909934be92a5e9c8 by Miss Islington (bot) in branch '3.7': bpo-33204: IDLE - revise and extend colorizer test. (GH-6347)

[issue33204] IDLE: remove \b from colorizer string prefix

Change by miss-islington : -- pull_requests: +6059 ___ Python tracker ___

[issue33204] IDLE: remove \b from colorizer string prefix

Change by miss-islington : -- pull_requests: +6058 ___ Python tracker ___

[issue33204] IDLE: remove \b from colorizer string prefix

Terry J. Reedy added the comment: New changeset 55966f3a0d5f1bf823bd22ce1abbde267b06552f by Terry Jan Reedy in branch 'master': bpo-33204: IDLE - revise and extend colorizer test. (GH-6347) https://github.com/python/cpython/commit/55966f3a0d5f1bf823bd22ce1abbde267b06552f

[issue33204] IDLE: remove \b from colorizer string prefix

Change by Terry J. Reedy : -- pull_requests: +6057 ___ Python tracker ___ ___

[issue33204] IDLE: remove \b from colorizer string prefix

Terry J. Reedy added the comment: Yes, still correct, as a keyword. Legal "x or''" is colored as "x" "or" "''" . Legal "if'':" is colored as "if" "''" ":" ". New PR revises colorizer test and add the above. -- ___ Python

[issue33204] IDLE: remove \b from colorizer string prefix

Serhiy Storchaka added the comment: Is ''or'' correctly colorized? -- nosy: +serhiy.storchaka ___ Python tracker ___

[issue33204] IDLE: remove \b from colorizer string prefix

miss-islington added the comment: New changeset 16cf84b4fbe78f9d876e0335f33459f1b92b7bf0 by Miss Islington (bot) in branch '3.6': bpo-33204: IDLE: consistently color invalid string prefixes (GH-6344)

[issue33204] IDLE: remove \b from colorizer string prefix

miss-islington added the comment: New changeset 6130675efeb3c23cebc43ab07c49eb62cd03cacf by Miss Islington (bot) in branch '3.7': bpo-33204: IDLE: consistently color invalid string prefixes (GH-6344)

[issue33204] IDLE: remove \b from colorizer string prefix

Change by miss-islington : -- pull_requests: +6056 ___ Python tracker ___

[issue33204] IDLE: remove \b from colorizer string prefix

Change by miss-islington : -- pull_requests: +6055 ___ Python tracker ___

[issue33204] IDLE: remove \b from colorizer string prefix

Terry J. Reedy added the comment: New changeset da58533ac67b01ce8f6466e6f03ff6b8b3bb04d5 by Terry Jan Reedy in branch 'master': bpo-33204: IDLE: consistently color invalid string prefixes (#6344)

[issue33204] IDLE: remove \b from colorizer string prefix

Change by Terry J. Reedy : -- stage: test needed -> patch review ___ Python tracker ___

[issue33204] IDLE: remove \b from colorizer string prefix

Terry J. Reedy added the comment: To see the visually verified test, run Lib/idlelib/colorizer as the main module, either from a command line or editor, and click the button. -- stage: patch review -> test needed ___ Python

[issue33204] IDLE: remove \b from colorizer string prefix

Change by Terry J. Reedy : -- keywords: +patch pull_requests: +6054 stage: test needed -> patch review ___ Python tracker ___

[issue33204] IDLE: remove \b from colorizer string prefix

Tim Peters added the comment: Sounds good (removing \b) to me, Terry! -- nosy: +tim.peters ___ Python tracker ___

[issue33204] IDLE: remove \b from colorizer string prefix

Terry J. Reedy added the comment: The current test is part of the htest. The section # All valid prefixes for unicode and byte strings should be colored contains "ur'is invalid'". This should be moved to a new section with other invalid combinations. # Invalid combinations

[issue33204] IDLE: remove \b from colorizer string prefix

New submission from Terry J. Reedy : Lib\idlelib\colorizer.py, line 25, is: stringprefix = r"(?i:\br|u|f|fr|rf|b|br|rb)?" The r prefix, but only the r prefix, must be preceded by a non-word character. On pydev thread "IDLE colorizer", MRAB noted: "The \b will apply only to