D492: context: remove unnecessary default values for matchers (API)
dsp (David Soria Parra)
phabricator at mercurial-scm.org
Thu Aug 24 01:05:12 EDT 2017
dsp requested changes to this revision.
dsp added a comment.
This revision now requires changes to proceed.
if match can't be None, then we should remove it as the default value. Otherwise I look at this in a year and think "why do we not handle a None matcher, when it clearly can be None if the value for match is not set".
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D492
To: martinvonz, #hg-reviewers, dsp
Cc: dsp, mercurial-devel
More information about the Mercurial-devel
mailing list