Patchwork D2472: pycompat: prevent encoding or decoding values if not required

login
register
mail settings
Submitter phabricator
Date Feb. 27, 2018, 5:09 p.m.
Message ID <ecf307728e98ea4db6ec957577474b17@localhost.localdomain>
Download mbox | patch
Permalink /patch/28461/
State Not Applicable
Headers show

Comments

phabricator - Feb. 27, 2018, 5:09 p.m.
pulkit updated this revision to Diff 6176.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D2472?vs=6152&id=6176

REVISION DETAIL
  https://phab.mercurial-scm.org/D2472

AFFECTED FILES
  mercurial/pycompat.py

CHANGE DETAILS




To: pulkit, #hg-reviewers, yuja
Cc: yuja, mercurial-devel

Patch

diff --git a/mercurial/pycompat.py b/mercurial/pycompat.py
--- a/mercurial/pycompat.py
+++ b/mercurial/pycompat.py
@@ -192,11 +192,15 @@ 
 
     def strurl(url):
         """Converts a bytes url back to str"""
-        return url.decode(u'ascii')
+        if isinstance(url, bytes):
+            return url.decode(u'ascii')
+        return url
 
     def bytesurl(url):
         """Converts a str url to bytes by encoding in ascii"""
-        return url.encode(u'ascii')
+        if isinstance(url, str):
+            return url.encode(u'ascii')
+        return url
 
     def raisewithtb(exc, tb):
         """Raise exception with the given traceback"""