Re: [PATCH] ui: add configlist doctest to document a bit more of the whitespace behavior

2017-11-02 Thread Yuya Nishihara 
On Wed, 01 Nov 2017 19:31:49 -0400, Augie Fackler wrote:
> # HG changeset patch
> # User Augie Fackler 
> # Date 1508986914 14400
> #  Wed Oct 25 23:01:54 2017 -0400
> # Node ID 7e5b4ce0c445d4fc8f906f6926c71163a27c85a2
> # Parent  3ce0e4b51f789eff195ec900a07c1fa5e8d5c5f2
> ui: add configlist doctest to document a bit more of the whitespace behavior

Queued, thanks.
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

[PATCH] ui: add configlist doctest to document a bit more of the whitespace behavior

2017-11-01 Thread Augie Fackler 
# HG changeset patch
# User Augie Fackler 
# Date 1508986914 14400
#  Wed Oct 25 23:01:54 2017 -0400
# Node ID 7e5b4ce0c445d4fc8f906f6926c71163a27c85a2
# Parent  3ce0e4b51f789eff195ec900a07c1fa5e8d5c5f2
ui: add configlist doctest to document a bit more of the whitespace behavior

I'm surprised this wasn't explicitly tested, so I added a test.

diff --git a/mercurial/ui.py b/mercurial/ui.py
--- a/mercurial/ui.py
+++ b/mercurial/ui.py
@@ -695,6 +695,9 @@ class ui(object):
 >>> u.setconfig(s, b'list1', b'this,is "a small" ,test')
 >>> u.configlist(s, b'list1')
 ['this', 'is', 'a small', 'test']
+>>> u.setconfig(s, b'list2', b'this, is "a small" , test ')
+>>> u.configlist(s, b'list2')
+['this', 'is', 'a small', 'test']
 """
 # default is not always a list
 v = self.configwith(config.parselist, section, name, default,
___
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel