refactor!: modify C implementation to accept double instead of int32 in math/base/special/fibonacci

[gunjjoshi]

---

type: pre_commit_static_analysis_report
description: Results of running static analysis checks when committing changes. report:

• task: lint_filenames status: passed
• task: lint_editorconfig status: passed
• task: lint_markdown status: passed
• task: lint_package_json status: na
• task: lint_repl_help status: na
• task: lint_javascript_src status: passed
• task: lint_javascript_cli status: na
• task: lint_javascript_examples status: na
• task: lint_javascript_tests status: na
• task: lint_javascript_benchmarks status: na
• task: lint_python status: na
• task: lint_r status: na
• task: lint_c_src status: passed
• task: lint_c_examples status: passed
• task: lint_c_benchmarks status: passed
• task: lint_c_tests_fixtures status: na
• task: lint_shell status: na
• task: lint_typescript_declarations status: na
• task: lint_typescript_tests status: na
• task: lint_license_headers status: passed ---

Resolves None.

Description

What is the purpose of this pull request?

This pull request:

• modifies the C implementation to accept double instead of int32 in math/base/special/fibonacci

Related Issues

Does this pull request have any related issues?

This pull request:

• resolves 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

[stdlib-bot]

Coverage Report

Package	Statements	Branches	Functions	Lines
math/base/special/fibonacci	$\color{green}223/223$ $\color{green}+100.00\%$	$\color{green}9/9$ $\color{green}+100.00\%$	$\color{green}2/2$ $\color{green}+100.00\%$	$\color{green}223/223$ $\color{green}+100.00\%$

The above coverage report was generated for the changes in this PR.

[kgryte]

cc @anandkaranubc as a heads up, this is something Gunj and I discussed. Namely, to update a few function signatures to ensure that we have consistent results between C and JS. Otherwise, if things are inconsistent, it makes scaffolding much more difficult and results in likely unexpected divergences between C and JS in userland.

[kgryte]

LGTM