safehasattr: pass attribute name as string instead of bytes
authorPierre-Yves David <pierre-yves.david@octobus.net>
Thu, 02 Feb 2023 17:36:32 +0100
changeset 50604 b539c60a79ac
parent 50603 d1502345f89c
child 50605 98b805d40c41
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/subrepoutil.py
--- a/mercurial/subrepoutil.py	Thu Feb 02 17:36:25 2023 +0100
+++ b/mercurial/subrepoutil.py	Thu Feb 02 17:36:32 2023 +0100
@@ -384,7 +384,7 @@
     Either absolute or relative the outermost repo"""
     parent = repo
     chunks = []
-    while util.safehasattr(parent, b'_subparent'):
+    while util.safehasattr(parent, '_subparent'):
         source = urlutil.url(parent._subsource)
         chunks.append(bytes(source))
         if source.isabs():