D1494: develwarn: do not emit warning if "config" is unspecified
spectral (Kyle Lippincott)
phabricator at mercurial-scm.org
Thu Nov 23 03:35:26 UTC 2017
spectral created this revision.
spectral added a reviewer: yuja.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
Previously, if the develwarn call site did not specify the category of warning,
and devel.all-warnings was False, it would emit the warning. If it was
intended that this emit a warning if config is unspecified, I would have
expected a comment, so I assumed this was unintentional and am changing the
behavior.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D1494
AFFECTED FILES
mercurial/ui.py
CHANGE DETAILS
diff --git a/mercurial/ui.py b/mercurial/ui.py
--- a/mercurial/ui.py
+++ b/mercurial/ui.py
@@ -1601,7 +1601,7 @@
stack.
"""
if not self.configbool('devel', 'all-warnings'):
- if config is not None and not self.configbool('devel', config):
+ if config is None or not self.configbool('devel', config):
return
msg = 'devel-warn: ' + msg
stacklevel += 1 # get in develwarn
To: spectral, yuja, #hg-reviewers
Cc: mercurial-devel
More information about the Mercurial-devel
mailing list