Skip to content

Fix for missing glist_findrtext in new Pd versions #76

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

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

Conversation

ben-wes
Copy link
Contributor

@ben-wes ben-wes commented Mar 11, 2025

this uses the mechanism we already had in place for the multichannel handling.

uses the new glist_getrtext if present and otherwise falls back to the previous glist_findrtext

tested with Pd vanilla for official 0.55-2 build and current master including the change.

@ben-wes
Copy link
Contributor Author

ben-wes commented Mar 11, 2025

ah, sorry - had tim's changes in my branch here already! will reset to master and force push ...

EDIT: done

@ben-wes ben-wes force-pushed the fix/glist_getrtext branch from 9fbb9b3 to 3011112 Compare March 11, 2025 09:55
@ben-wes ben-wes force-pushed the fix/glist_getrtext branch from 3011112 to d27256c Compare June 4, 2025 15:06
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.

1 participant