[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