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

ImGui related fixes #342

Merged
merged 2 commits into from
Sep 15, 2024
Merged

ImGui related fixes #342

merged 2 commits into from
Sep 15, 2024

Conversation

UncraftedName
Copy link
Contributor

  • Fix typo in comment #339
  • Remove unnecessary snprintf #339
  • Fix cvar name memory leak #339 & tidy code #339

I've added the SPT_MAX_CVAR_NAME_LEN macro and set it to 64. The memory leak issue is present in many features that I wrote, so I'll go through those and make a separate PR.

@UncraftedName UncraftedName changed the title Bug fix ImGui related fixes Sep 15, 2024
@lipsanen lipsanen merged commit 64aa9f4 into YaLTeR:master Sep 15, 2024
8 checks passed
@UncraftedName UncraftedName deleted the fix-imgui branch September 15, 2024 14:25
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.

2 participants