Mercurial > public > mercurial-scm > hg-stable
diff mercurial/scmutil.py @ 41144:7b80406b8271
progress: move cached debug flag from progress.progbar to scmutil.progress
It's simpler this way. One possible drawback (and a possisble
advantage) is that we now check the debug flag once per topic, so
processes that generate new topics all the time will still check the
flag frequently.
Differential Revision: https://phab.mercurial-scm.org/D5528
| author | Martin von Zweigbergk <martinvonz@google.com> |
|---|---|
| date | Tue, 08 Jan 2019 00:12:08 -0800 |
| parents | 8cf92ca92bfe |
| children | 3025fd3c2e71 |
line wrap: on
line diff
--- a/mercurial/scmutil.py Mon Jan 07 23:55:26 2019 -0800 +++ b/mercurial/scmutil.py Tue Jan 08 00:12:08 2019 -0800 @@ -1421,6 +1421,7 @@ self.topic = topic self.unit = unit self.total = total + self.debug = ui.configbool('progress', 'debug') def __enter__(self): return self @@ -1458,14 +1459,7 @@ self.ui._progbar.progress(self.topic, self.pos, item=item, unit=self.unit, total=self.total) - # Looking up progress.debug in tight loops is expensive. The value - # is cached on the progbar object and we can avoid the lookup in - # the common case where a progbar is active. - if self.pos is None or not self.ui._progbar.debug: - return - - # Keep this logic in sync with above. - if self.pos is None or not self.ui.configbool('progress', 'debug'): + if self.pos is None or not self.debug: return if self.unit: