Skip to content

Make ModifierAccessor an utility class#387

Merged
gastaldi merged 2 commits intomasterfrom
modifier_accessor
Jan 4, 2026
Merged

Make ModifierAccessor an utility class#387
gastaldi merged 2 commits intomasterfrom
modifier_accessor

Conversation

@gastaldi
Copy link
Member

@gastaldi gastaldi commented Jan 4, 2026

Because it makes no sense to instantiate this class every time you need to use its methods

- Configure Maven to ensure consistent source file encoding by adding the `UTF-8` property.
@gastaldi gastaldi merged commit 1677d04 into master Jan 4, 2026
4 checks passed
@gastaldi gastaldi deleted the modifier_accessor branch January 4, 2026 23:31
gastaldi added a commit to Lauriichan/roaster that referenced this pull request Jan 4, 2026
gastaldi added a commit that referenced this pull request Jan 5, 2026
* Add api for static initializers

* Update modifier handling in InitializerImpl

As introduced in #387

* Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Add more tests, extract common code and add equals/hashCode to init.

---------

Co-authored-by: George Gastaldi <gegastaldi@gmail.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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

Comments