# HG changeset patch
# User Martin von Zweigbergk <martinvonz@google.com>
# Date 2019-10-12 22:12:54
# Node ID 0e2a2fab4b2ed98e1c0b0dd43285b57ff94ebf00
# Parent  d3d1a3afe7aa5f2d72e08bbd83c676a6b3adb46a

fix: remove a never-true check for unset pattern in Fixer.affects()

We don't create an instance of a Fixer if the pattern is None, so we
don't need to check that in affects().

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

diff --git a/hgext/fix.py b/hgext/fix.py
--- a/hgext/fix.py
+++ b/hgext/fix.py
@@ -843,9 +843,7 @@ class Fixer(object):
 
     def affects(self, opts, fixctx, path):
         """Should this fixer run on the file at the given path and context?"""
-        return self._pattern is not None and scmutil.match(
-            fixctx, [self._pattern], opts
-        )(path)
+        return scmutil.match(fixctx, [self._pattern], opts)(path)
 
     def shouldoutputmetadata(self):
         """Should the stdout of this fixer start with JSON and a null byte?"""