December 2013 - Wine-Commits

Ken Thomases : winemac: Track whether our windows would be visible if the process weren't hidden.
by Alexandre Julliard Dec. 31, 2013

Dec. 31, 2013

Module: wine Branch: master Commit: ae4732360460914da61bedbed798511882d84719 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ae4732360460914da61bedbed… Author: Ken Thomases <ken(a)codeweavers.com> Date: Tue Dec 31 01:05:09 2013 -0600 winemac: Track whether our windows would be visible if the process weren't hidden. The -[NSWindow isVisible] method returns FALSE when the process is hidden, but that's not what we need to know in some cases. This fixes full-screen games which minimize their window when they lose focus. Command-Tabbing away hides the process. Because the window was not visible, the code didn't actually minimize it. When switching back to the process, no event was sent to the Wine back-end telling it the window had been restored, so it never resumed drawing to it. --- dlls/winemac.drv/cocoa_window.h | 1 + dlls/winemac.drv/cocoa_window.m | 40 +++++++++++++++++++++++++++++++++----- 2 files changed, 35 insertions(+), 6 deletions(-) diff --git a/dlls/winemac.drv/cocoa_window.h b/dlls/winemac.drv/cocoa_window.h index d3a244b..a74f6da 100644 --- a/dlls/winemac.drv/cocoa_window.h +++ b/dlls/winemac.drv/cocoa_window.h @@ -33,6 +33,7 @@ BOOL maximized; BOOL fullscreen; BOOL pendingMinimize; + BOOL savedVisibleState; WineWindow* latentParentWindow; NSMutableArray* latentChildWindows; diff --git a/dlls/winemac.drv/cocoa_window.m b/dlls/winemac.drv/cocoa_window.m index 002e359..a025d59 100644 --- a/dlls/winemac.drv/cocoa_window.m +++ b/dlls/winemac.drv/cocoa_window.m @@ -543,6 +543,7 @@ static inline NSUInteger adjusted_modifiers_for_option_behavior(NSUInteger modif WineWindow* window; WineContentView* contentView; NSTrackingArea* trackingArea; + NSNotificationCenter* nc = [NSNotificationCenter defaultCenter]; [[WineApplicationController sharedController] flipRect:&window_frame]; @@ -595,12 +596,21 @@ static inline NSUInteger adjusted_modifiers_for_option_behavior(NSUInteger modif [window setContentView:contentView]; [window setInitialFirstResponder:contentView]; - [[NSNotificationCenter defaultCenter] addObserver:window - selector:@selector(updateFullscreen) - name:NSApplicationDidChangeScreenParametersNotification - object:NSApp]; + [nc addObserver:window + selector:@selector(updateFullscreen) + name:NSApplicationDidChangeScreenParametersNotification + object:NSApp]; [window updateFullscreen]; + [nc addObserver:window + selector:@selector(applicationWillHide) + name:NSApplicationWillHideNotification + object:NSApp]; + [nc addObserver:window + selector:@selector(applicationDidUnhide) + name:NSApplicationDidUnhideNotification + object:NSApp]; + return window; } @@ -714,9 +724,15 @@ static inline NSUInteger adjusted_modifiers_for_option_behavior(NSUInteger modif [self setHasShadow:wf->shadow]; } + // Indicates if the window would be visible if the app were not hidden. + - (BOOL) wouldBeVisible + { + return [NSApp isHidden] ? savedVisibleState : [self isVisible]; + } + - (BOOL) isOrderedIn { - return [self isVisible] || [self isMiniaturized]; + return [self wouldBeVisible] || [self isMiniaturized]; } - (NSInteger) minimumLevelForActive:(BOOL)active @@ -827,7 +843,7 @@ static inline NSUInteger adjusted_modifiers_for_option_behavior(NSUInteger modif pendingMinimize = FALSE; if (state->minimized && ![self isMiniaturized]) { - if ([self isVisible]) + if ([self wouldBeVisible]) { if ([self styleMask] & NSFullScreenWindowMask) { @@ -1198,6 +1214,7 @@ static inline NSUInteger adjusted_modifiers_for_option_behavior(NSUInteger modif } else [self orderOut:nil]; + savedVisibleState = FALSE; if (wasVisible && wasOnActiveSpace && fullscreen) [controller updateFullscreenWindows]; [controller adjustWindowLevels]; @@ -1637,6 +1654,17 @@ static inline NSUInteger adjusted_modifiers_for_option_behavior(NSUInteger modif } } + - (void) applicationWillHide + { + savedVisibleState = [self isVisible]; + } + + - (void) applicationDidUnhide + { + if ([self isVisible]) + [self becameEligibleParentOrChild]; + } + /* * ---------- NSWindowDelegate methods ----------

1 0

Ken Thomases : winemac: Explicitly unhide the app early when showing windows.
by Alexandre Julliard Dec. 31, 2013

Dec. 31, 2013

