Fix unused-variable false positive when using same name for multiple exceptions

[zenlyj]

Type of Changes

	Type
✓	🐛 Bug fix
	✨ New feature
	🔨 Refactoring
	📜 Docs

Description

Somewhat related to #4791. When investigating the issue, I noticed that the false positive is only emitted when checking for global unused variables. It is fine when the try block is nested within a function like:

def foo():
    try:
        pass
    except TypeError as exc:
        print(exc)
    except ValueError as exc:
        print(exc)

There seems to be existing logic handling this as a special case, so I extracted it into a function and reused it for the global unused variable check.

Closes #10426

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.88%. Comparing base (f5aa766) to head (a9b2599).

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #10436   +/-   ##
=======================================
  Coverage   95.88%   95.88%           
=======================================
  Files         176      176           
  Lines       19140    19144    +4     
=======================================
+ Hits        18352    18356    +4     
  Misses        788      788           

Files with missing lines	Coverage Δ
pylint/checkers/variables.py	97.30% <100.00%> (+<0.01%)	⬆️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[github-actions]

🤖 According to the primer, this change has no effect on the checked open source code. 🤖🎉

This comment was generated for commit a9b2599

[Pierre-Sassoulas]

Great!

[jacobtylerwalls]

Can you help me understand what this is condition is for? No tests fail when I remove it.