[RFC]: refactor `blas/ext/base/dsort2sh` to follow current project conventions

[Planeshifter]

Description

This RFC proposes refactoring @stdlib/blas/ext/base/dsort2sh to follow current project conventions.

These conventions are outlined in more detail in #788. Namely, we want to migrate from C++ add-on interfaces to C add-on interfaces and then make various style and simplification changes.

Some example commits which perform most, if not all, of the requested changes include d66d80c (saxpy), 3c82fe7 (cswap), and 5b15c96 (dasum).

Related Issues

Related issues #788, #1152.

Questions

No.

Other

No.

Checklist

• I have read and understood the Code of Conduct.
• Searched for existing issues and pull requests.
• The issue name begins with RFC:.

[Pratik772846]

Hi @kgryte @Planeshifter , I would like to work on this issue.

[vexora-0]

hey @Planeshifter, I would like to work on this

[Raj-Ganatra2]

Hey! @Planeshifter ,I'm thrilled and exicted to work on this issue.

[hkcc375]

Hey @Planeshifter. Can you please assign this issue to me ? I have already begun working on this issue and have made several commits.
I have 2 doubts though :

• All the benchmark files in the benchmark folder, use floor. Do we get rid of floor usages here as well ? After all, floor is used to reduce the number of times the benchmark function is called. Or could I use some other alternative instead of floor, something like trunc ?
• why is the usage of floor discouraged ?

[kgryte]

@hkcc375 Not all floor usages; only the usages for computing the number of indexed elements in examples and unit tests. We made this change in order to be more explicit about how many values are indexed. This is not applicable for benchmarks.