Skip to content
This repository was archived by the owner on Apr 29, 2025. It is now read-only.

fix(server/player): handle player loading errors and cleanup#221

Merged
thelindat merged 1 commit intooverextended:mainfrom
subhampro:main
Jan 21, 2025
Merged

fix(server/player): handle player loading errors and cleanup#221
thelindat merged 1 commit intooverextended:mainfrom
subhampro:main

Conversation

@subhampro
Copy link
Contributor

Mainly done in this changes :

Added player variable declaration outside try block for proper scope
Added cleanup in catch block to prevent orphaned player instances
Maintained all existing return values and error messages
Added nested try-catch for cleanup operations
Kept all property assignments within main try block
Added proper error logging for both main error and cleanup errors

1 > Fix the build issue.
2 > Maintain all existing functionality.
3 > Prevent memory leaks.
4 > Provide better error handling.
5 > Keep all existing interfaces intact.

@thelindat thelindat merged commit 7a54904 into overextended:main Jan 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants