refactor: use stdlib equivalent instead of built-in, fix indentation in math/base/special/ldexp

[gunjjoshi]

Description

What is the purpose of this pull request?

This pull request:

• replaces the use of built-in pow with stdlib_base_pow.
• fixes indentation in example.c.

Related Issues

Does this pull request have any related issues?

None.

Questions

Any questions for reviewers of this pull request?

No.

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

• Read, understood, and followed the contributing guidelines.

---

@stdlib-js/reviewers

[gunjjoshi]

This error in examples comes even without applying the changes in this PR. I'll have a look on ldexp to find out the problem.

[gunjjoshi]

The error message comes as:

example.c:20:10: fatal error: 'stdlib/math/base/special/ldexp.h' file not found
#include "stdlib/math/base/special/ldexp.h"

even though the header files looks correct.

[Planeshifter]

@gunjjoshi Looks like ldexp is missing from the manifest.json' dependencies in the example configuration.

[gunjjoshi]

@gunjjoshi Looks like ldexp is missing from the manifest.json' dependencies in the example configuration.

@Planeshifter For other packages too, we do not include the package's path in the example configuration of manifest.json, right? That is included directly from the header file inside the package. Still, I tried after including that, but the condition was the same.

[Planeshifter]

@gunjjoshi Yeah, the package itself shouldn't be listed as a dependency under its own configuration. I was mistakenly under the impression that the failure was caused in a different package's example. Not clear to me what's going on right now... Also seeing failures loading different modules in the CI checks here.

[Planeshifter]

@gunjjoshi @stdlib/number/float32/base/to-word seems wrong as a dependency in manifest.json. We need @stdlib/number/float64/base/to-words, I think.

[gunjjoshi]

@Planeshifter It doesn't seem we have used that package anywhere in ldexp. We should remove that.

[gunjjoshi]

We have used float64/base/to_words in main.c, but we listed float32/base/to_word in manifest.json. Even after rectifying that, the error persists.

[Planeshifter]

After looking into this a bit more, I noticed that running the C examples for @stdlib/math/base/special/pow is failing as well.

Running the examples with DEBUG=library-manifest:main, it looks to me as if we have a cycle here and @stdlib/utils/library-manifest may not be handling those yet?

cc @kgryte