[Bug 31542] New: msvcrt sprintf doesn't respect " " flag for E format.

http://bugs.winehq.org/show_bug.cgi?id=31542

Bug #: 31542
           Summary: msvcrt sprintf doesn't respect " " flag for E format.
           Product: Wine
           Version: unspecified
          Platform: x86
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: msvcrt
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
    Classification: Unclassified

This problem was discovered by pitt at the QB64 forums:
http://www.qb64.net/forum/index.php?topic=6886.0

For a format string of "% .6E", and a value of .00314159265359, sprintf is
supposed to write " 3.141593E-003" to the buffer, and return 14. Instead, on
Wine, it seems to be writing "3.141593E-003", and returning 13. It is omitting
the leading space.

I am unable to test this, as I am not a Wine user myself. I am relying on
information from pitt, and I do not Wine version. His Linux version is Fedora
16, but I don't know whether 32 or 64 bit.

For what it's worth, I tried it in ReactOS .3.14, and it worked correctly.

Could someone please verify this problem? If this has already been fixed in a
new version, please say so.

Regards,
Michael

[Bug 31542] msvcrt sprintf doesn't respect " " flag for E format.

http://bugs.winehq.org/show_bug.cgi?id=31542

Vijay Kamuju  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #1 from Vijay Kamuju  2012-08-24 08:40:59 CDT ---
Please attach a small test windows executable :)

[Bug 31542] msvcrt sprintf doesn't respect " " flag for E format.

http://bugs.winehq.org/show_bug.cgi?id=31542

--- Comment #2 from Michael Calkins  2012-08-24 18:21:43 CDT ---
Created attachment 41477
  --> http://bugs.winehq.org/attachment.cgi?id=41477
two executables with source

the attached .7z file contains 2 executables, one C++ source file which can be
built with mingw-w32, and one assembly source file which can be assembled with
Nasm, and linked with mingw-w32's ld.

winet.exe should display:
" 3.141593E-003" 14
" 3.141593E-003" 14

wineta.exe should display:
" 0.000000E+000" 14

I'm going to try to install FreeBSD and Wine on a spare computer, so that I can
test this myself.

Regards,
Michael

[Bug 31542] msvcrt sprintf doesn't respect " " flag for E format.

http://bugs.winehq.org/show_bug.cgi?id=31542

--- Comment #3 from Michael Calkins  2012-08-25 19:02:31 CDT ---
Created attachment 41482
  --> http://bugs.winehq.org/attachment.cgi?id=41482
photo and screenshot

I have confirmed the problem using Wine 1.3.34 on FreeBSD 9.0 i686. This
doesn't seem to be the latest version of Wine, but it was the version that got
installed when I ran "cd /usr/ports/emulators/wine/ && make install clean" as
instructed at http://www.freshports.org/emulators/wine/. Please confirm whether
this problem has been fixed in newer versions, or still exists.

The .7z file contains a photo of the monitor of the FreeBSD computer, and a
partial screenshot from an XP computer.

Regards,
Michael

[Bug 31542] msvcrt sprintf doesn't respect " " flag for E format.

http://bugs.winehq.org/show_bug.cgi?id=31542

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #4 from Dan Kegel  2012-08-26 16:20:52 CDT ---
winet.exe doesn't run here, it complains
err:module:import_dll Library libgcc_s_sjlj-1.dll (which is needed by
L"Z:\home\dank\Downloads\winet.exe") not found

but wineta.exe does.  Does the attached patch help your real app?

[Bug 31542] msvcrt sprintf doesn't respect " " flag for E format.

http://bugs.winehq.org/show_bug.cgi?id=31542

--- Comment #5 from Dan Kegel  2012-08-26 16:22:03 CDT ---
Created attachment 41502
  --> http://bugs.winehq.org/attachment.cgi?id=41502
Kludge patch to make " " flag work in this case

[Bug 31542] msvcrt sprintf doesn't respect " " flag for E format.

http://bugs.winehq.org/show_bug.cgi?id=31542

--- Comment #6 from Michael Calkins  2012-08-26 17:12:00 CDT ---
Sorry, I forgot the .DLLs. I believe that testa.exe is a sufficient test.

Thank you for the patch. I'll try to test it tomorrow, and I hope that pitt can
test it on the real app.

Regards,
Michael

[Bug 31542] msvcrt sprintf doesn't respect " " flag for E format.

http://bugs.winehq.org/show_bug.cgi?id=31542

Piotr Caban  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |d958ecbc6931cc560b5b2c562a4
                   |                            |bcd1b1b84a68d
             Status|UNCONFIRMED                 |RESOLVED
                 CC|                            |[e-mail]
         Resolution|                            |FIXED

--- Comment #7 from Piotr Caban  2012-12-12 15:52:01 CST ---
Fixed by commit d958ecbc6931cc560b5b2c562a4bcd1b1b84a68d.

[Bug 31542] msvcrt sprintf doesn't respect " " flag for E format.

http://bugs.winehq.org/show_bug.cgi?id=31542

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #8 from Alexandre Julliard  2012-12-21 13:28:36 CST ---
Closing bugs fixed in 1.5.20.