Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: fix checkstyle warnings #24

Merged
merged 4 commits into from
Sep 9, 2021
Merged

Conversation

skaldarnar
Copy link
Contributor

  • chore: update copyright
  • chore: fix various checkstyle warnings
  • chore: fix checkstyle warnings in MultiInvStorageHandler

@skaldarnar skaldarnar added the Type: Chore Request for or implementation of maintenance changes label Aug 28, 2021
if (amount == 0) {
inventoryManager.giveItem(chest, EntityRef.NULL, item);

remaining -= byteAmount;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

before we subtracted the full amountForChest here. Looking at the code, I believe this was a potential bug, as we only give byteAmount many items to the chest.

@jdrueckert jdrueckert merged commit 15c30f1 into develop Sep 9, 2021
@jdrueckert jdrueckert deleted the chore/fix-checkstyle-warnings branch September 9, 2021 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Chore Request for or implementation of maintenance changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants