Skip to content

Conversation

@SirYwell
Copy link
Member

Overview

Description

CompoundTag is deprecated and its implementation is now backed by LinCompoundTag. This causes significant overhead when access its data due to the needed transformation/copy.

This PR aims to address spots where CompoundTag is used in FAWE api, as well as some uses that can benefit from the reduced overhead. There is more to do to get CompoundTag fully out of FAWE, but more changes can be addressed later.

### Submitter Checklist
- [x] Make sure you are opening from a topic branch (**/feature/fix/docs/ branch** (right side)) and not your main branch.
- [x] Ensure that the pull request title represents the desired changelog entry.
- [x] New public fields and methods are annotated with `@since TODO`.
- [x] I read and followed the [contribution guidelines](https://github.com/IntellectualSites/.github/blob/main/CONTRIBUTING.md).

@SirYwell SirYwell requested a review from a team as a code owner August 18, 2024 11:44
@github-actions github-actions bot added the chore label Aug 18, 2024
@NotMyFault NotMyFault requested a review from a team August 22, 2024 16:30
@dordsor21
Copy link
Member

Was moving to jetbrains annotations from javax done on purpose? I believe we're relatively consistent for the latter

@NotMyFault
Copy link
Collaborator

Was moving to jetbrains annotations from javax done on purpose? I believe we're relatively consistent for the latter

JavaX is superseded by Jakarta. A migration away from JavaX is a good choice, whether it's its successor or a replacement.

@dordsor21
Copy link
Member

Was moving to jetbrains annotations from javax done on purpose? I believe we're relatively consistent for the latter

JavaX is superseded by Jakarta. A migration away from JavaX is a good choice, whether it's its successor or a replacement.

Should probably make a decision and move all of them at once imo

@SirYwell
Copy link
Member Author

SirYwell commented Sep 1, 2024

Was moving to jetbrains annotations from javax done on purpose? I believe we're relatively consistent for the latter

No, it's mostly because IntelliJ likes to use its annotations by default. I switched to javax for now, but we might want to consider switching to jspecify or checkerframework in future.

@github-actions
Copy link

Please take a moment and address the merge conflicts of your pull request. Thanks!

@SirYwell SirYwell force-pushed the refactor/avoid-CompoundTag branch from 54fafe2 to 149cf0d Compare September 12, 2024 07:46
@NotMyFault NotMyFault merged commit 1e8778b into main Sep 14, 2024
@NotMyFault NotMyFault deleted the refactor/avoid-CompoundTag branch September 14, 2024 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants