This is the mail archive of the ecos-bugs@sourceware.org mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug 20804] Misbehavior of printf %e format.


Please do not reply to this email. Use the web interface provided at:
http://bugs.ecos.sourceware.org/show_bug.cgi?id=20804

Jonathan Larmour <jifl@ecoscentric.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jifl@ecoscentric.com

--- Comment #7 from Jonathan Larmour <jifl@ecoscentric.com> 2012-08-13 21:57:40 BST ---
Yes I object, looking at the patch, I don't think it's right. I don't see the
reason for this:

-                                if ((ch != 'g' && ch != 'G') || (flags&ALT))
+                                if (ch == 'f')

The precision (e.g. 18 in "%.18f") is defined as the number of digits to appear
after the radix character for e, E, and f conversions, or the maximum number of
significant digits for g and G conversions. I don't see how the above can fit
into that, nor do I have confidence that the ALT flag ('#') is correctly
handled.

And Bernd seems to have doubts too - he says in bug 1001490 it's only an
interim solution. I don't want to be swapping one set of problems for a
different but now less known set of problems.

Unfortunately I don't have the time to look at this at the moment as I'm
leaving on holiday soon and have to finish work things off. But I'd want to see
confidence that the changes made are correct before committing.

Jifl

-- 
Configure bugmail: http://bugs.ecos.sourceware.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]