https://bugs.winehq.org/show_bug.cgi?id=42010
Bug ID: 42010 Summary: ReactOS Calc does not show dots in radio buttons. Product: Wine Version: 1.9.24 Hardware: x86 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: user32 Assignee: wine-bugs@winehq.org Reporter: carlo.bramix@libero.it Distribution: ---
Created attachment 56449 --> https://bugs.winehq.org/attachment.cgi?id=56449 ReactOS Calc running in WINE.
I tried to run my ReactOS Calc on WINE. It works fine but it does not show the dots inside the radio buttons. It works fine from Windows 9x to Windows 7, but not in WINE. Tested in lubuntu with latest trunk.
https://bugs.winehq.org/show_bug.cgi?id=42010
Fabian Maurer dark.shadow4@web.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dark.shadow4@web.de
--- Comment #1 from Fabian Maurer dark.shadow4@web.de --- Can confirm this issue. Also, if you select the "Hex" option, the Qword/Dword/Word/Byte buttons work.
https://bugs.winehq.org/show_bug.cgi?id=42010
winetest@luukku.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |winetest@luukku.com
--- Comment #2 from winetest@luukku.com --- Possible dupe of bug 16845 it contains a patch. Could you test that? Also possible dupe of bug 32542.
https://bugs.winehq.org/show_bug.cgi?id=42010
--- Comment #3 from Fabian Maurer dark.shadow4@web.de --- The patch won't work since it's for a keyboard issue, but I'm using mouse to click the buttons.
https://bugs.winehq.org/show_bug.cgi?id=42010
--- Comment #4 from Austin English austinenglish@gmail.com --- Could you attach/link to the ReactOS calc.exe binary?
https://bugs.winehq.org/show_bug.cgi?id=42010
Dmitry Timoshkov dmitry@baikal.ru changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW Ever confirmed|0 |1
--- Comment #5 from Dmitry Timoshkov dmitry@baikal.ru --- 1. calc sources from ReactOS won't compile neither with PSDK nor Wine headers, you may want to fix that (calc.h should include winbase.h after windef.h). 2. the problem with radio buttons is caused by broken logic in dlls/user32/ button.c,BM_SETCHECK handler. In particular BUTTON_CheckAutoRadioButton() helper does weird things, simply disabling this call fixes the bug.
https://bugs.winehq.org/show_bug.cgi?id=42010
--- Comment #6 from Dmitry Timoshkov dmitry@baikal.ru --- (In reply to Dmitry Timoshkov from comment #5)
- the problem with radio buttons is caused by broken logic in dlls/user32/
button.c,BM_SETCHECK handler. In particular BUTTON_CheckAutoRadioButton() helper does weird things, simply disabling this call fixes the bug.
That happens because ros calc managed radio buttons state manually: and it happens that it first turns on some button with BM_SETCHECK/1, and later turns on another button in the same group with BM_SETCHECK/1, and the latter one causes the first button to be turned off.
In my (pretty limited) testing I could observe something that resembles what BUTTON_CheckAutoRadioButton() is doing only on user interaction, i.e. with keyboard and mouse clicks. Perhaps managing button state programmatically with BM_SETCHECK shouldn't lead to automatic button state handling in user32.
All of this needs quite a bit of test cases.
https://bugs.winehq.org/show_bug.cgi?id=42010
--- Comment #7 from Dmitry Timoshkov dmitry@baikal.ru --- Created attachment 56468 --> https://bugs.winehq.org/attachment.cgi?id=56468 patch
Attached patch seems to fix the problem for me.
https://bugs.winehq.org/show_bug.cgi?id=42010
--- Comment #8 from Carlo Bramini carlo.bramix@libero.it --- ReactOS Calc can be found here:
https://sourceforge.net/projects/reactos-calc/files/reactos-calc/ReactOS%20C...
The link includes sources, binaries ASCII and UNICODE, with IEEE or GMP support as math library.
https://bugs.winehq.org/show_bug.cgi?id=42010
--- Comment #9 from Carlo Bramini carlo.bramix@libero.it ---
- calc sources from ReactOS won't compile neither with PSDK nor Wine headers
Actually this is a change made by ReactOS team, my original sources included just windows.h to be "universal".
https://bugs.winehq.org/show_bug.cgi?id=42010
--- Comment #10 from Carlo Bramini carlo.bramix@libero.it --- Created attachment 56473 --> https://bugs.winehq.org/attachment.cgi?id=56473 Precompiled binary of ReactOS Calc.
https://bugs.winehq.org/show_bug.cgi?id=42010
Dmitry Timoshkov dmitry@baikal.ru changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #56468|0 |1 is obsolete| |
--- Comment #11 from Dmitry Timoshkov dmitry@baikal.ru --- Created attachment 56475 --> https://bugs.winehq.org/attachment.cgi?id=56475 patch
Updated patch with redundant check removed.
https://bugs.winehq.org/show_bug.cgi?id=42010
WineBuG winebugs140@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |winebugs140@gmail.com
--- Comment #12 from WineBuG winebugs140@gmail.com --- Is this a DUPLICATE of Bug 32542?
https://bugs.winehq.org/show_bug.cgi?id=42010
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download, source URL| |https://sourceforge.net/pro | |jects/reactos-calc/files/re | |actos-calc/ReactOS%20Calc%2 | |01.11/
https://bugs.winehq.org/show_bug.cgi?id=42010
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch
https://bugs.winehq.org/show_bug.cgi?id=42010
--- Comment #13 from Carlo Bramini carlo.bramix@libero.it --- The patch written by Dmitry Timoshkov has resolved this issue on my side.
https://bugs.winehq.org/show_bug.cgi?id=42010
Sebastian Lackner sebastian@fds-team.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |dmitry@baikal.ru, | |erich.e.hoover@wine-staging | |.com, michael@fds-team.de, | |sebastian@fds-team.de Status|NEW |STAGED Staged patchset| |https://github.com/wine-com | |pholio/wine-staging/tree/ma | |ster/patches/user32-Auto_Ra | |dio_Button
https://bugs.winehq.org/show_bug.cgi?id=42010
--- Comment #14 from WineBuG winebugs140@gmail.com --- This patch also fixes the Bug 32542. Now it really looks like a DUPLICATE.
https://bugs.winehq.org/show_bug.cgi?id=42010
Rosanne DiMesio dimesio@earthlink.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |bkbk9@yahoo.ca
--- Comment #15 from Rosanne DiMesio dimesio@earthlink.net --- *** Bug 32542 has been marked as a duplicate of this bug. ***
https://bugs.winehq.org/show_bug.cgi?id=42010
Jeff Zaroyko jeffz@jeffz.name changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |asko.havumaki@gmail.com
--- Comment #16 from Jeff Zaroyko jeffz@jeffz.name --- *** Bug 34370 has been marked as a duplicate of this bug. ***
https://bugs.winehq.org/show_bug.cgi?id=42010
André H. nerv@dawncrow.de changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |nerv@dawncrow.de Staged patchset|https://github.com/wine-com |https://github.com/wine-sta |pholio/wine-staging/tree/ma |ging/wine-staging/tree/mast |ster/patches/user32-Auto_Ra |er/patches/user32-Auto_Radi |dio_Button |o_Button
https://bugs.winehq.org/show_bug.cgi?id=42010
Dmitry Timoshkov dmitry@baikal.ru changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Fixed by SHA1| |96d0af52eb0d14084397647b974 | |c5efebb59d0f0 Status|STAGED |RESOLVED
--- Comment #17 from Dmitry Timoshkov dmitry@baikal.ru --- Committed as 96d0af52eb0d14084397647b974c5efebb59d0f0.
https://bugs.winehq.org/show_bug.cgi?id=42010
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #18 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 4.0-rc1.
https://bugs.winehq.org/show_bug.cgi?id=42010
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |3.0.x
https://bugs.winehq.org/show_bug.cgi?id=42010
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|3.0.x |---
--- Comment #19 from Michael Stefaniuc mstefani@winehq.org --- Removing the 3.0.x milestone from bug fixes included in 3.0.5.