Skip to content

add 'containsId' method to containers#344

Open
tremblap wants to merge 1 commit into
flucoma:mainfrom
tremblap:feature/adds-contains-method-to-containers
Open

add 'containsId' method to containers#344
tremblap wants to merge 1 commit into
flucoma:mainfrom
tremblap:feature/adds-contains-method-to-containers

Conversation

@tremblap
Copy link
Copy Markdown
Member

I needed that for stuff in SC, and now Balint needs it for something in Max... and I needed an easy way back in the codebase so I implemented it the way @weefuzzy suggested a few years ago.

It works. Comments/Questions for the boss:

  • containsid (in max) is ugly but just 'contains' is ambiguous, especially for LabelSet.
  • I need to do the docs and sc method and include in help of Max+Pd but before I do so, maybe we should see if my code is clean enough

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