D757: dirstate: move opendirstatefile to dirstatemap

2017-10-01 Thread foozy (Katsunori FUJIWARA)
foozy added inline comments. INLINE COMMENTS > durham wrote in dirstate.py:1373-1374 > Agreed. I'm not even sure what it's actually saying as a mercurial > developer... (sorry for late response) With current implementation of dirstate, _pl() reads "dirstate" file only for parents of wdir,

D757: dirstate: move opendirstatefile to dirstatemap

2017-09-29 Thread durham (Durham Goode)
This revision was automatically updated to reflect the committed changes. Closed by commit rHG0407c611b7be: dirstate: move opendirstatefile to dirstatemap (authored by durham, committed by ). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D757?vs=2081=2160

D757: dirstate: move opendirstatefile to dirstatemap

2017-09-26 Thread durham (Durham Goode)
durham marked an inline comment as done. durham added inline comments. INLINE COMMENTS > indygreg wrote in dirstate.py:1373-1374 > This (pre-existing) error message is pretty bad because typical end users > won't have a clue what it means or what to do if they see it. Agreed. I'm not even

D757: dirstate: move opendirstatefile to dirstatemap

2017-09-26 Thread durham (Durham Goode)
durham updated this revision to Diff 2081. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D757?vs=1952=2081 REVISION DETAIL https://phab.mercurial-scm.org/D757 AFFECTED FILES mercurial/dirstate.py CHANGE DETAILS diff --git a/mercurial/dirstate.py

D757: dirstate: move opendirstatefile to dirstatemap

2017-09-20 Thread durham (Durham Goode)
durham created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY As part of moving the dirstate storage logic to another class, let's move opendirstatefile to dirstatemap. This will allow extensions to replace the pending