[issue30349] Preparation for advanced set syntax in regular expressions

2021-09-21 Thread Philippe Ombredanne
Philippe Ombredanne added the comment: Sorry, my comment was at best nonsensical gibberish! I meant to say that this warning message should include the actual regex at fault; otherwise it is hard to fix when the regex in question comes from some data structure like a list; then the line

[issue30349] Preparation for advanced set syntax in regular expressions

2021-09-21 Thread Philippe Ombredanne
Philippe Ombredanne added the comment: FWIW, this warning is annoying because it is hard to fix in the case where the regex are source from data: the warning message does not include the regex at fault; it should otherwise the warning is noisy and ineffective IMHO. -- nosy:

[issue30349] Preparation for advanced set syntax in regular expressions

2018-02-05 Thread Tim Graham
Tim Graham added the comment: Okay, I created #32775. -- ___ Python tracker ___ ___

[issue30349] Preparation for advanced set syntax in regular expressions

2018-02-05 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Good catch! fnmatch.translate() can produce a pattern which emits a warning when compiled. Could you please open a separate issue for this? -- ___ Python tracker

[issue30349] Preparation for advanced set syntax in regular expressions

2018-02-05 Thread Tim Graham
Tim Graham added the comment: It might be worth adding part of the problematic regex to the warning message. For Django's tests, I see an error like "FutureWarning: Possible nested set at position 17 return re.compile(res).match". It took some effort to track down the

[issue30349] Preparation for advanced set syntax in regular expressions

2017-11-16 Thread Serhiy Storchaka
Change by Serhiy Storchaka : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue30349] Preparation for advanced set syntax in regular expressions

2017-11-16 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: New changeset 05cb728d68a278d11466f9a6c8258d914135c96c by Serhiy Storchaka in branch 'master': bpo-30349: Raise FutureWarning for nested sets and set operations (#1553)

[issue30349] Preparation for advanced set syntax in regular expressions

2017-10-05 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Made a warning for '[' be emitted only at the start of a set. This significantly decrease the breakage of other code. I think we can get around without implicit union of nested sets, like in [_[0-9][:Latin:]]. This can be

[issue30349] Preparation for advanced set syntax in regular expressions

2017-05-12 Thread Serhiy Storchaka
Changes by Serhiy Storchaka : -- pull_requests: +1650 ___ Python tracker ___ ___

[issue30349] Preparation for advanced set syntax in regular expressions

2017-05-12 Thread Serhiy Storchaka 
New submission from Serhiy Storchaka:

Currently the re module supports only simple sets. They can include literal 
characters, character ranges, some simple character classes and support the 
negation. The Unicode standard [1] defines set operations (union, intersection, 
difference and