Fix pylint-dev#5586: False positive for used-before-assignment with homonyms in filtered comprehensions and except blocks

Author: jacobtylerwalls

ChangeLog

@@ -237,6 +237,12 @@ Release date: 2021-11-25
 
   Closes #5342
 
+* Fixed a false positive (affecting unreleased development) for
+  ``used-before-assignment`` involving homonyms between filtered comprehensions
+  and assignments in except blocks.
+
+  Closes #5586
+
 * Specified that the ``ignore-paths`` option considers "\" to represent a
   windows directory delimiter instead of a regular expression escape
   character.

doc/whatsnew/2.13.rst

@@ -139,6 +139,12 @@ Other Changes
 
   Closes #5360, #3877
 
+* Fixed a false positive (affecting unreleased development) for
+  ``used-before-assignment`` involving homonyms between filtered comprehensions
+  and assignments in except blocks.
+
+  Closes #5586
+
 * Fixed crash on list comprehensions that used ``type`` as inner variable name.
 
   Closes #5461

pylint/checkers/variables.py

@@ -1316,6 +1316,10 @@ def _check_consumer(
             if utils.is_func_decorator(current_consumer.node) or not (
                 current_consumer.scope_type == "comprehension"
                 and self._has_homonym_in_upper_function_scope(node, consumer_level)
+                and not (
+                    isinstance(node.parent.parent, nodes.Comprehension)
+                    and node.parent in node.parent.parent.ifs
+                )
             ):
                 self._check_late_binding_closure(node)
                 self._loopvar_name(node)

tests/functional/u/use/used_before_assignment_filtered_comprehension.py

@@ -0,0 +1,9 @@
+"""Homonym between filtered comprehension and assignment in except block."""
+
+def func():
+    """https://github.com/PyCQA/pylint/issues/5586"""
+    try:
+        print(value for value in range(1 / 0) if isinstance(value, int))
+    except ZeroDivisionError:
+        value = 1
+        print(value)