# HG changeset patch
# User Matt Harbison <matt_harbi...@yahoo.com>
# Date 1484192530 18000
# Wed Jan 11 22:42:10 2017 -0500
# Node ID bdbffc45e32a206ffca59ee9664feabab795aef4
# Parent f8d510b7f4bac4bbf519f098a4ca70e54165c960
revset: stop lowercasing the regex pattern for 'author'
It was probably unintentional for regex, as the meaning of some sequences like
\S and \s is actually inverted by changing the case. For backward compatibility
however, the matching is forced to case insensitive.
diff --git a/mercurial/revset.py b/mercurial/revset.py
--- a/mercurial/revset.py
+++ b/mercurial/revset.py
@@ -556,9 +556,9 @@
"""Alias for ``user(string)``.
"""
# i18n: "author" is a keyword
- n = encoding.lower(getstring(x, _("author requires a string")))
- kind, pattern, matcher = _substringmatcher(n)
- return subset.filter(lambda x: matcher(encoding.lower(repo[x].user())),
+ n = getstring(x, _("author requires a string"))
+ kind, pattern, matcher = _substringmatcher(n, casesensitive=False)
+ return subset.filter(lambda x: matcher(repo[x].user()),
condrepr=('<user %r>', n))
@predicate('bisect(string)', safe=True)
@@ -2246,10 +2246,15 @@
return subset.filter(matches, condrepr=('<subrepo %r>', pat))
-def _substringmatcher(pattern):
- kind, pattern, matcher = util.stringmatcher(pattern)
+def _substringmatcher(pattern, casesensitive=True):
+ kind, pattern, matcher = util.stringmatcher(pattern,
+ casesensitive=casesensitive)
if kind == 'literal':
- matcher = lambda s: pattern in s
+ if not casesensitive:
+ pattern = encoding.lower(pattern)
+ matcher = lambda s: pattern in encoding.lower(s)
+ else:
+ matcher = lambda s: pattern in s
return kind, pattern, matcher
@predicate('tag([name])', safe=True)
diff --git a/tests/test-revset.t b/tests/test-revset.t
--- a/tests/test-revset.t
+++ b/tests/test-revset.t
@@ -865,6 +865,17 @@
7
8
9
+ $ log 'author(r"re:\S")'
+ 0
+ 1
+ 2
+ 3
+ 4
+ 5
+ 6
+ 7
+ 8
+ 9
$ log 'branch(é)'
8
9
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
