Simplify if cond: return True/False else: return False/True patterns

[yarikoptic]

Summary

Seeing those code patterns triggerred me ;)

Small readability cleanup: replace several if cond: return True else: return False (and the inverse) constructs with a direct return cond (or return not cond, or a swapped ==/!=). No behavior change.

Commits

• 5731ace Simplify if cond: return True/False else: return False/True patterns

Per-commit details

5731ace Simplify if cond: return True/False else: return False/True patterns

Refactored 6 instances across 3 files:

• spikeinterface_gui/curationview.py — 3 methods (_conditional_refresh_merge / _delete / _split): if cond: return True else: return False → return cond.
• spikeinterface_gui/main.py (the SortingAnalyzer-folder check, both the non-zarr and zarr branches): if X != Y: return False else: return True → return X == Y (swapped != to ==).
• spikeinterface_gui/view_base.py (is_warning_active): if active_window and isinstance(active_window, QT.QMessageBox): return True; return False → return isinstance(active_window, QT.QMessageBox). isinstance(None, ...) already returns False, so the truthiness guard was redundant.

Left controller.py:check_is_view_possible alone because its if-branch has a side-effecting print, so the pattern doesn't simplify cleanly.

Test plan

• python -m py_compile on the three edited files
• Reviewer: spot-check the is_warning_active simplification — relying on isinstance(None, ...) being False