[Bug 37706] New: ScrollWindowEx() returns ERROR if the window is not visible (in the Windows API sense); real Windows returns NULLREGION
https://bugs.winehq.org/show_bug.cgi?id=37706 Bug ID: 37706 Summary: ScrollWindowEx() returns ERROR if the window is not visible (in the Windows API sense); real Windows returns NULLREGION Product: Wine Version: 1.7.28 Hardware: x86 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: user32 Assignee: wine-bugs(a)winehq.org Reporter: pietro10(a)mac.com Distribution: --- Created attachment 50192 --> https://bugs.winehq.org/attachment.cgi?id=50192 Sample program that demonstrates bug (source) I'm using a dummy scroll (as in http://blogs.msdn.com/b/oldnewthing/archive/2003/07/31/54601.aspx) to update the scrollbars in my custom Windows control when necessary. In my test program, this happens before the window is shown for the first time, and in wine, my error checking code triggers because the ScrollWindowEx() call fails with ERROR. This does not happen on real Windows; it instead returns NULLREGION. The attached program creates several configurations that attempt to reproduce this issue under different circumstances. On wine, I get ScrollWindowEx() before ShowWindow(nCmdShow): ERROR ScrollWindowEx() after ShowWindow(nCmdShow) and ShowWindow(SW_HIDE): ERROR ScrollWindowEx() after only ShowWindow(nCmdShow): SIMPLEREGION On Windows XP 32-bit, Windows 7 32-bit, and Windows 7 64-bit, I get ScrollWindowEx() before ShowWindow(nCmdShow): NULLREGION ScrollWindowEx() after ShowWindow(nCmdShow) and ShowWindow(SW_HIDE): NULLREGION ScrollWindowEx() after only ShowWindow(nCmdShow): SIMPLEREGION I can't find anything pointing to this use of ScrollWindowEx() being wrong... I also do not know if any other scroll functions are equally affected. I am running wine-1.7.28 from the Wine PPA on Ubuntu GNOME 14.10. Thanks. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37706 --- Comment #1 from Pietro Gagliardi (andlabs) <pietro10(a)mac.com> --- Created attachment 50193 --> https://bugs.winehq.org/attachment.cgi?id=50193 Executable of said example program -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37706 --- Comment #2 from Bruno Jesus <00cpxxx(a)gmail.com> --- I'm in a Windows 8.1 machine right now, so just to increase tests: ScrollWindowEx() before ShowWindow(nCmdShow): NULLREGION ScrollWindowEx() after ShowWindow(nCmdShow) and ShowWindow(SW_HIDE): NULLREGION ScrollWindowEx() after only ShowWindow(nCmdShow): SIMPLEREGION -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37706 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download, testcase -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37706 Sebastian Lackner <sebastian(a)fds-team.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sebastian(a)fds-team.de -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37706 --- Comment #3 from Dmitry Timoshkov <dmitry(a)baikal.ru> --- Created attachment 50197 --> https://bugs.winehq.org/attachment.cgi?id=50197 patch Something like the attached patch should help, but it's probably worth adding some tests. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37706 --- Comment #4 from Sebastian Lackner <sebastian(a)fds-team.de> --- @Dmitry: Patch works fine here, thx. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37706 --- Comment #5 from Pietro Gagliardi (andlabs) <pietro10(a)mac.com> --- For documentation purposes, when this does get patched into wine, would someone please point out which version it winds up being patched into? Thanks. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37706 Anastasius Focht <focht(a)gmx.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch Status|UNCONFIRMED |NEW CC| |focht(a)gmx.net Ever confirmed|0 |1 --- Comment #6 from Anastasius Focht <focht(a)gmx.net> --- Hello folks, confirming on behalf of others. Still present as of Wine 1.7.43 Dmitry's patch is also available/included in Wine-Staging by default: https://github.com/wine-compholio/wine-staging/blob/master/patches/user32-Sc... Regards -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37706 Michael Müller <michael(a)fds-team.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |STAGED CC| |michael(a)fds-team.de Staged patchset| |https://github.com/wine-com | |pholio/wine-staging/tree/ma | |ster/patches/user32-ScrollW | |indowEx -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37706 André H. <nerv(a)dawncrow.de> changed: What |Removed |Added ---------------------------------------------------------------------------- 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-ScrollW |er/patches/user32-ScrollWin |indowEx |dowEx CC| |nerv(a)dawncrow.de -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=37706 joaopa <jeremielapuree(a)yahoo.fr> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jeremielapuree(a)yahoo.fr --- Comment #7 from joaopa <jeremielapuree(a)yahoo.fr> --- Bug still occurs with wine-5.3 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=37706 Dmitry Timoshkov <dmitry(a)baikal.ru> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Fixed by SHA1| |2fc1b63c63ac3752f620e972e4c | |093e7cb2fdfdd Status|STAGED |RESOLVED --- Comment #8 from Dmitry Timoshkov <dmitry(a)baikal.ru> --- Should be fixed by 2fc1b63c63ac3752f620e972e4c093e7cb2fdfdd. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=37706 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #9 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 10.7. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (2)
-
wine-bugs＠winehq.org -
WineHQ Bugzilla