Skip to content

Conversation

@PierreSchwang
Copy link
Member

Overview

NbtOps returns a EndTag for the empty entry, which doesn't work with Lin (does make sense tbf) and fails when, for example, appending empty to a map. Switched to an empty compound tag which works and should work for other cases as well.

I don't really know under which circumstances this is actually used - possibly some Codec haven't been able to test.

Submitter Checklist

  • Make sure you are opening from a topic branch (/feature/fix/docs/ branch (right side)) and not your main branch.
  • Ensure that the pull request title represents the desired changelog entry.
  • New public fields and methods are annotated with @since TODO.
  • I read and followed the contribution guidelines.

@PierreSchwang PierreSchwang requested a review from a team as a code owner December 17, 2025 20:54
@github-actions github-actions bot added the Bugfix This PR fixes a bug label Dec 17, 2025
@NotMyFault NotMyFault requested a review from a team December 17, 2025 22:44
@NotMyFault NotMyFault merged commit 066c773 into main Dec 18, 2025
9 checks passed
@NotMyFault NotMyFault deleted the fix/linOpsEmptyTag branch December 18, 2025 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bugfix This PR fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants