Skip to content

Comments

1.12#5

Open
LiskoSlayer63 wants to merge 6 commits intoabused:1.12from
LiskoSlayer63:1.12
Open

1.12#5
LiskoSlayer63 wants to merge 6 commits intoabused:1.12from
LiskoSlayer63:1.12

Conversation

@LiskoSlayer63
Copy link

@LiskoSlayer63 LiskoSlayer63 commented Jan 21, 2018

I tried to fix the fill command and achieved something. Now it fills the world, but SpongeForge causes odd phenomenoms.

Also fixed couple of crashes and cleaned some warnings.

Tries to fix #1, #2 and #3.

I honestly have no idea what I'm doing.

- Changed deprecated event bus register/unregister.
- Fixed (partially) the fill task to not pre-generating world (now generates but SpongeForge MUST NOT be installed).
- Fixed the crash caused by fill/trim.
@smmmadden
Copy link

smmmadden commented Jan 21, 2018

at least you tried to fix it. :-) I give credit where it is well deserved. I'll test it on my test server with todays forge/sponge builds and see how it goes. Thanks! -Steve

Edit: I was going to download the jar but looks like it isn't posted yet. Once it is, I'll test it out.

@abused
Copy link
Owner

abused commented Jan 21, 2018

I appreciate you're attempt to fix it! The reason however is persisting from me using Access Transformers to get and add to a private list from the world class, with the server not detecting the AT's thus refusing the existence of the public field. I could however easily end up fixing it by using some reflection, the problem is currently time on my hands has been very tight

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.

3 participants