Patchwork [01,of,10] revset: add braces after wdir since its a function

login
register
mail settings
Submitter Pulkit Goyal
Date May 22, 2017, 8:52 p.m.
Message ID <4f81eb36137f66f8ce2a.1495486331@workspace>
Download mbox | patch
Permalink /patch/20842/
State Accepted
Headers show

Comments

Pulkit Goyal - May 22, 2017, 8:52 p.m.
# HG changeset patch
# User Pulkit Goyal <7895pulkit@gmail.com>
# Date 1495394642 -19800
#      Mon May 22 00:54:02 2017 +0530
# Node ID 4f81eb36137f66f8ce2a271404255fa6e0d84634
# Parent  e8c043375b53b30c4b468687f08323cbeeb452ef
revset: add braces after wdir since its a function

wdir was shown in help rather than wdir().
Yuya Nishihara - May 23, 2017, 2:51 p.m.
On Tue, 23 May 2017 02:22:11 +0530, Pulkit Goyal wrote:
> # HG changeset patch
> # User Pulkit Goyal <7895pulkit@gmail.com>
> # Date 1495394642 -19800
> #      Mon May 22 00:54:02 2017 +0530
> # Node ID 4f81eb36137f66f8ce2a271404255fa6e0d84634
> # Parent  e8c043375b53b30c4b468687f08323cbeeb452ef
> revset: add braces after wdir since its a function

Queued 1-9, thanks.

s/braces/parenthesis/, I think. Fixed in flight.

Patch

diff --git a/mercurial/revset.py b/mercurial/revset.py
--- a/mercurial/revset.py
+++ b/mercurial/revset.py
@@ -2130,7 +2130,7 @@ 
     """
     return author(repo, subset, x)
 
-@predicate('wdir', safe=True)
+@predicate('wdir()', safe=True)
 def wdir(repo, subset, x):
     """Working directory. (EXPERIMENTAL)"""
     # i18n: "wdir" is a keyword