indygreg added inline comments.

INLINE COMMENTS

> mbthomas wrote in test-revlog-mmapindex.t:12-13
> For (1) There's no `ui` object available in the `revlog.revlog` constructor.  
> Is there a way to get one?
> For (2), the rest of the test suite will be loading the revlog without mmap, 
> so I didn't think it necessary to test it again here.

I'd do it with inline Python. Something like:

>>> from mercurial import hg, ui as uimod
>>>  myui = uimod.ui.load()
>>>  repo = hg.repository(myui, path='.')
>>> 
>>> 1. Test repo.changelog here.

Also, excessive `hg` commands makes tests slow. Unfortunately, doing this in 
Python is probably a bit of work :/

REPOSITORY
  rHG Mercurial

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

To: mbthomas, #fbhgext, indygreg, #hg-reviewers, durin42
Cc: quark, durin42, simonfar, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to