Dylan T.

Results 339 issues of Dylan T.

replaceAll() is not suitable for converting between different domains of IDs. This is problematic if we want to alter the subchunks returned by SubChunkConverter (e.g. to expand metadata size), because...

Currently `RuntimeException` is used in almost all cases. This is too generic.

enhancement

"word array" is very undescriptive and very confusing to new users. "palette offset table" or "palette offset array" might be better. It might also be worth sealing the palette offset...

enhancement

Since SubChunk is final (or should be), there's clear bounds on its functionality. Since all the types of its fields are also sealed, we can safely assume that the PHP...

Plugin devs are not going to understand what's happening when an assert failure happens (or a segfault).

In a sane world where block conversion is handled properly, we're almost never going to want the original block IDs that were contained in an anvil, mcregion or legacy leveldb...

enhancement

The vast majority of devices and machines these days are little-endian, but this extension doesn't consider the possibility of being compiled and run on a big-endian platform. If it is...

bug

During deserialization of complex NBT, we're likely to encounter NBT with many repeated values which could be cached. For example, when deserializing a block palette file, the following types of...

Type: Enhancement

in very brief tests, this yielded significant performance improvements, on the order of 2x for encoding and 1.5x for decoding. This doesn't change any public API, but inheritors of `BaseNbtSerializer`...

It may be desirable to keep an encoded binary cache of an NBT tree, to be able to do fast bytewise comparison (such comparisons are vectorizable, and as such, considerably...