Remove unused Ptr_xxx_get P/Invoke declarations from features2d NativeMethods

[shimat]

Summary

Cleanup of dead code found while investigating the #1846 migration.

All feature detector Create() methods (SIFT, BRISK, ORB, MSER, FastFeatureDetector, AgastFeatureDetector, GFTTDetector, SimpleBlobDetector, KAZE, AKAZE) retrieve the raw pointer via the shared features2d_Ptr_Feature2D_get. The individual Ptr_xxx_get declarations were added to NativeMethods_features2d_Feature2D.cs during #1846 but:

• Are never called anywhere in the managed codebase
• Have no corresponding C++ implementations in features2d_Feature2D.h

Removes all 10 dead declarations.

Summary by CodeRabbit

• Refactor
  • Cleaned up internal method declarations for feature detection algorithms (SIFT, BRISK, ORB, MSER, FastFeatureDetector, AgastFeatureDetector, GFTTDetector, SimpleBlobDetector, KAZE, AKAZE). Core functionality remains intact.

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR removes ten unused P/Invoke method declarations from the features2d native method bindings. The removed methods are pointer getter operations (*_get) for various feature detector types. All other P/Invoke declarations in those regions remain unchanged.

Changes

Cohort / File(s)	Summary
P/Invoke cleanup src/OpenCvSharp/Internal/PInvoke/NativeMethods/features2d/NativeMethods_features2d_Feature2D.cs	Removed 10 P/Invoke declarations for feature detector pointer getters (SIFT, BRISK, ORB, MSER, FastFeatureDetector, AgastFeatureDetector, GFTTDetector, SimpleBlobDetector, KAZE, AKAZE). No additions; only deletions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main change: removal of ten unused Ptr_xxx_get P/Invoke declarations from the features2d NativeMethods file.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch claude/remove-dead-pinvoke-declarations

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}