Skip to content

completion: Use a shared locale instance#116

Merged
Waguramu merged 1 commit intomainfrom
use-shared-locale
Jul 21, 2025
Merged

completion: Use a shared locale instance#116
Waguramu merged 1 commit intomainfrom
use-shared-locale

Conversation

@johannes-wolf
Copy link
Collaborator

@johannes-wolf johannes-wolf commented Jul 20, 2025

Important fix. Creating lots of std::locale in a loop leads to an overflow_error (at least in WASM).

@sonarqubecloud
Copy link

@johannes-wolf johannes-wolf added the bug 🐛 Something isn't working label Jul 20, 2025
@github-actions
Copy link

Test Results

 1 files  ±0   1 suites  ±0   2m 53s ⏱️ +2s
52 tests ±0  52 ✅ ±0  0 💤 ±0  0 ❌ ±0 
57 runs  ±0  57 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 07d9f9d. ± Comparison against base commit ff14792.

@github-actions
Copy link

Package Line Rate Branch Rate Health
include.simfil 67% 48%
include.simfil.model 91% 53%
src 70% 42%
src.model 83% 48%
Summary 73% (2845 / 3877) 44% (2521 / 5741)

Minimum allowed line rate is 50%

@Waguramu Waguramu self-requested a review July 21, 2025 10:48
Copy link
Contributor

@Waguramu Waguramu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍

@Waguramu Waguramu merged commit 9cc05c6 into main Jul 21, 2025
9 checks passed
@Waguramu Waguramu deleted the use-shared-locale branch July 21, 2025 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🐛 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants