D2457: py3: slice over bytes or use .startswith() to prevent getting ascii values

pulkit (Pulkit Goyal) phabricator at mercurial-scm.org
Mon Feb 26 13:53:13 EST 2018


pulkit updated this revision to Diff 6127.
pulkit retitled this revision from "py3: slice over bytes to prevent getting ascii values" to "py3: slice over bytes or use .startswith() to prevent getting ascii values".

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D2457?vs=6112&id=6127

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

AFFECTED FILES
  mercurial/archival.py
  mercurial/patch.py

CHANGE DETAILS

diff --git a/mercurial/patch.py b/mercurial/patch.py
--- a/mercurial/patch.py
+++ b/mercurial/patch.py
@@ -2518,7 +2518,7 @@
                                 yield (t, l)
                         else:
                             for token in tabsplitter.findall(stripline):
-                                if '\t' == token[0]:
+                                if token.startswith('\t'):
                                     yield (token, 'diff.tab')
                                 else:
                                     yield (token, label)
diff --git a/mercurial/archival.py b/mercurial/archival.py
--- a/mercurial/archival.py
+++ b/mercurial/archival.py
@@ -154,7 +154,7 @@
 
         def taropen(mode, name='', fileobj=None):
             if kind == 'gz':
-                mode = mode[0]
+                mode = mode[0:1]
                 if not fileobj:
                     fileobj = open(name, mode + 'b')
                 gzfileobj = self.GzipFileWithTime(name, mode + 'b',



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


More information about the Mercurial-devel mailing list