[PATCH 1 of 9] formatter: remove unused private attributes from baseformatter
Gregory Szorc
gregory.szorc at gmail.com
Thu Apr 12 12:48:17 EDT 2018
On Thu, Apr 12, 2018 at 9:17 AM, Yuya Nishihara <yuya at tcha.org> wrote:
> # HG changeset patch
> # User Yuya Nishihara <yuya at tcha.org>
> # Date 1523543073 -32400
> # Thu Apr 12 23:24:33 2018 +0900
> # Node ID e6609e390f284931c10f06f3c8258769fad9bbc5
> # Parent 9c7a25ef5b49f3ce2fe698b58246b622ca9fd382
> formatter: remove unused private attributes from baseformatter
>
Queued this series with great delight, thanks!
This is infinitely better than my hacky series which was attempting to add
CBOR support. Thank you for coding this up.
>
> No idea what they were for.
>
AFAICT they are as old as this class and weren't even used when the class
was introduced! (3c0327ea20c0).
>
> diff --git a/mercurial/formatter.py b/mercurial/formatter.py
> --- a/mercurial/formatter.py
> +++ b/mercurial/formatter.py
> @@ -160,8 +160,6 @@ class baseformatter(object):
> def __init__(self, ui, topic, opts, converter):
> self._ui = ui
> self._topic = topic
> - self._style = opts.get("style")
> - self._template = opts.get("template")
> self._converter = converter
> self._item = None
> # function to convert node to string suitable for this output
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at mercurial-scm.org
> https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20180412/a78532a8/attachment.html>
More information about the Mercurial-devel
mailing list