[Bug 28153] New: AutoIt - @DesktopRefresh

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

Summary: AutoIt - @DesktopRefresh
           Product: Wine
           Version: 1.3.15
          Platform: x86
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

In wine-1.2.2 and wine-1.3.15 on Ubuntu @DesktopRefresh for autoit does not
give correct output.

sample script: 
MsgBox(0x0, "Desktop Refresh", @DesktopRefresh)

Expected result: 
Message Box showing Desktop Refresh

Actual result: 
Message Box showing Desktop Refresh as 0

[Bug 28153] AutoIt - @DesktopRefresh

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

--- Comment #1 from Austin English  2013-11-13 16:49:31 CST ---
This is your friendly reminder that there has been no bug activity for 2 years.
Is this still an issue in current (1.7.6 or newer) wine? If so, please attach
the terminal output in 1.7.6 (see http://wiki.winehq.org/FAQ#get_log).

[Bug 28153] AutoIt - @DesktopRefresh

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

--- Comment #2 from Robert Maehl  ---
Created attachment 47040
  --> http://bugs.winehq.org/attachment.cgi?id=47040
Debugging Log

Tested on wine 1.6 on a 64-bit machine. It's running
1.7.1-actually1.6-0ubuntu1? Anyway, like bug #28154 it still doesn't work.

I will be testing this bug and #28154 on wine 1.7 on another machine later on
and will update these 2 tickets accordingly.

[Bug 28153] AutoIt v3.x @DesktopRefresh macro returns 0 (device caps VREFRESH hard-coded to 0)

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

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download
             Status|UNCONFIRMED                 |NEW
                URL|                            |http://www.autoitscript.com
                   |                            |/site/autoit/downloads/
                 CC|                            |[e-mail]
          Component|-unknown                    |winex11.drv
            Summary|AutoIt - @DesktopRefresh    |AutoIt v3.x @DesktopRefresh
                   |                            |macro returns 0 (device
                   |                            |caps VREFRESH hard-coded to
                   |                            |0)
     Ever confirmed|0                           |1

--- Comment #3 from Anastasius Focht  ---
Hello folks,

confirming.

Macro Reference for AutoIt: http://www.autoitscript.com/autoit3/docs/macros.htm

--- quote ---
@DesktopRefresh        Refresh rate of the primary display in hertz.
--- quote ---

Relevant part of trace log:

--- snip ---
$ pwd
/home/focht/.wine/drive_c/Program Files/AutoIt3

$ WINEDEBUG=+tid,+seh,+relay,+x11dr wine ./AutoIt3.exe test.au3 >>log.txt 2>&1
...
0009:Call user32.GetDesktopWindow() ret=0047de0b
0009:Ret  user32.GetDesktopWindow() retval=00010020 ret=0047de0b
0009:Call user32.GetDC(00010020) ret=0047de15
0009:Call
winex11.drv.GetDC(00160045,00010020,00010020,008ef1a8,008ef198,0000001a)
ret=7eb2656a
0009:Ret  winex11.drv.GetDC() retval=00000001 ret=7eb2656a
0009:Call winex11.drv.EnumDisplayMonitors(00000000,00000000,7eb35888,008ef0cc)
ret=7eb1f0d1
0009:Ret  winex11.drv.EnumDisplayMonitors() retval=00000001 ret=7eb1f0d1
0009:Ret  user32.GetDC() retval=00160045 ret=0047de15
0009:Call gdi32.GetDeviceCaps(00160045,00000074) ret=0047de20
0009:Ret  gdi32.GetDeviceCaps() retval=00000000 ret=0047de20 
...
0009:Call user32.MessageBoxW(00000000,0014f960 L"0",00148bd0 L"Desktop
Refresh",00010000) ret=004592cc
--- snip ---

Source:
http://source.winehq.org/git/wine.git/blob/0f03f264b772e8638d4f1311a2cbdfc51...

--- snip ---
194 static INT X11DRV_GetDeviceCaps( PHYSDEV dev, INT cap )
195 {
196     switch(cap)
197     {
...
295     case NUMRESERVED:
296     case PHYSICALWIDTH:
297     case PHYSICALHEIGHT:
298     case PHYSICALOFFSETX:
299     case PHYSICALOFFSETY:
300     case SCALINGFACTORX:
301     case SCALINGFACTORY:
302     case VREFRESH:
303     case BLTALIGNMENT:
304         return 0;
305     default:
306         FIXME("(%p): unsupported capability %d, will return 0\n", dev->hdc,
cap );
307         return 0;
308     }
309 }
--- snip ---

Various settings are already managed with registry (setup_options,
read_registry_settings -> DefaultSettings.VRefresh).

$ sha1sum autoit-v3-setup.exe
ba7499fe9fb55438348ac013109febfc12c73437  autoit-v3-setup.exe

$ du -sh autoit-v3-setup.exe
11M    autoit-v3-setup.exe

$ wine --version
wine-1.7.9-209-gb231b4b

Regards

[Bug 28153] AutoIt v3.x @DesktopRefresh macro returns 0 (device caps VREFRESH hard-coded to 0)

https://bugs.winehq.org/show_bug.cgi?id=28153

[e-mail] changed:

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

--- Comment #4 from [e-mail] ---
still an issue 1.7.51

[Bug 28153] AutoIt v3.x @DesktopRefresh macro returns 0 ( device caps VREFRESH hard-coded to 0)

https://bugs.winehq.org/show_bug.cgi?id=28153

Vincent Povirk  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from Vincent Povirk  ---
In wine from the master branch (wine-4.10-279-g48f93bc3290), this outputs 1,
which I think is a reasonable value. I'm going to say this is fixed.

[Bug 28153] AutoIt v3.x @DesktopRefresh macro returns 0 ( device caps VREFRESH hard-coded to 0)

https://bugs.winehq.org/show_bug.cgi?id=28153

Alexandre Julliard  changed:

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

--- Comment #6 from Alexandre Julliard  ---
Closing bugs fixed in 4.11.