[Changed Subscribers] D8640: ui: add option to timestamp status and diagnostic messages
pulkit (Pulkit Goyal)
phabricator at mercurial-scm.org
Thu Jun 25 14:56:19 UTC 2020
pulkit added inline comments.
INLINE COMMENTS
> ui.py:1737
> if not self.quiet:
> - self._writemsg(self._fmsgout, type=b'status', *msg, **opts)
> + self._writemsg(self._fmsgout, type=b'status', timestamp=self.showtimestamp, *msg, **opts)
>
Any reason why we are passing `self.showtimestamp` when it can be accessed directly in `self._writemsg`?
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST ACTION
https://phab.mercurial-scm.org/D8640/new/
REVISION DETAIL
https://phab.mercurial-scm.org/D8640
To: joerg.sonnenberger, #hg-reviewers
Cc: pulkit, durin42, mercurial-patches
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mercurial-scm.org/pipermail/mercurial-patches/attachments/20200625/dbb5aba2/attachment-0002.html>
More information about the Mercurial-patches
mailing list