encoding: make sure "wide" variable never be referenced from other modules
authorYuya Nishihara <yuya@tcha.org>
Mon, 29 May 2017 21:57:51 +0900
changeset 32570 044f3d7eb9ae
parent 32569 aa333c1982ab
child 32571 2dd8d4108249
encoding: make sure "wide" variable never be referenced from other modules Better to not expose (maybe-) unicode objects.
mercurial/encoding.py
--- a/mercurial/encoding.py	Sun May 28 21:29:58 2017 -0400
+++ b/mercurial/encoding.py	Mon May 29 21:57:51 2017 +0900
@@ -194,8 +194,8 @@
                    for k, v in os.environ.items())  # re-exports
 
 # How to treat ambiguous-width characters. Set to 'wide' to treat as wide.
-wide = _sysstr(environ.get("HGENCODINGAMBIGUOUS", "narrow") == "wide"
-               and "WFA" or "WF")
+_wide = _sysstr(environ.get("HGENCODINGAMBIGUOUS", "narrow") == "wide"
+                and "WFA" or "WF")
 
 def colwidth(s):
     "Find the column width of a string for display in the local encoding"
@@ -205,7 +205,7 @@
     "Find the column width of a Unicode string for display"
     eaw = getattr(unicodedata, 'east_asian_width', None)
     if eaw is not None:
-        return sum([eaw(c) in wide and 2 or 1 for c in d])
+        return sum([eaw(c) in _wide and 2 or 1 for c in d])
     return len(d)
 
 def getcols(s, start, c):