Module: wine Branch: master Commit: bcde44fb8a34a9fc9bab60a8ff90700f07b0d6c4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bcde44fb8a34a9fc9bab60a8f… Author: Ken Thomases <ken(a)codeweavers.com> Date: Tue Dec 31 01:05:13 2013 -0600 winemac: Explicitly unhide the app early when showing windows. Cocoa would implictly unhide it when we order a window, anyway. Doing it early avoids problems from querying -[NSWindow isVisible] while the app is hidden. That method returns FALSE even for windows which would be visible if the app weren't hidden. --- dlls/winemac.drv/cocoa_window.m | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/dlls/winemac.drv/cocoa_window.m b/dlls/winemac.drv/cocoa_window.m index a025d59..07abbca 100644 --- a/dlls/winemac.drv/cocoa_window.m +++ b/dlls/winemac.drv/cocoa_window.m @@ -1118,9 +1118,11 @@ static inline NSUInteger adjusted_modifiers_for_option_behavior(NSUInteger modif if (![self isMiniaturized]) { BOOL needAdjustWindowLevels = FALSE; - BOOL wasVisible = [self isVisible]; + BOOL wasVisible; [controller transformProcessToForeground]; + [NSApp unhide:nil]; + wasVisible = [self isVisible]; if (activate) [NSApp activateIgnoringOtherApps:YES];

1 0

Andrey Gusev : po: Ukrainian translation updates and spelling fixes.
by Alexandre Julliard Dec. 31, 2013

Dec. 31, 2013

Module: wine Branch: master Commit: 33132ce7621dcf33551b0417cf8902a51ba9be3c URL: http://source.winehq.org/git/wine.git/?a=commit;h=33132ce7621dcf33551b0417c… Author: Andrey Gusev <andrey.goosev(a)gmail.com> Date: Tue Dec 31 16:06:34 2013 +0200 po: Ukrainian translation updates and spelling fixes. --- po/uk.po | 76 +++++++++++++++++++++++++++++++------------------------------- 1 files changed, 38 insertions(+), 38 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=33132ce7621dcf33551b0…

1 0

Dmitry Timoshkov : cabinet: Remove a no longer valid comment about seeking and restoring CAB position.
by Alexandre Julliard Dec. 31, 2013

Dec. 31, 2013

