Skip to content

Conversation

@jkbelcher
Copy link
Collaborator

No description provided.

@mcslee mcslee merged commit f5d341c into heronarts:dev Oct 9, 2025
2 checks passed
@mcslee
Copy link
Member

mcslee commented Oct 9, 2025

Made a few changes on top of that:
029d4c8

Don't love maintaining separate mutableTags and tags - just gonna wrap with Collections.unmodifiableList when getTags is called (not often), very cheap/lightweight wrapper to avoid risk of data inconsistency.

Also changed init to use the proper addPattern/addEffect methods, so the default tags are added that way. Avoid the risk of maintaining logic for component registration in two places. Got rid of the fugly static block for List initialization in the process.

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