[PATCH 7 of 7] verify: get rid of some unnecessary local variables
Martin von Zweigbergk
martinvonz at google.com
Tue Jan 5 23:31:07 CST 2016
On Tue, Jan 5, 2016 at 9:19 PM Martin von Zweigbergk <martinvonz at google.com>
wrote:
> Looks good, although I've mostly reviewed the subject lines only (and
> tests pass :-)) . Seems like a nice improvement to readability too. Pushed
> to the clowncopter!
>
After this series, _verifymanifest() and _verifychangelog() accept
some dictionaries as "output parameters". Do your extensions rely on that?
If not, I'll send a followup patch to make them return values.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20160106/134f176f/attachment.html>
More information about the Mercurial-devel
mailing list