Module: wine Branch: master Commit: 3b8f4a280c904b084733dacab722199728242135 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3b8f4a280c904b084733dacab… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Wed Dec 25 13:25:04 2013 +0900 cabinet: Remove a no longer valid comment about seeking and restoring CAB position. --- dlls/cabinet/fdi.c | 30 ------------------------------ 1 files changed, 0 insertions(+), 30 deletions(-) diff --git a/dlls/cabinet/fdi.c b/dlls/cabinet/fdi.c index f9e6522..ae0576b 100644 --- a/dlls/cabinet/fdi.c +++ b/dlls/cabinet/fdi.c @@ -543,36 +543,6 @@ static BOOL FDI_read_entries( TRACE("(fdi == ^%p, hf == %ld, pfdici == ^%p)\n", fdi, hf, pfdici); - /* - * FIXME: I just noticed that I am memorizing the initial file pointer - * offset and restoring it before reading in the rest of the header - * information in the cabinet. Perhaps that's correct -- that is, perhaps - * this API is supposed to support "streaming" cabinets which are embedded - * in other files, or cabinets which begin at file offsets other than zero. - * Otherwise, I should instead go to the absolute beginning of the file. - * (Either way, the semantics of wine's FDICopy require me to leave the - * file pointer where it is afterwards -- If Windows does not do so, we - * ought to duplicate the native behavior in the FDIIsCabinet API, not here. - * - * So, the answer lies in Windows; will native cabinet.dll recognize a - * cabinet "file" embedded in another file? Note that cabextract.c does - * support this, which implies that Microsoft's might. I haven't tried it - * yet so I don't know. ATM, most of wine's FDI cabinet routines (except - * this one) would not work in this way. To fix it, we could just make the - * various references to absolute file positions in the code relative to an - * initial "beginning" offset. Because the FDICopy API doesn't take a - * file-handle like this one, we would therein need to search through the - * file for the beginning of the cabinet (as we also do in cabextract.c). - * Note that this limits us to a maximum of one cabinet per. file: the first. - * - * So, in summary: either the code below is wrong, or the rest of fdi.c is - * wrong... I cannot imagine that both are correct ;) One of these flaws - * should be fixed after determining the behavior on Windows. We ought - * to check both FDIIsCabinet and FDICopy for the right behavior. - * - * -gmt - */ - /* read in the CFHEADER */ if (fdi->read(hf, buf, cfhead_SIZEOF) != cfhead_SIZEOF) { if (pmii) set_error( fdi, FDIERROR_NOT_A_CABINET, 0 );

1 0

Dmitry Timoshkov : cabinet: Read CAB size from the file header instead of calling FDI seek callback with SEEK_END .
by Alexandre Julliard Dec. 31, 2013

Dec. 31, 2013

Module: wine Branch: master Commit: 4ddb9f691597b71cc083eef0079daf91b224b3ba URL: http://source.winehq.org/git/wine.git/?a=commit;h=4ddb9f691597b71cc083eef00… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Wed Dec 25 13:24:58 2013 +0900 cabinet: Read CAB size from the file header instead of calling FDI seek callback with SEEK_END. SEEK_END is not supported by some callback implementations. --- dlls/cabinet/fdi.c | 23 +++++------------------ dlls/cabinet/tests/fdi.c | 5 +---- 2 files changed, 6 insertions(+), 22 deletions(-) diff --git a/dlls/cabinet/fdi.c b/dlls/cabinet/fdi.c index 2e39444..f9e6522 100644 --- a/dlls/cabinet/fdi.c +++ b/dlls/cabinet/fdi.c @@ -536,7 +536,7 @@ static BOOL FDI_read_entries( PMORE_ISCAB_INFO pmii) { int num_folders, num_files, header_resv, folder_resv = 0; - LONG base_offset, cabsize; + LONG cabsize; USHORT setid, cabidx, flags; cab_UBYTE buf[64], block_resv; char *prevname = NULL, *previnfo = NULL, *nextname = NULL, *nextinfo = NULL; @@ -573,34 +573,21 @@ static BOOL FDI_read_entries( * -gmt */ - /* get basic offset & size info */ - base_offset = FDI_getoffset(fdi, hf); - - if (fdi->seek(hf, 0, SEEK_END) == -1) { - if (pmii) set_error( fdi, FDIERROR_NOT_A_CABINET, 0 ); - return FALSE; - } - - cabsize = FDI_getoffset(fdi, hf); - - if ((cabsize == -1) || (base_offset == -1) || - ( fdi->seek(hf, base_offset, SEEK_SET) == -1 )) { - if (pmii) set_error( fdi, FDIERROR_NOT_A_CABINET, 0 ); - return FALSE; - } - /* read in the CFHEADER */ if (fdi->read(hf, buf, cfhead_SIZEOF) != cfhead_SIZEOF) { if (pmii) set_error( fdi, FDIERROR_NOT_A_CABINET, 0 ); return FALSE; } - + /* check basic MSCF signature */ if (EndGetI32(buf+cfhead_Signature) != 0x4643534d) { if (pmii) set_error( fdi, FDIERROR_NOT_A_CABINET, 0 ); return FALSE; } + /* get the cabinet size */ + cabsize = EndGetI32(buf+cfhead_CabinetSize); + /* get the number of folders */ num_folders = EndGetI16(buf+cfhead_NumFolders); diff --git a/dlls/cabinet/tests/fdi.c b/dlls/cabinet/tests/fdi.c index a2c305d..afae707 100644 --- a/dlls/cabinet/tests/fdi.c +++ b/dlls/cabinet/tests/fdi.c @@ -762,7 +762,6 @@ static LONG CDECL fdi_mem_seek(INT_PTR hf, LONG dist, int seektype) case SEEK_END: default: -todo_wine ok(0, "seek: not expected type %d\n", seektype); return -1; } @@ -879,18 +878,16 @@ static void test_FDICopy(void) memset(&info, 0, sizeof(info)); ret = FDIIsCabinet(hfdi, fd, &info); -todo_wine { ok(ret, "FDIIsCabinet error %d\n", erf.erfOper); ok(info.cbCabinet == 0x59, "expected 0x59, got %#x\n", info.cbCabinet); ok(info.cFiles == 1, "expected 1, got %d\n", info.cFiles); ok(info.cFolders == 1, "expected 1, got %d\n", info.cFolders); ok(info.setID == 0x1225, "expected 0x1225, got %#x\n", info.setID); ok(info.iCabinet == 0x2013, "expected 0x2013, got %#x\n", info.iCabinet); -} + fdi_mem_close(fd); ret = FDICopy(hfdi, block, memory, 0, fdi_mem_notify, NULL, 0); -todo_wine ok(ret, "FDICopy error %d\n", erf.erfOper); FDIDestroy(hfdi);

1 0

Dmitry Timoshkov : cabinet/tests: Add an FDICopy test which emulates extracting from a memory block.
by Alexandre Julliard Dec. 31, 2013

Dec. 31, 2013

Module: wine Branch: master Commit: 3cb5b93db996fe67dbbd18c111d3d826c72c6ecf URL: http://source.winehq.org/git/wine.git/?a=commit;h=3cb5b93db996fe67dbbd18c11… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Wed Dec 25 13:24:50 2013 +0900 cabinet/tests: Add an FDICopy test which emulates extracting from a memory block. --- dlls/cabinet/tests/fdi.c | 246 ++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 246 insertions(+), 0 deletions(-) diff --git a/dlls/cabinet/tests/fdi.c b/dlls/cabinet/tests/fdi.c index 919fdbf..a2c305d 100644 --- a/dlls/cabinet/tests/fdi.c +++ b/dlls/cabinet/tests/fdi.c @@ -2,6 +2,7 @@ * Unit tests for the File Decompression Interface * * Copyright (C) 2006 James Hawkins + * Copyright (C) 2013 Dmitry Timoshkov * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -30,6 +31,105 @@ static CHAR CURR_DIR[MAX_PATH]; +/* avoid including CRT headers */ +#ifndef _O_BINARY +# define _O_BINARY 0x8000 +#endif + +#ifndef _S_IREAD +# define _S_IREAD 0x0100 +# define _S_IWRITE 0x0080 +#endif + +#include <pshpack1.h> + +struct CFHEADER +{ + UCHAR signature[4]; /* file signature */ + ULONG reserved1; /* reserved */ + ULONG cbCabinet; /* size of this cabinet file in bytes */ + ULONG reserved2; /* reserved */ + ULONG coffFiles; /* offset of the first CFFILE entry */ + ULONG reserved3; /* reserved */ + UCHAR versionMinor; /* cabinet file format version, minor */ + UCHAR versionMajor; /* cabinet file format version, major */ + USHORT cFolders; /* number of CFFOLDER entries in this cabinet */ + USHORT cFiles; /* number of CFFILE entries in this cabinet */ + USHORT flags; /* cabinet file option indicators */ + USHORT setID; /* must be the same for all cabinets in a set */ + USHORT iCabinet; /* number of this cabinet file in a set */ +#if 0 + USHORT cbCFHeader; /* (optional) size of per-cabinet reserved area */ + UCHAR cbCFFolder; /* (optional) size of per-folder reserved area */ + UCHAR cbCFData; /* (optional) size of per-datablock reserved area */ + UCHAR abReserve; /* (optional) per-cabinet reserved area */ + UCHAR szCabinetPrev; /* (optional) name of previous cabinet file */ + UCHAR szDiskPrev; /* (optional) name of previous disk */ + UCHAR szCabinetNext; /* (optional) name of next cabinet file */ + UCHAR szDiskNext; /* (optional) name of next disk */ +#endif +}; + +struct CFFOLDER +{ + ULONG coffCabStart; /* offset of the first CFDATA block in this folder */ + USHORT cCFData; /* number of CFDATA blocks in this folder */ + USHORT typeCompress; /* compression type indicator */ +#if 0 + UCHAR abReserve[]; /* (optional) per-folder reserved area */ +#endif +}; + +struct CFFILE +{ + ULONG cbFile; /* uncompressed size of this file in bytes */ + ULONG uoffFolderStart; /* uncompressed offset of this file in the folder */ + USHORT iFolder; /* index into the CFFOLDER area */ + USHORT date; /* date stamp for this file */ + USHORT time; /* time stamp for this file */ + USHORT attribs; /* attribute flags for this file */ +#if 0 + UCHAR szName[]; /* name of this file */ +#endif +}; + +struct CFDATA +{ + ULONG csum; /* checksum of this CFDATA entry */ + USHORT cbData; /* number of compressed bytes in this block */ + USHORT cbUncomp; /* number of uncompressed bytes in this block */ +#if 0 + UCHAR abReserve[]; /* (optional) per-datablock reserved area */ + UCHAR ab[cbData]; /* compressed data bytes */ +#endif +}; + +static const struct +{ + struct CFHEADER header; + struct CFFOLDER folder; + struct CFFILE file; + UCHAR szName[sizeof("file.dat")]; + struct CFDATA data; + UCHAR ab[sizeof("Hello World!")-1]; +} cab_data = +{ + { {'M','S','C','F'}, 0, 0x59, 0, sizeof(struct CFHEADER) + sizeof(struct CFFOLDER), 0, 3,1, 1, 1, 0, 0x1225, 0x2013 }, + { sizeof(struct CFHEADER) + sizeof(struct CFFOLDER) + sizeof(struct CFFILE) + sizeof("file.dat"), 1, tcompTYPE_NONE }, + { sizeof("Hello World!")-1, 0, 0, 0x1225, 0x2013, 0xa114 }, + { 'f','i','l','e','.','d','a','t',0 }, + { 0, sizeof("Hello World!")-1, sizeof("Hello World!")-1 }, + { 'H','e','l','l','o',' ','W','o','r','l','d','!' } +}; + +#include <poppack.h> + +struct mem_data +{ + const char *base; + LONG size, pos; +}; + /* FDI callbacks */ static void * CDECL fdi_alloc(ULONG cb) @@ -592,6 +692,121 @@ static INT_PTR __cdecl CopyProgress(FDINOTIFICATIONTYPE fdint, PFDINOTIFICATION return 0; } +static INT_PTR CDECL fdi_mem_open(char *name, int oflag, int pmode) +{ + static const char expected[] = "memory\\block"; + struct mem_data *data; + + ok(!strcmp(name, expected), "expected %s, got %s\n", expected, name); + ok(oflag == _O_BINARY, "expected _O_BINARY, got %x\n", oflag); + ok(pmode == (_S_IREAD | _S_IWRITE), "expected _S_IREAD | _S_IWRITE, got %x\n", pmode); + + data = HeapAlloc(GetProcessHeap(), 0, sizeof(*data)); + if (!data) return -1; + + data->base = (const char *)&cab_data; + data->size = sizeof(cab_data); + data->pos = 0; + + trace("mem_open(%s,%x,%x) => %p\n", name, oflag, pmode, data); + return (INT_PTR)data; +} + +static UINT CDECL fdi_mem_read(INT_PTR hf, void *pv, UINT cb) +{ + struct mem_data *data = (struct mem_data *)hf; + UINT available, cb_read; + + available = data->size - data->pos; + cb_read = (available >= cb) ? cb : available; + + memcpy(pv, data->base + data->pos, cb_read); + data->pos += cb; + + /*trace("mem_read(%p,%p,%u) => %u\n", hf, pv, cb, cb_read);*/ + return cb_read; +} + +static UINT CDECL fdi_mem_write(INT_PTR hf, void *pv, UINT cb) +{ + static const char expected[12] = "Hello World!"; + + trace("mem_write(%#lx,%p,%u)\n", hf, pv, cb); + + ok(hf == 0x12345678, "expected 0x12345678, got %#lx\n", hf); + ok(cb == 12, "expected 12, got %u\n", cb); + ok(!memcmp(pv, expected, 12), "expected %s, got %s\n", expected, (const char *)pv); + + return cb; +} + +static int CDECL fdi_mem_close(INT_PTR hf) +{ + HeapFree(GetProcessHeap(), 0, (void *)hf); + return 0; +} + +static LONG CDECL fdi_mem_seek(INT_PTR hf, LONG dist, int seektype) +{ + struct mem_data *data = (struct mem_data *)hf; + + switch (seektype) + { + case SEEK_SET: + data->pos = dist; + break; + + case SEEK_CUR: + data->pos += dist; + break; + + case SEEK_END: + default: +todo_wine + ok(0, "seek: not expected type %d\n", seektype); + return -1; + } + + if (data->pos < 0) data->pos = 0; + if (data->pos > data->size) data->pos = data->size; + + /*mem_seek(%p,%d,%d) => %u\n", hf, dist, seektype, data->pos);*/ + return data->pos; +} + +static INT_PTR CDECL fdi_mem_notify(FDINOTIFICATIONTYPE fdint, FDINOTIFICATION *info) +{ + static const char expected[9] = "file.dat\0"; + + switch (fdint) + { + case fdintCLOSE_FILE_INFO: + trace("mem_notify: CLOSE_FILE_INFO %s, handle %#lx\n", info->psz1, info->hf); + + ok(!strcmp(info->psz1, expected), "expected %s, got %s\n", expected, info->psz1); + ok(info->date == 0x1225, "expected 0x1225, got %#x\n", info->date); + ok(info->time == 0x2013, "expected 0x2013, got %#x\n", info->time); + ok(info->attribs == 0xa114, "expected 0xa114, got %#x\n", info->attribs); + ok(info->iFolder == 0, "expected 0, got %#x\n", info->iFolder); + return 1; + + case fdintCOPY_FILE: + { + trace("mem_notify: COPY_FILE %s, %d bytes\n", info->psz1, info->cb); + + ok(info->cb == 12, "expected 12, got %u\n", info->cb); + ok(!strcmp(info->psz1, expected), "expected %s, got %s\n", expected, info->psz1); + return 0x12345678; /* call write() callback */ + } + + default: + trace("mem_notify(%d,%p)\n", fdint, info); + return 0; + } + + return 0; +} + static void test_FDICopy(void) { CCAB cabParams; @@ -601,6 +816,11 @@ static void test_FDICopy(void) BOOL ret; char name[] = "extract.cab"; char path[MAX_PATH + 1]; + char memory_block[] = "memory\\block"; + char memory[] = "memory\\"; + char block[] = "block"; + FDICABINETINFO info; + INT_PTR fd; set_cab_parameters(&cabParams); @@ -648,6 +868,32 @@ static void test_FDICopy(void) FDIDestroy(hfdi); DeleteFileA(name); + + /* test extracting from a memory block */ + hfdi = FDICreate(fdi_alloc, fdi_free, fdi_mem_open, fdi_mem_read, + fdi_mem_write, fdi_mem_close, fdi_mem_seek, cpuUNKNOWN, &erf); + ok(hfdi != NULL, "FDICreate error %d\n", erf.erfOper); + + fd = fdi_mem_open(memory_block, _O_BINARY, _S_IREAD | _S_IWRITE); + ok(fd != -1, "fdi_open failed\n"); + + memset(&info, 0, sizeof(info)); + ret = FDIIsCabinet(hfdi, fd, &info); +todo_wine { + ok(ret, "FDIIsCabinet error %d\n", erf.erfOper); + ok(info.cbCabinet == 0x59, "expected 0x59, got %#x\n", info.cbCabinet); + ok(info.cFiles == 1, "expected 1, got %d\n", info.cFiles); + ok(info.cFolders == 1, "expected 1, got %d\n", info.cFolders); + ok(info.setID == 0x1225, "expected 0x1225, got %#x\n", info.setID); + ok(info.iCabinet == 0x2013, "expected 0x2013, got %#x\n", info.iCabinet); +} + fdi_mem_close(fd); + + ret = FDICopy(hfdi, block, memory, 0, fdi_mem_notify, NULL, 0); +todo_wine + ok(ret, "FDICopy error %d\n", erf.erfOper); + + FDIDestroy(hfdi); }

1 0

Ken Thomases : winemac: More thoroughly discard events which have been obsoleted by subsequent Wine- or program-driven changes .
by Alexandre Julliard Dec. 31, 2013

Dec. 31, 2013

Module: wine Branch: master Commit: 4f9de6bcdf1722a258413ce99c26f2ee5b36dbf2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4f9de6bcdf1722a258413ce99… Author: Ken Thomases <ken(a)codeweavers.com> Date: Tue Dec 31 01:05:18 2013 -0600 winemac: More thoroughly discard events which have been obsoleted by subsequent Wine- or program-driven changes. Among other things, this fixes Syberia 2. That game shows, hides, and then shows its window. Hiding it caused a WINDOW_LOST_FOCUS event to be queued. By the time it was processed, the window was the foreground window again. In response to being told it had lost focus, the game minimized its window. Hiding the window should have prevented or discarded the WINDOW_LOST_FOCUS event since the change was driven from Wine and the Win32 foreground/active window state would already be correct. In addition, when the program re-showed its window and made it foreground, that should have discarded the event as being out of date. Now they do. --- dlls/winemac.drv/cocoa_app.m | 2 ++ dlls/winemac.drv/cocoa_event.m | 3 +++ dlls/winemac.drv/cocoa_window.m | 34 ++++++++++++++++++++++++++-------- 3 files changed, 31 insertions(+), 8 deletions(-) diff --git a/dlls/winemac.drv/cocoa_app.m b/dlls/winemac.drv/cocoa_app.m index 17644d0..fea30e3 100644 --- a/dlls/winemac.drv/cocoa_app.m +++ b/dlls/winemac.drv/cocoa_app.m @@ -2515,6 +2515,8 @@ void macdrv_set_mouse_capture_window(macdrv_window window) { WineWindow* w = (WineWindow*)window; + [w.queue discardEventsMatchingMask:event_mask_for_type(RELEASE_CAPTURE) forWindow:w]; + OnMainThread(^{ [[WineApplicationController sharedController] setMouseCaptureWindow:w]; }); diff --git a/dlls/winemac.drv/cocoa_event.m b/dlls/winemac.drv/cocoa_event.m index 00f6f8e..85cac5f 100644 --- a/dlls/winemac.drv/cocoa_event.m +++ b/dlls/winemac.drv/cocoa_event.m @@ -269,6 +269,7 @@ static const OSType WineHotKeySignature = 'Wine'; - (void) discardEventsMatchingMask:(macdrv_event_mask)mask forWindow:(NSWindow*)window { + NSAutoreleasePool* pool = [[NSAutoreleasePool alloc] init]; NSIndexSet* indexes; [eventsLock lock]; @@ -282,6 +283,8 @@ static const OSType WineHotKeySignature = 'Wine'; [events removeObjectsAtIndexes:indexes]; [eventsLock unlock]; + + [pool release]; } - (BOOL) query:(macdrv_query*)query timeout:(NSTimeInterval)timeout processEvents:(BOOL)processEvents diff --git a/dlls/winemac.drv/cocoa_window.m b/dlls/winemac.drv/cocoa_window.m index 07abbca..54cb8db 100644 --- a/dlls/winemac.drv/cocoa_window.m +++ b/dlls/winemac.drv/cocoa_window.m @@ -838,7 +838,7 @@ static inline NSUInteger adjusted_modifiers_for_option_behavior(NSUInteger modif if (state->minimized_valid) { - BOOL discardUnminimize = TRUE; + macdrv_event_mask discard = event_mask_for_type(WINDOW_DID_UNMINIMIZE); pendingMinimize = FALSE; if (state->minimized && ![self isMiniaturized]) @@ -848,10 +848,15 @@ static inline NSUInteger adjusted_modifiers_for_option_behavior(NSUInteger modif if ([self styleMask] & NSFullScreenWindowMask) { [self postDidUnminimizeEvent]; - discardUnminimize = FALSE; + discard &= ~event_mask_for_type(WINDOW_DID_UNMINIMIZE); } else + { [super miniaturize:nil]; + discard |= event_mask_for_type(WINDOW_BROUGHT_FORWARD) | + event_mask_for_type(WINDOW_GOT_FOCUS) | + event_mask_for_type(WINDOW_LOST_FOCUS); + } } else pendingMinimize = TRUE; @@ -860,14 +865,11 @@ static inline NSUInteger adjusted_modifiers_for_option_behavior(NSUInteger modif { ignore_windowDeminiaturize = TRUE; [self deminiaturize:nil]; + discard |= event_mask_for_type(WINDOW_LOST_FOCUS); } - if (discardUnminimize) - { - /* Whatever events regarding minimization might have been in the queue are now stale. */ - [queue discardEventsMatchingMask:event_mask_for_type(WINDOW_DID_UNMINIMIZE) - forWindow:self]; - } + if (discard) + [queue discardEventsMatchingMask:discard forWindow:self]; } if (state->maximized != maximized) @@ -1221,6 +1223,14 @@ static inline NSUInteger adjusted_modifiers_for_option_behavior(NSUInteger modif [controller updateFullscreenWindows]; [controller adjustWindowLevels]; [NSApp removeWindowsItem:self]; + + [queue discardEventsMatchingMask:event_mask_for_type(WINDOW_BROUGHT_FORWARD) | + event_mask_for_type(WINDOW_GOT_FOCUS) | + event_mask_for_type(WINDOW_LOST_FOCUS) | + event_mask_for_type(WINDOW_MAXIMIZE_REQUESTED) | + event_mask_for_type(WINDOW_MINIMIZE_REQUESTED) | + event_mask_for_type(WINDOW_RESTORE_REQUESTED) + forWindow:self]; } - (void) updateFullscreen @@ -1399,6 +1409,10 @@ static inline NSUInteger adjusted_modifiers_for_option_behavior(NSUInteger modif causing_becomeKeyWindow = self; [self makeKeyWindow]; causing_becomeKeyWindow = nil; + + [queue discardEventsMatchingMask:event_mask_for_type(WINDOW_GOT_FOCUS) | + event_mask_for_type(WINDOW_LOST_FOCUS) + forWindow:self]; } - (void) postKey:(uint16_t)keyCode @@ -2168,6 +2182,10 @@ void macdrv_order_cocoa_window(macdrv_window w, macdrv_window p, orAbove:next activate:activate]; }); + [window.queue discardEventsMatchingMask:event_mask_for_type(WINDOW_BROUGHT_FORWARD) + forWindow:window]; + [next.queue discardEventsMatchingMask:event_mask_for_type(WINDOW_BROUGHT_FORWARD) + forWindow:next]; } /***********************************************************************

1 0

Ken Thomases : winemac: For WINDOW_DID_UNMINIMIZE events, don' t attempt to restore windows which aren't minimized and visible.
by Alexandre Julliard Dec. 31, 2013

Dec. 31, 2013

Module: wine Branch: master Commit: 94cfa7799bac562a82ef9f403be93fce73519520 URL: http://source.winehq.org/git/wine.git/?a=commit;h=94cfa7799bac562a82ef9f403… Author: Ken Thomases <ken(a)codeweavers.com> Date: Tue Dec 31 01:05:15 2013 -0600 winemac: For WINDOW_DID_UNMINIMIZE events, don't attempt to restore windows which aren't minimized and visible. The Win32 window state might have changed while the event was in the queue, making it obsolete. Sending WM_SYSCOMMAND/SC_RESTORE might re-show a hidden window, for example. --- dlls/winemac.drv/window.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/winemac.drv/window.c b/dlls/winemac.drv/window.c index fd47ea2..5a1d5a7 100644 --- a/dlls/winemac.drv/window.c +++ b/dlls/winemac.drv/window.c @@ -2095,7 +2095,7 @@ void macdrv_window_did_unminimize(HWND hwnd) style = GetWindowLongW(hwnd, GWL_STYLE); data->minimized = FALSE; - if (style & (WS_MINIMIZE | WS_MAXIMIZE)) + if ((style & (WS_MINIMIZE | WS_VISIBLE)) == (WS_MINIMIZE | WS_VISIBLE)) { TRACE("restoring win %p/%p\n", hwnd, data->cocoa_window); release_win_data(data);

1 0

Ken Thomases : winemac: Defer adjusting window levels while the process is hidden.
by Alexandre Julliard Dec. 31, 2013

Dec. 31, 2013

Module: wine Branch: master Commit: 6447e8e75cd8eb7b50b385d1b98b2f9ae76e4b04 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6447e8e75cd8eb7b50b385d1b… Author: Ken Thomases <ken(a)codeweavers.com> Date: Tue Dec 31 01:05:05 2013 -0600 winemac: Defer adjusting window levels while the process is hidden. While the process is hidden, it can't get a list of its windows which would be visible were it not hidden. --- dlls/winemac.drv/cocoa_app.m | 14 ++++++++++++-- 1 files changed, 12 insertions(+), 2 deletions(-) diff --git a/dlls/winemac.drv/cocoa_app.m b/dlls/winemac.drv/cocoa_app.m index c31bceb..17644d0 100644 --- a/dlls/winemac.drv/cocoa_app.m +++ b/dlls/winemac.drv/cocoa_app.m @@ -488,8 +488,8 @@ int macdrv_err_on; - (void) adjustWindowLevels:(BOOL)active { - NSArray* windowNumbers = [NSWindow windowNumbersWithOptions:0]; - NSMutableArray* wineWindows = [[NSMutableArray alloc] initWithCapacity:[windowNumbers count]]; + NSArray* windowNumbers; + NSMutableArray* wineWindows; NSNumber* windowNumber; NSUInteger nextFloatingIndex = 0; __block NSInteger maxLevel = NSIntegerMin; @@ -497,6 +497,11 @@ int macdrv_err_on; __block WineWindow* prev = nil; WineWindow* window; + if ([NSApp isHidden]) return; + + windowNumbers = [NSWindow windowNumbersWithOptions:0]; + wineWindows = [[NSMutableArray alloc] initWithCapacity:[windowNumbers count]]; + // For the most part, we rely on the window server's ordering of the windows // to be authoritative. The one exception is if the "floating" property of // one of the windows has been changed, it may be in the wrong level and thus @@ -2127,6 +2132,11 @@ int macdrv_err_on; [self releaseMouseCapture]; } + - (void) applicationDidUnhide:(NSNotification*)aNotification + { + [self adjustWindowLevels]; + } + - (BOOL) applicationShouldHandleReopen:(NSApplication*)theApplication hasVisibleWindows:(BOOL)flag { // Note that "flag" is often wrong. WineWindows are NSPanels and NSPanels

1 0

Nikolay Sivov : ole32: Call moniker methods with proper wrappers.
by Alexandre Julliard Dec. 31, 2013

Dec. 31, 2013

Module: wine Branch: master Commit: f041c2c5549e6a24c73004180d29fe327b058441 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f041c2c5549e6a24c73004180… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Dec 31 13:12:21 2013 +0400 ole32: Call moniker methods with proper wrappers. --- dlls/ole32/classmoniker.c | 35 +++++++++++++---------------------- 1 files changed, 13 insertions(+), 22 deletions(-) diff --git a/dlls/ole32/classmoniker.c b/dlls/ole32/classmoniker.c index 0c7e751..89c7297 100644 --- a/dlls/ole32/classmoniker.c +++ b/dlls/ole32/classmoniker.c @@ -119,20 +119,6 @@ static ULONG WINAPI ClassMoniker_AddRef(IMoniker* iface) } /****************************************************************************** - * ClassMoniker_Destroy (local function) - *******************************************************************************/ -static HRESULT ClassMoniker_Destroy(ClassMoniker* This) -{ - TRACE("(%p)\n",This); - - if (This->pMarshal) IUnknown_Release(This->pMarshal); - - HeapFree(GetProcessHeap(),0,This); - - return S_OK; -} - -/****************************************************************************** * ClassMoniker_Release ******************************************************************************/ static ULONG WINAPI ClassMoniker_Release(IMoniker* iface) @@ -145,7 +131,11 @@ static ULONG WINAPI ClassMoniker_Release(IMoniker* iface) ref = InterlockedDecrement(&This->ref); /* destroy the object if there are no more references to it */ - if (ref == 0) ClassMoniker_Destroy(This); + if (ref == 0) + { + if (This->pMarshal) IUnknown_Release(This->pMarshal); + HeapFree(GetProcessHeap(),0,This); + } return ref; } @@ -277,7 +267,7 @@ static HRESULT WINAPI ClassMoniker_BindToStorage(IMoniker* iface, VOID** ppvResult) { TRACE("(%p,%p,%p,%p)\n",pbc, pmkToLeft, riid, ppvResult); - return ClassMoniker_BindToObject(iface, pbc, pmkToLeft, riid, ppvResult); + return IMoniker_BindToObject(iface, pbc, pmkToLeft, riid, ppvResult); } /****************************************************************************** @@ -294,7 +284,7 @@ static HRESULT WINAPI ClassMoniker_Reduce(IMoniker* iface, if (!ppmkReduced) return E_POINTER; - ClassMoniker_AddRef(iface); + IMoniker_AddRef(iface); *ppmkReduced = iface; @@ -600,7 +590,7 @@ static HRESULT WINAPI ClassMonikerROTData_QueryInterface(IROTData *iface,REFIID TRACE("(%p,%p,%p)\n",iface,riid,ppvObject); - return ClassMoniker_QueryInterface(&This->IMoniker_iface, riid, ppvObject); + return IMoniker_QueryInterface(&This->IMoniker_iface, riid, ppvObject); } /*********************************************************************** @@ -612,7 +602,7 @@ static ULONG WINAPI ClassMonikerROTData_AddRef(IROTData *iface) TRACE("(%p)\n",iface); - return ClassMoniker_AddRef(&This->IMoniker_iface); + return IMoniker_AddRef(&This->IMoniker_iface); } /*********************************************************************** @@ -624,7 +614,7 @@ static ULONG WINAPI ClassMonikerROTData_Release(IROTData* iface) TRACE("(%p)\n",iface); - return ClassMoniker_Release(&This->IMoniker_iface); + return IMoniker_Release(&This->IMoniker_iface); } /****************************************************************************** @@ -874,9 +864,10 @@ static const IClassFactoryVtbl ClassMonikerCFVtbl = ClassMonikerCF_CreateInstance, ClassMonikerCF_LockServer }; -static const IClassFactoryVtbl *ClassMonikerCF = &ClassMonikerCFVtbl; + +static IClassFactory ClassMonikerCF = { &ClassMonikerCFVtbl }; HRESULT ClassMonikerCF_Create(REFIID riid, LPVOID *ppv) { - return IClassFactory_QueryInterface((IClassFactory *)&ClassMonikerCF, riid, ppv); + return IClassFactory_QueryInterface(&ClassMonikerCF, riid, ppv); }

1 0