[GH-ISSUE #42] Dyno possible optimization #36

Closed
opened 2026-05-05 22:02:49 -06:00 by gitea-mirror · 1 comment
Owner

Originally created by @NeverKnewTheName on GitHub (Aug 4, 2022).
Original GitHub issue: https://github.com/airwindows/airwindows/issues/42

the line double dyno = pow(fabs(inputSampleL),4); could be simplified to dyno=pow( inputSampleL, 4)
the same applies to the right channel. sth. to the power of an even number is always positive.

Originally created by @NeverKnewTheName on GitHub (Aug 4, 2022). Original GitHub issue: https://github.com/airwindows/airwindows/issues/42 the line `double dyno = pow(fabs(inputSampleL),4);` could be simplified to `dyno=pow( inputSampleL, 4)` the same applies to the right channel. sth. to the power of an even number is always positive.
Author
Owner

@airwindows commented on GitHub (Aug 5, 2022):

For what it's worth I'm paying attention and making note of any changes that seem like I should find time to fit in the revision. It will always involve updating EVERY build subject to the bug like retro and modern Mac AU and VST, the WIndows builds, both Linux builds (intel and Raspberry Pi) and if there is a change in sound (even if the sound is due to a full-on bug and working not as intended) I'll need to do a new version so that saved mixes with the busted plugin don't change upon recall later :)

<!-- gh-comment-id:1205995865 --> @airwindows commented on GitHub (Aug 5, 2022): For what it's worth I'm paying attention and making note of any changes that seem like I should find time to fit in the revision. It will always involve updating EVERY build subject to the bug like retro and modern Mac AU and VST, the WIndows builds, both Linux builds (intel and Raspberry Pi) and if there is a change in sound (even if the sound is due to a full-on bug and working not as intended) I'll need to do a new version so that saved mixes with the busted plugin don't change upon recall later :)
Sign in to join this conversation.
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/airwindows#36
No description provided.