# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@octobus.net>
# Date 1602552197 -7200
# Node ID 232c88dd89e3a42613c5fe13030bcdf83750a520
# Parent  d4c4391aa7f26c8a5c329330f14ab339a34b9130
changing-files: add a shorthand property to check for copy relevant info

We are going to reuse this change in more place, so we factor it out first.

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

diff -r d4c4391aa7f2 -r 232c88dd89e3 mercurial/changelog.py
--- a/mercurial/changelog.py	Wed Oct 14 17:51:56 2020 +0200
+++ b/mercurial/changelog.py	Tue Oct 13 03:23:17 2020 +0200
@@ -583,13 +583,7 @@
         flags = 0
         sidedata = None
         if self._copiesstorage == b'changeset-sidedata':
-            if (
-                files.removed
-                or files.merged
-                or files.salvaged
-                or files.copied_from_p1
-                or files.copied_from_p2
-            ):
+            if files.has_copies_info:
                 flags |= flagutil.REVIDX_HASCOPIESINFO
             sidedata = metadata.encode_files_sidedata(files)
 
diff -r d4c4391aa7f2 -r 232c88dd89e3 mercurial/metadata.py
--- a/mercurial/metadata.py	Wed Oct 14 17:51:56 2020 +0200
+++ b/mercurial/metadata.py	Tue Oct 13 03:23:17 2020 +0200
@@ -75,6 +75,16 @@
             and self.copied_from_p2 == other.copied_from_p2
         )
 
+    @property
+    def has_copies_info(self):
+        return bool(
+            self.removed
+            or self.merged
+            or self.salvaged
+            or self.copied_from_p1
+            or self.copied_from_p2
+        )
+
     @util.propertycache
     def added(self):
         """files actively added in the changeset