Skip to content

fix: resolve compiler warnings in PQ, PreTransform, and Refine Index files#4941

Closed
maxwbuckley wants to merge 2 commits intofacebookresearch:mainfrom
maxwbuckley:fix/warnings-index-pq-refine
Closed

fix: resolve compiler warnings in PQ, PreTransform, and Refine Index files#4941
maxwbuckley wants to merge 2 commits intofacebookresearch:mainfrom
maxwbuckley:fix/warnings-index-pq-refine

Conversation

@maxwbuckley
Copy link
Copy Markdown
Contributor

Summary

  • Fix compiler warnings in IndexNeuralNetCodec, IndexPQ, IndexPQFastScan, IndexPreTransform, IndexRaBitQ, IndexRaBitQFastScan, IndexRefine
  • Fixes include -Wshadow, -Wunused-parameter, and -Wnon-virtual-dtor

All changes are mechanical. No functional changes.

Part 10/13 of the compiler warnings cleanup (split from #4810 per maintainer request).

Test plan

  • No functional changes — all fixes are mechanical renames and annotations

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync bot commented Mar 17, 2026

@alibeklfc has imported this pull request. If you are a Meta employee, you can view this in D96996735.

@maxwbuckley maxwbuckley force-pushed the fix/warnings-index-pq-refine branch from 4b94341 to 205314f Compare March 18, 2026 10:06
maxwbuckley and others added 2 commits March 19, 2026 21:27
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync bot commented Mar 20, 2026

@alibeklfc merged this pull request in a05f13f.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant