Skip to content

mprintf: fix long double output - #22706

Closed
bagder wants to merge 7 commits into
masterfrom
bagder/mprintf-long-double
Closed

bagder wants to merge 7 commits into
masterfrom
bagder/mprintf-long-double

Conversation

@bagder

@bagder bagder commented Aug 27, 2026

Copy link
Copy Markdown
Member

curl does not use this, and we discourage users from using these functions but since we claim printf() compatibility this still seems worth fixing.

Verified in test 557.

Reported-by: Stanislav Fort

@github-actions github-actions Bot added the tests label Aug 27, 2026
Comment thread tests/libtest/lib557.c Fixed
@testclutch

Copy link
Copy Markdown

Analysis of PR #22706 at ef945792:

Test 557 failed, which has NOT been flaky recently, so there could be a real issue in this PR. Note that this test has failed in 498 different CI jobs (the link just goes to one of them).

Test 599 failed, which has NOT been flaky recently, so there could be a real issue in this PR. Note that this test has failed in 16 different CI jobs (the link just goes to one of them).

Test 1148 failed, which has NOT been flaky recently, so there could be a real issue in this PR. Note that this test has failed in 12 different CI jobs (the link just goes to one of them).

Generated by Testclutch

bagder added 3 commits August 27, 2026 14:01
curl does not use this, and we discourage users from using these
functions but since we claim printf() compatibility this still
seems worth fixing.

Verified in test 557.

Reported-by: Stanislav Fort
@bagder
bagder force-pushed the bagder/mprintf-long-double branch from ef94579 to 4a14cda Compare August 27, 2026 12:03
@bagder

bagder commented Aug 27, 2026

Copy link
Copy Markdown
Member Author

Different libc versions/installs have different output. Of course. I need to reduce the accuracy of the check...

@bagder

bagder commented Aug 27, 2026

Copy link
Copy Markdown
Member Author

This makes me very tired...

@bagder bagder closed this in bdda3a2 Aug 27, 2026
@bagder
bagder deleted the bagder/mprintf-long-double branch August 27, 2026 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants