On 06/02/2011 03:10 PM, Dmitry Timoshkov wrote:
Adam Martinson<amartin...@codeweavers.com>  wrote:

+    osver.dwOSVersionInfoSize = sizeof(osver);
+    GetVersionExW(&osver);
+
...
+    if (osver.dwMajorVersion>= 5)
Once again, that's wrong way of detecting win9x, or please try to explain
better how the patch does what the subject describes, and for which app
it's needed.

The stuff in that block is for win2k+ mode only. Win9x doesn't add [Total|Avail]Phys to [Total|Avail]PageFile. If wine is in win9x mode it should behave like win9x, or it confuses old apps. This is needed for the 2nd patch, IIRC removing the photoshop 4 hack in GlobalMemoryStatus() screws up programs like Adobe Illustrator 8 in win98 mode without it.


Reply via email to