[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