yuja added a comment.

  >   nit-pick is resolved. `getintrange()` will throw a `ParseError` on 
setting size to one of `min:`, `:max` or `:`. In ideal case, on calling from 
`expectsize()` it shouldn't fail.
  
  So you need to pass in the default min/max values to `getintrange()`.
  The default min can be just `0`. The default max can be `len(repo) + 1`
  (i.e. # of null:tip) for example, or `None` and test `maxsize is not None`
  explicitly.

REPOSITORY
  rHG Mercurial

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

To: navaneeth.suresh, #hg-reviewers
Cc: yuja, pulkit, durin42, mjpieters, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to