refactor/test: extract is-same-note logic from InstrumentClipView

SynthstromAudible / DelugeFirmware

GNU General Public License v3.0

565 stars 94 forks source link

Closed nikodemus closed 3 months ago

nikodemus commented 3 months ago

There were a couple of variations, both seemingly needlessly complex, and the second one buggy. Test case includes the verbatim versions of the old code, and demonstrates bug.
Drop a couple of comments that aren't needed given a descriptive function name.

github-actions[bot] commented 3 months ago

47 tests +1 47 :white_check_mark: +1 0s :stopwatch: ±0s 9 suites +1 0 :zzz: ±0 9 files +1 0 :x: ±0

Results for commit 58c8f63a. ± Comparison against base commit 4daa5bb1.

:recycle: This comment has been updated with latest results.