# HG changeset patch # User Pierre-Yves David # Date 1675355000 -3600 # Node ID 309a7ecef246b6dde626e6b970e739e6f3dda29a # Parent 2b1cd454793a4465d57f53c9e4eb0af0adac36d1 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 -r 2b1cd454793a -r 309a7ecef246 mercurial/dispatch.py --- a/mercurial/dispatch.py Thu Feb 02 17:23:12 2023 +0100 +++ b/mercurial/dispatch.py Thu Feb 02 17:23:20 2023 +0100 @@ -107,7 +107,7 @@ def _flushstdio(ui, err): status = None # In all cases we try to flush stdio streams. - if util.safehasattr(ui, b'fout'): + if util.safehasattr(ui, 'fout'): assert ui is not None # help pytype assert ui.fout is not None # help pytype try: