COMP/PERF: Qt deprecation fixes and range-loop detach prevention

[hjmjohnson]

Summary

• COMP: Replace deprecated QAbstractItemModel::invalidateFilter() with beginFilterChange()/endFilterChange() (Qt 6.0+)
• COMP: Replace deprecated QImage::mirrored() with flipped() (Qt 6.0+)
• COMP: Replace deprecated Q_ENUMS with Q_ENUM (deprecated since Qt 5.5)
• PERF: Prevent implicit container detach in range-for loops using std::as_const() (clazy range-loop-detach)

These are the first four commits from #1372 extracted for focused review.

Test plan

• All 4 commits build cleanly (verified individually via per-commit build check)
• Qt5 and Qt6 test suites pass with no new failures
• No new compiler warnings introduced

🤖 Generated with Claude Code

[hjmjohnson]

@jamesobutler making smaller PRs to make these easier to review. These are all part of the same effort in #1372

[jamesobutler]

COMP: Replace deprecated QImage::mirrored() with flipped()

Qt 6.9 deprecated QImage::mirrored() in favor of flipped(Qt::Orientations).
Use #if QT_VERSION guards to select the appropriate API while maintaining
Qt5 compatibility.

^Not sure if you care to change the logic, or just make a note in the commit message, but mirrored() was marked deprecated in Qt 6.9, but then changed in Qt 6.10 to indicate that it was now scheduled for deprecation starting with Qt 6.13, thereby un-deprecating it for a bit. See qt/qtbase@0aa07ce

[hjmjohnson]

Thanks for the detailed review @jamesobutler! Addressed all three points:

1. beginFilterChange() guard — Fixed to use >= Qt 6.9 (commit 9cbf246a). You're right that beginFilterChange() was introduced in 6.9; I've updated all four filter setters so beginFilterChange() is called for Qt 6.9+ while endFilterChange() remains guarded at Qt 6.10+. For Qt 6.9, the sequence is beginFilterChange() → update field → invalidateFilter().

2. QImage::mirrored() commit message — Updated (commit aba272b4) to clarify that flipped() was introduced in Qt 6.9 while mirrored() was briefly deprecated, then un-deprecated in Qt 6.10 (scheduled for removal from Qt 6.13). The guards remain at >= 6.9 since flipped() is the recommended forward-going API.

3. Q_ENUM(ParamterValueRole) typo — Fixed in commit e2361bc8. The misplaced/typo'd Q_ENUM before the enum declaration was removed; the correct Q_ENUM(ParameterValueRole) after the enum closing brace remains.

Force-pushed to clazy-fixes-part1.