Skip to content
This repository has been archived by the owner on Dec 15, 2022. It is now read-only.

Cleaning: removed garbage symbols from regexes #123

Closed
wants to merge 1 commit into from

Conversation

zr9
Copy link
Contributor

@zr9 zr9 commented Jan 24, 2016

removed garbage symbols from regexes, and make them more nice look

@@ -219,7 +223,17 @@
'constants':
'patterns': [
{
'begin': '(?xi)(?=\n\t\t\t (\n\t\t\t (\\\\[a-z_][a-z_0-9]*\\\\[a-z_][a-z_0-9\\\\]*)|\n\t\t\t ([a-z_][a-z_0-9]*\\\\[a-z_][a-z_0-9\\\\]*)\n\t\t\t )\n\t\t\t [^a-z_0-9\\\\])'
'begin': '''
(?xi)
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you move this (?xi) to the above line? So 'begin': '''(?xi)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup possible

@50Wliu
Copy link
Contributor

50Wliu commented Feb 5, 2016

In general, please move the (?x) modifier on the same line as begin to save some space.

@Kroc Kroc mentioned this pull request May 22, 2016
@50Wliu 50Wliu mentioned this pull request Jun 8, 2017
@50Wliu 50Wliu closed this in #203 Jun 8, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants