remove match_

[luizirber]

I think match_ is a leftover from the first impl of Index on the rust side, and not needed?

[codecov]

Codecov Report

Attention: Patch coverage is 81.81818% with 2 lines in your changes missing coverage. Please review.

Project coverage is 86.43%. Comparing base (11cab80) to head (06867ac).

Files with missing lines	Patch %	Lines
src/core/src/ffi/index/revindex.rs	0.00%	1 Missing ⚠️
src/core/src/index/revindex/mem_revindex.rs	87.50%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           latest    #3351      +/-   ##
==========================================
- Coverage   86.45%   86.43%   -0.03%     
==========================================
  Files         137      137              
  Lines       16092    16086       -6     
  Branches     2219     2219              
==========================================
- Hits        13913    13904       -9     
- Misses       1872     1875       +3     
  Partials      307      307              

Flag	Coverage Δ
hypothesis-py	25.43% <ø> (ø)
python	92.40% <ø> (ø)
rust	62.03% <81.81%> (-0.17%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ctb]

This seems ready, yes?

[luizirber]

This seems ready, yes?

sorta, still need to resolve the todo!() in FFI code (for something that is not used in the Python layer, but requires access to the sig...), and without passing the sig in GatherResult I think there is more that can be refactor. Might leave this one for a future PR