safehasattr: pass attribute name as string instead of bytes
authorPierre-Yves David <pierre-yves.david@octobus.net>
Thu, 02 Feb 2023 17:19:46 +0100
changeset 50554 62a1ef6d2811
parent 50553 55886050a583
child 50555 b0e3f1508a25
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/chgserver.py
--- a/mercurial/chgserver.py	Thu Feb 02 17:19:35 2023 +0100
+++ b/mercurial/chgserver.py	Thu Feb 02 17:19:46 2023 +0100
@@ -262,7 +262,7 @@
     newui = srcui.__class__.load()
     for a in [b'fin', b'fout', b'ferr', b'environ']:
         setattr(newui, a, getattr(srcui, a))
-    if util.safehasattr(srcui, b'_csystem'):
+    if util.safehasattr(srcui, '_csystem'):
         newui._csystem = srcui._csystem
 
     # command line args