printf() can strip sign of -INF #15964

Activity

1. divinity76

   added

   Bug

   Status: Needs Triage

   on Sep 20, 2024
2. divinity76

   changed the title [-]sprintf %.17g negative infinty bug[/-] [+]sprintf %.17g negative infinity bug[/+] on Sep 20, 2024
3. divinity76 commented on Sep 20, 2024

   

   divinity76

   on Sep 20, 2024

   ContributorAuthor

   More actions

   duplicate of #9209
4. divinity76

   closed this as not plannedon Sep 20, 2024
5. cmb69 commented on Sep 20, 2024

   

   cmb69

   on Sep 20, 2024

   Member

   More actions

   duplicate of #9209

   Nope, not really. That is about truncation of INF due to very low precision, but the reported issue here is about loosing the minus sign.

   👍React with 👍1Reacted by divinity76
6. cmb69

   reopened this on Sep 20, 2024
7. cmb69

   added

   Extension: standard

   Status: Verified

   and removed

   Status: Needs Triage

   on Sep 20, 2024
8. cmb69 commented on Sep 20, 2024

   

   cmb69

   on Sep 20, 2024

   Member

   More actions

   Also note that the problem is not particularly related to the precision, see https://3v4l.org/3F8nm. The only case where the minus sign would properly be retained is for padding="0", e.g. printf("%06g", -INF);, but that apparently looses the "I". Having had more test cases could have helped.
9. cmb69

   changed the title [-]sprintf %.17g negative infinity bug[/-] [+]printf() can strip sign of -INF[/+] on Dec 2, 2024
10. cmb69

    added a commit that references this issue on Dec 2, 2024

    Fix GH-15964: printf() can strip sign of -INF

    ...

    This issue will close once commit c2d3734 is merged into the 'master' branch.

    c2d3734
11. cmb69

    closed this as completedin c2d3734on Dec 2, 2024
12. cmb69

    added a commit that references this issue on Dec 2, 2024

    Merge branch 'PHP-8.4'

    ...

    This issue will close once commit 3b517e0 is merged into the 'master' branch.

    3b517e0