Skip to content

Add string count to stat_memory and OOM handler#337

Merged
themrdemonized merged 2 commits intothemrdemonized:landerfrom
Lander-Modding:stat-memory-string-count
Aug 15, 2025
Merged

Add string count to stat_memory and OOM handler#337
themrdemonized merged 2 commits intothemrdemonized:landerfrom
Lander-Modding:stat-memory-string-count

Conversation

@ProfLander
Copy link
Contributor

@ProfLander ProfLander commented Aug 15, 2025 •
edited
Loading

  • Thread a u32 reference through stat_economy to avoid adding a global
  • Rearrange existing Msg calls to segment string and shared, add count to string

Xottab-DUTY
Xottab-DUTY reviewed Aug 15, 2025
View reviewed changes
@ProfLander @Xottab-DUTY
More explicit wording for string memory stats
f8bd955 
Co-authored-by: Sultan Luchezarniy <Xottab-DUTY@users.noreply.github.com>
@themrdemonized themrdemonized changed the base branch from all-in-one-vs2022-wpo to lander August 15, 2025 23:54
@themrdemonized themrdemonized merged commit 13b3caa into themrdemonized:lander Aug 15, 2025
9 of 10 checks passed
Xottab-DUTY added a commit to OpenXRay/xray-16 that referenced this pull request Aug 18, 2025
@Xottab-DUTY
Add string count to stat_memory and OOM handler
f63674d 
Almost same as themrdemonized/xray-monolith#337
Xottab-DUTY added a commit to OpenXRay/xray-16 that referenced this pull request Aug 18, 2025
@Xottab-DUTY
Add string count to stat_memory and OOM handler
4005b54 
Almost same as themrdemonized/xray-monolith#337
themrdemonized added a commit that referenced this pull request Jan 24, 2026
@themrdemonized
Merge pull request #337 from ProfLander/stat-memory-string-count
409a2b7 
Add string count to `stat_memory` and OOM handler
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@Xottab-DUTY Xottab-DUTY Xottab-DUTY left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ProfLander @Xottab-DUTY @themrdemonized