Remove unused internal utilities

[IndrajeetPatil]

No description provided.

[MichaelChirico]

are odd_index/even_index still used?

[IndrajeetPatil]

Yes, they are.

[lorenzwalthert]

btw I wonder if {lintr} should offer such a check, i.e. if all symbols are used. Does it already?

[IndrajeetPatil]

I have created an issue to track this: r-lib/lintr#1513

We currently have only unused_import_linter(), but it'd be good to also have something like unused_internal_object() linter.

[github-actions]

This is how benchmark results would change (along with a 95% confidence interval in relative change) if 1e96379 is merged into main:

•   :ballot_box_with_check:cache_applying: 36.2ms -> 35.7ms [-4.36%, +1.25%]
•   :ballot_box_with_check:cache_recording: 1.88s -> 1.86s [-1.72%, +0.16%]
•   :ballot_box_with_check:without_cache: 4.94s -> 4.98s [-0.13%, +2.09%]

Further explanation regarding interpretation and methodology can be found in the documentation.

[lorenzwalthert]

Thanks @IndrajeetPatil. Until we use {fledge}, can you please also add a new bullet for this and your previous contributions? I'll merge afterwards.

[lorenzwalthert]

Thanks @IndrajeetPatil for this contribution 🚀 .

[github-actions]

This is how benchmark results would change (along with a 95% confidence interval in relative change) if c01a2ab is merged into main:

•   :ballot_box_with_check:cache_applying: 28.8ms -> 28.8ms [-1.7%, +1.12%]
•   :ballot_box_with_check:cache_recording: 1.3s -> 1.3s [-1.07%, +0.59%]
•   :ballot_box_with_check:without_cache: 3.44s -> 3.45s [-0.65%, +1.19%]

Further explanation regarding interpretation and methodology can be found in the documentation.