Add precise frequency mode to BuffrGlitch #160
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
My implementation of what was proposed in #138.
This is code that wasn’t tested nor built locally so I lay my hopes on CI making sure if it builds at all. Hopefully it should because I didn’t need to use depths of Rust I’m not accustomed with.
Also I tried to write the code according to style I was able to see in this project but if there are parts that should be reformatted, just say so, of course.
Hopefully it does what I intended it to do. And the default behavior should be the old one, the new mode is opt-in.
Also I wasn’t satisfied very much with the mode change applying only on note-on events but that was the easiest way to go without wreaking havoc on the current API, and also it seems pretty reasonable not to mess with a currently played note, and pretty reasonable that the user shouldn’t change the mode back and forth at all.
Well, and also my description in the changelog is probably not very instructive; I’m open for suggestions.