safehasattr: pass attribute name as string instead of bytes
authorPierre-Yves David <pierre-yves.david@octobus.net>
Thu, 02 Feb 2023 17:22:55 +0100
changeset 50563 8020be6a9d28
parent 50562 87b004378fc4
child 50564 1ea84989c389
safehasattr: pass attribute name as string instead of bytes This is a step toward replacing `util.safehasattr` usage with plain `hasattr`. The builtin function behave poorly in Python2 but this was fixed in Python3. These change are done one by one as they tend to have a small odd to trigger puzzling breackage.
mercurial/debugcommands.py
--- a/mercurial/debugcommands.py	Thu Feb 02 17:21:45 2023 +0100
+++ b/mercurial/debugcommands.py	Thu Feb 02 17:22:55 2023 +0100
@@ -1278,7 +1278,7 @@
     if opts.get(b'old'):
 
         def doit(pushedrevs, remoteheads, remote=remote):
-            if not util.safehasattr(remote, b'branches'):
+            if not util.safehasattr(remote, 'branches'):
                 # enable in-client legacy support
                 remote = localrepo.locallegacypeer(remote.local())
                 if remote_revs: