darcs

Issue 2718 review use of 'conflictedPaths' to decide if there are conflicts

Title review use of 'conflictedPaths' to decide if there are conflicts
Priority Status resolved
Milestone Resolved in 2.18.1
Superseder Nosy List bfrk
Assigned To
Topics

Created on 2024-05-10.11:38:26 by bfrk, last changed 2024-05-11.20:21:46 by noreply.

Messages
msg23796 (view) Author: bfrk Date: 2024-05-10.11:38:26
As Ganesh noted in a comment to patch2358:

> Using "conflictedPaths" as the indicator of "are there any conflicts"
> seems standard in other places in the code, but it does feel a bit
> ugly. It doesn't seem intrinsic that a conflict has to be associated
> with some path

I agree so here is a ticket to remind us about looking into this.
msg23811 (view) Author: noreply Date: 2024-05-11.20:21:45
The following patch sent by Ben Franksen <ben.franksen@online.de> updated issue issue2718 with
status=resolved;resolvedin=2.18.1

Hash: 2a82da8f0a29f0ec62e0b5aa7853f5a8886073ee
Author: Ben Franksen <ben.franksen@online.de>
* resolve issue2718: avoid using conflictedPaths to detect whether we have conflicts

  This adds an explicit test named haveConflicts, which looks at the "sum" of
  the mangled and unmangled conflicts. This is cleaner because in principle
  (albeit not in practice) there could be conflicts that do not involve any
  file paths. Consequently, the warning message for conflicts is now separate
  from the listing of the affected paths.
History
Date User Action Args
2024-05-10 11:38:26bfrkcreate
2024-05-11 20:21:46noreplysetstatus: unknown -> resolved
messages: + msg23811
resolvedin: 2.18.1