Skip to content

Conversation

@bazineta
Copy link

A number of fixes and improvements to statistics. Closes #1147 and #1151. While this does not include a fix for #1146, there's an existing PR (#1186) for that issue. Note that while the documentation states that PRs should be landed against dev, issue #1147 necessitates landing this against dev3.

A number of fixes and improvements to statistics. Closes microsoft#1147 and microsoft#1151. While this does not include a fix for microsoft#1146, there's an existing PR (microsoft#1186) for that issue. Note that while the documentation states that PRs should be landed against dev, issue microsoft#1147 necessitates landing this against dev3.
Jarred-Sumner added a commit to oven-sh/mimalloc that referenced this pull request Jan 6, 2026
daanx added a commit that referenced this pull request Jan 8, 2026
@daanx
Copy link
Collaborator

daanx commented Jan 8, 2026

Hi @bazineta, thank you for the fixes and the earlier issues, and apologies for the late integration :( I addressed #1146 / #1186 separately, and merged your other changes directly onto dev -- I like your clean solution with the mi_stat_average_t but since it is only needed for the searches I opted for the simpler solution by just adding a counter (which also makes it easier to merge downstream). Thanks again!

@daanx daanx closed this Jan 8, 2026
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