Skip to content

[5.0.0] [M] Remove root jfieldvar - #177

Open
glelouet wants to merge 3 commits into
phax:masterfrom
glelouet:remove_root_jfieldvar
Open

[5.0.0] [M] Remove root jfieldvar#177
glelouet wants to merge 3 commits into
phax:masterfrom
glelouet:remove_root_jfieldvar

Conversation

@glelouet

@glelouet glelouet commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

closes #174 #171
embarks #176 #175

root JFieldVar is removed, all reference point to the vars/ one.

glelouet added 3 commits July 23, 2026 20:15
Instead of moving the JFieldVar, this creates a new class in the vars/
package that extends it, and use its constructor instead of the original
one.
@glelouet
glelouet requested a review from phax July 23, 2026 18:36
@glelouet glelouet linked an issue Jul 23, 2026 that may be closed by this pull request
@glelouet glelouet added this to the 5.0.0 milestone Jul 23, 2026
@phax

phax commented Jul 23, 2026

Copy link
Copy Markdown
Owner

Will review after the rest is merged

@glelouet glelouet added the major major-level change : breaks the compatibility. label Jul 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

major major-level change : breaks the compatibility.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

/JFieldVar removed move JFieldVar into the vars package

2 participants