D6140: revset: add new contiguous(x) function for "x::x"

mharbison72 (Matt Harbison) phabricator at mercurial-scm.org
Tue Mar 19 23:36:41 EDT 2019


mharbison72 added a comment.


  I've only used `X::X` where X was trivial, so I'm still trying to get my mind around this.  Out of curiosity, what are the scenarios where a nontrivial X is useful?
  
  I'm sure I've used the word "contiguous" when describing the `::` operator to people, but the case @marmoute referenced and even the `contiguous(9+3+4)` result don't match my expectations of the English word.  (For the latter, the 3 doesn't seem contiguous with the rest of the set.)  FWIW, the first 3 words in the help for `::` is "A DAG range".
  
  Maybe if there are a couple of entries in the example section, it would reduce the surprise, whatever the name?

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D6140

To: martinvonz, #hg-reviewers
Cc: mharbison72, yuja, av6, spectral, gracinet, marmoute, mercurial-devel


More information about the Mercurial-devel mailing list