# HG changeset patch # User Yuya Nishihara # Date 1524984721 -32400 # Node ID 3fe1c92630240b4483b5bc558304c8b9d6c889df # Parent e0f30c91dfd8e02235b27d253a4d6a79cf976a77 log: consume --stat/patch options at constructor of changesetprinter The variable name, self.diffopts, was confusing. Let's split it to two booleans. diff -r e0f30c91dfd8 -r 3fe1c9263024 mercurial/logcmdutil.py --- a/mercurial/logcmdutil.py Sun Apr 29 15:44:17 2018 +0900 +++ b/mercurial/logcmdutil.py Sun Apr 29 15:52:01 2018 +0900 @@ -155,7 +155,8 @@ self.buffered = buffered self._differ = differ or changesetdiffer() self._diffopts = patch.diffallopts(ui, diffopts) - self.diffopts = diffopts or {} + self._includestat = diffopts and diffopts.get('stat') + self._includediff = diffopts and diffopts.get('patch') self.header = {} self.hunk = {} self.lastheader = None @@ -299,15 +300,13 @@ ''' def _showpatch(self, ctx): - stat = self.diffopts.get('stat') - diff = self.diffopts.get('patch') - if stat: + if self._includestat: self._differ.showdiff(self.ui, ctx, self._diffopts, stat=True) - if stat and diff: + if self._includestat and self._includediff: self.ui.write("\n") - if diff: + if self._includediff: self._differ.showdiff(self.ui, ctx, self._diffopts, stat=False) - if stat or diff: + if self._includestat or self._includediff: self.ui.write("\n") class changesetformatter(changesetprinter): @@ -368,13 +367,11 @@ fm.data(copies=fm.formatdict(copies, key='name', value='source')) - stat = self.diffopts.get('stat') - diff = self.diffopts.get('patch') - if stat: + if self._includestat: self.ui.pushbuffer() self._differ.showdiff(self.ui, ctx, self._diffopts, stat=True) fm.data(diffstat=self.ui.popbuffer()) - if diff: + if self._includediff: self.ui.pushbuffer() self._differ.showdiff(self.ui, ctx, self._diffopts, stat=False) fm.data(diff=self.ui.popbuffer())