Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add basic legato support to MidiSynth and VoiceAllocator #786

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mmontag
Copy link
Contributor

@mmontag mmontag commented Oct 3, 2021

Adding a basic legato support in MidiSynth.
The VoiceAllocator will not retrig voices when legato is enabled. This only affects kPolyModeMono.

@olilarkin
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@olilarkin
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@mmontag
Copy link
Contributor Author

mmontag commented Oct 15, 2021

Hmm... the build errors are use of undefined type 'SkSVGNode'

@asedev
Copy link

asedev commented Oct 15, 2021

@olilarkin
Copy link
Member

olilarkin commented Oct 15, 2021 via email

@olilarkin
Copy link
Member

/azp run

@azure-pipelines
Copy link
Contributor

Azure Pipelines successfully started running 1 pipeline(s).

@AlexHarker
Copy link
Collaborator

@olilarkin - as this is now completing tests are you happy to merge?

@olilarkin
Copy link
Member

I wasn't able to test it yet, so not yet

@mmontag
Copy link
Contributor Author

mmontag commented Feb 5, 2022

Friendly nudge :)

@olilarkin
Copy link
Member

sorry i will try and look at this soon @mmontag

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants