Skip to content
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

Extending instance misaligns the containing class #31

Open
Synraw opened this issue Aug 14, 2017 · 3 comments
Open

Extending instance misaligns the containing class #31

Synraw opened this issue Aug 14, 2017 · 3 comments

Comments

@Synraw
Copy link

Synraw commented Aug 14, 2017

This GIF should explain the bug better than I can with words lol
http://i.imgur.com/GenRzIR.gifv

@smeckels
Copy link

This is not a bug. That instance is in line with the super class in memory. When you add 4 to the instance you bump member variables of the door class down because this is how reclass handles insertions

@ajkhoury
Copy link
Owner

ajkhoury commented Aug 15, 2017

Smeckels was right on. Hooray for no bugs

@Timboy67678
Copy link
Collaborator

Timboy67678 commented Aug 19, 2017

while this isn't exactly a bug, we should do something to the field that already exists to either be absorbed by the class instance thats grown or just delete the field (dont like that option but it would be easier)

I can understand where the issue is, its not exactly easy or necessary to fix, but we should label it as an enhancement

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants