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