Avoid clobbering `r` variable in Bessel function #7

Merged
gavin merged 1 commits from mcf/bc:bessel into master 2 years ago
mcf commented 2 years ago

The j function uses r to accumulate its result, but did not
declare it with auto, causing the caller's value to be clobbered.

To fix this, declare r as auto to make it a local variable.

The `j` function uses `r` to accumulate its result, but did not declare it with auto, causing the caller's value to be clobbered. To fix this, declare `r` as auto to make it a local variable.
gavin merged commit 021527d2bc into master 2 years ago
Owner

Obvious bug is obvious, and obvious fix is obvious. I will put out a release very soon. Thank you!

Obvious bug is obvious, and obvious fix is obvious. I will put out a release very soon. Thank you!
Owner

Sorry, I didn't mean that what you did wasn't valuable. I obviously didn't catch it.

What I meant is that is was obvious that your fix was good.

Words...

Sorry, I didn't mean that what you did wasn't valuable. I obviously didn't catch it. What I meant is that is was obvious that your fix was good. Words...
Owner

Release is out!

Release is out!
The pull request has been merged as 021527d2bc.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.