# HG changeset patch # User Pierre-Yves David # Date 2023-02-02 16:24:05 # Node ID 35e420abaef3e0d227e28421702db025c6463794 # Parent dbd57239acb81878f19600d7aa6be144a2e1ae0e 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. diff --git a/mercurial/dispatch.py b/mercurial/dispatch.py --- a/mercurial/dispatch.py +++ b/mercurial/dispatch.py @@ -734,7 +734,7 @@ class cmdalias: self.name, self.definition, ) - if util.safehasattr(self, b'shell'): + if util.safehasattr(self, 'shell'): return self.fn(ui, *args, **opts) else: try: