Skip to content

perf: player actions #136

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

Closed
wants to merge 2 commits into from
Closed

perf: player actions #136

wants to merge 2 commits into from

Conversation

jprzimba
Copy link
Collaborator

This PR minimize potential alocations in:

  • Game::findItemOfType
  • Game::playerLookInTrade
  • Game::removeMoney
  • Player::autoCloseContainers
  • Player::queryDestination
  • Player::removeItemOfType
  • Player::postAddNotification
  • Player::getMoney

jprzimba and others added 2 commits April 23, 2025 20:49
This PR minimize potential alocations in:

- Game::findItemOfType
- Game::playerLookInTrade
- Game::removeMoney
- Player::autoCloseContainers
- Player::queryDestination
- Player::removeItemOfType
- Player::postAddNotification
- Player::getMoney
@jprzimba jprzimba added the enhancement New feature or request label Apr 23, 2025
Copy link

@jprzimba jprzimba changed the title otimize: player actions optimize: player actions Apr 25, 2025
@jprzimba jprzimba changed the title optimize: player actions perf: player actions May 2, 2025
Copy link
Contributor

github-actions bot commented Jun 2, 2025

This PR is stale because it has been open 45 days with no activity.

@github-actions github-actions bot added the Stale label Jun 2, 2025
@jprzimba jprzimba closed this Jun 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant