Christian Hughes

Results 34 comments of Christian Hughes

No, per mumfrey in discord: ``` INVOKE_ASSIGN is looking for a method call where the return value is assigned to a local constructors don't return a value, the "value" is...

I've looked into this a bit. From what I can tell Forge has unhooked their permission handler (PermissionAPI) from loading in their own lifecycle. A revamp is also in progress...

As of December https://github.com/MinecraftForge/MinecraftForge/pull/7780 has been merged, so we can take a look at this again for API-9/1.18. For the tracking issue for API-9, see: https://github.com/SpongePowered/Sponge/issues/3627

Well, the MemoryDataView seems like it should know how to serialize, but doesn't work properly: https://github.com/SpongePowered/Sponge/blob/5a0778bc25a66ea96b6799e6c347b65aa4473592/src/main/java/org/spongepowered/common/data/MemoryDataView.java#L271-L281

An EconomyService for proxy-wide money management would be nice.

Just want to point out that Sponge doesn't have any plans for a conversation API for API 8.

Needs capitalization after `@return`: https://github.com/SpongePowered/SpongeAPI/blob/bleeding/src/main/java/org/spongepowered/api/util/ban/Ban.java#L108 Also, there seems to be some inconsistencies in the file, where sometimes it is referred to as `this ban`, and sometimes as `the ban`.

Ideas on which databases we should support as official plugins? MySQL and H2 are a given, since they are currently supported.

Perhaps represent the 'key strings' as CatalogTypes?

Or perhaps a "Users also downloaded:" section?