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

OSC: Make plugin documentation complete #266

Merged

Conversation

matthijskooijman
Copy link
Contributor

This adds some missing documentation about manipulating plugin (parameters), fixes some incorrectness and formatting inconsistencies. See individual commit messages for details.

This is already mentioned on the feedback page, but mentioning it above
the list of control messages helps interpreting that list.
The rest of the docs do not use commas to separate the address from
arguments, so remove them here. These commas seem to have been
introduced first in commit 1138f1d (OSC: update OSC docs.) and were
probably copy-pasted into docs added after that.
This makes the documentation complete by adding missing messages,
clarifying some existind documenation and fixing `/select/plug_page`
(address and parameter description were incorrect).
@prokoudine prokoudine merged commit f9b6f72 into Ardour:master Jul 2, 2024
pauldavisthefirst pushed a commit that referenced this pull request Jul 2, 2024
* OSC: Clarify control messages are also sent as feedback

This is already mentioned on the feedback page, but mentioning it above
the list of control messages helps interpreting that list.

* OSC: Remove errant commas in addresses

The rest of the docs do not use commas to separate the address from
arguments, so remove them here. These commas seem to have been
introduced first in commit 1138f1d (OSC: update OSC docs.) and were
probably copy-pasted into docs added after that.

* OSC: Fix and complete plugin operations and feedback

This makes the documentation complete by adding missing messages,
clarifying some existind documenation and fixing `/select/plug_page`
(address and parameter description were incorrect).
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.

2 participants