[Commented On] D8552: fix: use context to fetch mergestate instead of loading it directly

[martinvonz (Martin von Zweigbergk)]

martinvonz added a comment.


  In D8552#128629 <https://phab.mercurial-scm.org/D8552#128629>, @marmoute wrote:
  
  > In D8552#128621 <https://phab.mercurial-scm.org/D8552#128621>, @martinvonz wrote:
  >
  >> In D8552#128605 <https://phab.mercurial-scm.org/D8552#128605>, @marmoute wrote:
  >>
  >>> @martinvonz can you clarify why mergestate on an `overlayworkingctx` would never make sense ?
  >>
  >> That makes sense. What I said does not make sense is to check for unresolved merge conflicts (before starting an operation).
  >
  > Checking for unresolved merge conflict on an `overlayworkingctx` does not make sense because the overlayctx is new and therefor will always be conflict free ?
  
  I view `overlayworkingctx` as something that exists for creating commits without touching the working copy and without caring about the working copy state. In the code here, we explicitly care about the working copy state (because we're going to modify it).

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D8552/new/

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

To: durin42, #hg-reviewers, marmoute
Cc: marmoute, martinvonz, mercurial-patches
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mercurial-scm.org/pipermail/mercurial-patches/attachments/20200609/ec0cd7c6/attachment-0002.html>