Skip to content

fix(cpp23): shadow identifier + C-style cast cleanup in Wave files (ADR-0839) - #258

Merged
lusoris merged 1 commit into
masterfrom
refactor/cpp23-shadow-cast-cleanup-clean-20260530
May 30, 2026
Merged

lusoris merged 1 commit into
masterfrom
refactor/cpp23-shadow-cast-cleanup-clean-20260530

Conversation

@lusoris

@lusoris lusoris commented May 30, 2026

Copy link
Copy Markdown
Contributor

Recreates closed PR #196's 6-file targeted change from current master. Mechanical cleanup; no behaviour change.

no docs needed: internal C++23 conversion TUs only.

Test plan

  • Local diff clean (6 files only)
  • CI: meson test --suite=fast green

Deep-dive deliverables (ADR-0108)

  • Research digest — no digest needed: trivial mechanical cast replacement
  • Decision matrix — ADR-0839 ## Alternatives considered (in diff)
  • AGENTS.md invariant note — no rebase-sensitive invariants
  • Reproducer — meson test -C build --suite=fast
  • Changelog fragment — changelog.d/fixed/cpp23-shadow-const-fixes.md (in diff)
  • Rebase-notes entry — docs/rebase-notes.md updated (in diff)

Lint clean (CLAUDE §12 r12)

  • 4 source TUs + 2 docs files touched

State drift (CLAUDE §12 r13)

  • No bug-tracking change — pure mechanical cleanup

FFmpeg-patch sync (CLAUDE §12 r14)

  • No public-API change — internal TUs only

🤖 Generated with Claude Code

…DR-0839)

Mechanical cleanup in 3 post-C++23-wave cpp files:
- fex_ctx_vector.cpp: rename local 'capacity' → 'new_capacity' (shadow fix)
- feature_collector.cpp: every (T*) C-style cast → static_cast<T*>
- sycl/common.cpp: same C-style → static_cast<> pattern

Recreates closed PR #196's 6-file targeted change from current master
(the original had cascading rebase conflicts from #197+#212+#256 etc).
docs/adr/README.md index row already present on master; skipping that file.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@lusoris
lusoris enabled auto-merge (squash) May 30, 2026 04:16
@lusoris
lusoris merged commit 95cc2b1 into master May 30, 2026
53 of 64 checks passed
@lusoris
lusoris deleted the refactor/cpp23-shadow-cast-cleanup-clean-20260530 branch May 30, 2026 04:32
lusoris added a commit that referenced this pull request May 30, 2026
…ADR-0840) (#259)

Recreates closed PR #199's 6-file targeted change from current master
(the original had cascading rebase conflicts from #197+#212+#256+#258).

- gpu_dispatch_env.c: TOCTOU window between getenv() and snapshot
- vmaf.c: cu_state leak on import failure path

Co-authored-by: lusoris <lusoris@pm.me>
Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
@lusoris lusoris added this to the 1.0.0 — First release milestone Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant