# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@octobus.net>
# Date 1569535783 -7200
# Node ID 15badd621825e3b112b95f3bb46f8090f95adc0c
# Parent  008e74b34fb728c98cecb29e337c56a58d93d533
context: clarify the various mode in the filesremoved method

The previous code was compact but a bit dense. The new proposed code deal with
each mode separately, there are some duplicated lines, but the meaning of each
mode stand out.

One of the benefit it to make it simpler to add further mode in the future.

Differential Revision: https://phab.mercurial-scm.org/D6932

diff -r 008e74b34fb7 -r 15badd621825 mercurial/context.py
--- a/mercurial/context.py	Fri Sep 27 00:06:02 2019 +0200
+++ b/mercurial/context.py	Fri Sep 27 00:09:43 2019 +0200
@@ -469,11 +469,16 @@
 
     def filesremoved(self):
         source = self._repo.ui.config('experimental', 'copies.read-from')
-        if (source == 'changeset-only' or
-            (source == 'compatibility' and
-             self._changeset.filesremoved is not None)):
-            return self._changeset.filesremoved or []
-        return scmutil.computechangesetfilesremoved(self)
+        filesremoved = self._changeset.filesremoved
+        if source == 'changeset-only':
+            if filesremoved is None:
+                filesremoved = []
+        elif source == 'compatibility':
+            if filesremoved is None:
+                filesremoved = scmutil.computechangesetfilesremoved(self)
+        else:
+            filesremoved = scmutil.computechangesetfilesremoved(self)
+        return filesremoved
 
     @propertycache
     def _copies(self):