Block Conflict Resolution Possibility

[sschr15]

Perhaps in the future write a more extensive api that has all modded blocks be instances of a FabricBlock, and thus all with same numerical id, but also have a unique String id, which is checked every time the numerical id is checked, but that would require many mixins to fix that, and that is probably later down the road

Originally posted by @amusingimpala75 in #3 (comment)

[BoogieMonster1O1]

I recommend remapping the blocks array per world and saving a id -> string map

[amusingimpala75]

BoogieMonster's idea would probably require fewer mixins, we would just have to find the right spot in the world loading to remap the "registry".

[BoogieMonster1O1]

That should be done at world load. There's a method in MinecraftClient to do that.

[BoogieMonster1O1]

Note that storing multiple blocks with the same numeric id would involve rewriting the chunk palette as blocks are currently stored in chunks as bytes (numeric ids)
Numeric ids are also used in many return types, parameters and fields.

[BoogieMonster1O1]

https://github.com/Legacy-Fabric/fabric/blob/1.8.9/fabric-content-registries-v1/src/main/java/net/fabricmc/fabric/impl/content/registries/ContentRegistryImpl.java
i'm thinking this would work

[sschr15]

If we're just mapping the 256 block IDs, then we only have 256 block IDs. Allowing more would maybe be possible (the main location is all the Blocks registering themselves into their corresponding item IDs), but there may be more in the code that I have seen.

[BoogieMonster1O1]

If we're just mapping the 256 block IDs, then we only have 256 block IDs. Allowing more would maybe be possible (the main location is all the Blocks registering themselves into their corresponding item IDs), but there may be more in the code that I have seen.

As I said, chunks store blocks in a byte palette, so there can't be any more than 256

[amusingimpala75]

at least not without rewriting the chunk storage method

[BoogieMonster1O1]

at least without rewriting the chunk storage method

yep
iirc in mcregion and anvil the palette is split up, so one map stores random number -> blockstate and the palette stores a list of random numbers

[amusingimpala75]

See #3 with it's recent commit