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

Fix outfit window crash #4422

Closed
wants to merge 1 commit into from
Closed

Conversation

maattch
Copy link
Contributor

@maattch maattch commented Apr 24, 2023

Pull Request Prelude

Changes Proposed

Issues addressed:
Fix #4383 introduced after the changes in #4079.
Since ProtocolOutfit holds a reference to a string, we cannot pass it directly.

@omarcopires
Copy link
Contributor

Random mount is also a problem in case of gm outfit. Perhaps it should be blocked in this case, or simply remove the gm outfit for good, as it is well known that it has been discontinued by cip.

@MillhioreBT
Copy link
Contributor

MillhioreBT commented Apr 24, 2023

Pull Request Prelude

Changes Proposed

Issues addressed: Fix #4383 introduced after the changes in #4079. Since ProtocolOutfit holds a reference to a string, we cannot pass it directly.

How can the crash be reproduced?
because it works normally for me, and PR #4400 fixes the name problem.

Edit:

Ok I found the problem in the podiums, you were right, I didn't do more tests but probably there would also be more problems if there were several players with Access when trying to see the window.

I opened a PR #4425 with some changes that solved the problem, but we have to wait for some expert on the subject to decide if your PR is better or maybe mine. since I do not fully understand regarding this type of optimizations

@maattch
Copy link
Contributor Author

maattch commented Apr 25, 2023

Closed. A better fix was made in #4425

@maattch maattch closed this Apr 25, 2023
@maattch maattch deleted the fix-outfit-crash branch May 15, 2023 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Crash when opening outfits window with GM
3 participants