October 2008 - Wine-commits

Alexandre Julliard : mpr: Use a mountmgr ioctl to retrieve the UNC name for a drive.
by Alexandre Julliard 24 Oct '08

24 Oct '08

Module: wine Branch: master Commit: cb566755f73905196817770b610f74dc43283517 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cb566755f73905196817770b6… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Oct 23 19:59:17 2008 +0200 mpr: Use a mountmgr ioctl to retrieve the UNC name for a drive. --- dlls/mpr/wnet.c | 74 +++++++++++++++++++++++++++++++++++++------------------ 1 files changed, 50 insertions(+), 24 deletions(-) diff --git a/dlls/mpr/wnet.c b/dlls/mpr/wnet.c index 959efb3..3b05ede 100644 --- a/dlls/mpr/wnet.c +++ b/dlls/mpr/wnet.c @@ -24,10 +24,13 @@ #include "windef.h" #include "winbase.h" #include "winnls.h" +#include "winioctl.h" #include "winnetwk.h" #include "npapi.h" #include "winreg.h" #include "winuser.h" +#define WINE_MOUNTMGR_EXTENSIONS +#include "ddk/mountmgr.h" #include "wine/debug.h" #include "wine/unicode.h" #include "mprres.h" @@ -1750,6 +1753,52 @@ DWORD WINAPI WNetGetConnectionA( LPCSTR lpLocalName, return ret; } +/* find the network connection for a given drive; helper for WNetGetConnection */ +static DWORD get_drive_connection( WCHAR letter, LPWSTR remote, LPDWORD size ) +{ + char buffer[1024]; + struct mountmgr_unix_drive *data = (struct mountmgr_unix_drive *)buffer; + HANDLE mgr; + DWORD ret = WN_NOT_CONNECTED; + + if ((mgr = CreateFileW( MOUNTMGR_DOS_DEVICE_NAME, GENERIC_READ|GENERIC_WRITE, + FILE_SHARE_READ|FILE_SHARE_WRITE, NULL, OPEN_EXISTING, + 0, 0 )) == INVALID_HANDLE_VALUE) + { + ERR( "failed to open mount manager err %u\n", GetLastError() ); + return ret; + } + memset( data, 0, sizeof(*data) ); + data->letter = letter; + if (DeviceIoControl( mgr, IOCTL_MOUNTMGR_QUERY_UNIX_DRIVE, data, sizeof(*data), + data, sizeof(buffer), NULL, NULL )) + { + char *p, *mount_point = buffer + data->mount_point_offset; + DWORD len; + + if (data->mount_point_offset && !strncmp( mount_point, "unc/", 4 )) + { + mount_point += 2; + mount_point[0] = '\\'; + for (p = mount_point; *p; p++) if (*p == '/') *p = '\\'; + + len = MultiByteToWideChar( CP_UNIXCP, 0, mount_point, -1, NULL, 0 ); + if (len > *size) + { + *size = len; + ret = WN_MORE_DATA; + } + else + { + *size = MultiByteToWideChar( CP_UNIXCP, 0, mount_point, -1, remote, *size); + ret = WN_SUCCESS; + } + } + } + CloseHandle( mgr ); + return ret; +} + /************************************************************************** * WNetGetConnectionW [MPR.@] * @@ -1778,31 +1827,8 @@ DWORD WINAPI WNetGetConnectionW( LPCWSTR lpLocalName, switch(GetDriveTypeW(lpLocalName)) { case DRIVE_REMOTE: - { - static const WCHAR unc[] = { 'u','n','c','\\' }; - WCHAR rremote[MAX_PATH], *remote = rremote; - if (!QueryDosDeviceW( lpLocalName, remote, MAX_PATH )) remote[0] = 0; - else if (!strncmpW(remote, unc, 4)) - { - remote += 2; - remote[0] = '\\'; - } - else if (remote[0] != '\\' || remote[1] != '\\') - FIXME("Don't know how to convert %s to an unc\n", debugstr_w(remote)); - - if (strlenW(remote) + 1 > *lpBufferSize) - { - *lpBufferSize = strlenW(remote) + 1; - ret = WN_MORE_DATA; - } - else - { - strcpyW( lpRemoteName, remote ); - *lpBufferSize = strlenW(lpRemoteName) + 1; - ret = WN_SUCCESS; - } + ret = get_drive_connection( lpLocalName[0], lpRemoteName, lpBufferSize ); break; - } case DRIVE_REMOVABLE: case DRIVE_FIXED: case DRIVE_CDROM:

1 0

Alexandre Julliard : mountmgr: Store the contents of the device symlink in the drive object.
by Alexandre Julliard 24 Oct '08

24 Oct '08

Module: wine Branch: master Commit: 5f2d2c7df07249b8128d9626bb8b67f72c47ac92 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5f2d2c7df07249b8128d9626b… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Oct 23 15:43:46 2008 +0200 mountmgr: Store the contents of the device symlink in the drive object. --- dlls/mountmgr.sys/device.c | 30 +++++++++++++++++++++++++----- 1 files changed, 25 insertions(+), 5 deletions(-) diff --git a/dlls/mountmgr.sys/device.c b/dlls/mountmgr.sys/device.c index dd98c16..3ed689f 100644 --- a/dlls/mountmgr.sys/device.c +++ b/dlls/mountmgr.sys/device.c @@ -67,6 +67,7 @@ struct dos_drive STORAGE_DEVICE_NUMBER devnum; /* device number info */ struct mount_point *dosdev; /* DosDevices mount point */ struct mount_point *volume; /* Volume{xxx} mount point */ + char *unix_device; /* unix device path */ char *unix_mount; /* unix mount point path */ }; @@ -88,6 +89,15 @@ static char *get_dosdevices_path( char **drive ) return path; } +static char *strdupA( const char *str ) +{ + char *ret; + + if (!str) return NULL; + if ((ret = RtlAllocateHeap( GetProcessHeap(), 0, strlen(str) + 1 ))) strcpy( ret, str ); + return ret; +} + /* read a Unix symlink; returned buffer must be freed by caller */ static char *read_symlink( const char *path ) { @@ -186,7 +196,8 @@ static NTSTATUS create_disk_device( const char *udi, DWORD type, struct dos_driv drive->type = type; drive->dosdev = NULL; drive->volume = NULL; - drive->unix_mount = NULL; + drive->unix_device = NULL; + drive->unix_mount = NULL; drive->symlink.Buffer = NULL; if (udi) { @@ -258,6 +269,7 @@ static void delete_disk_device( struct dos_drive *drive ) IoDeleteSymbolicLink( &drive->symlink ); RtlFreeUnicodeString( &drive->symlink ); } + RtlFreeHeap( GetProcessHeap(), 0, drive->unix_device ); RtlFreeHeap( GetProcessHeap(), 0, drive->unix_mount ); RtlFreeHeap( GetProcessHeap(), 0, drive->udi ); RtlFreeUnicodeString( &drive->name ); @@ -379,8 +391,7 @@ static BOOL set_unix_mount_point( struct dos_drive *drive, const char *mount_poi modified = TRUE; } RtlFreeHeap( GetProcessHeap(), 0, drive->unix_mount ); - if ((drive->unix_mount = RtlAllocateHeap( GetProcessHeap(), 0, strlen(mount_point) + 1 ))) - strcpy( drive->unix_mount, mount_point ); + drive->unix_mount = strdupA( mount_point ); if (drive->dosdev) set_mount_point_id( drive->dosdev, mount_point, strlen(mount_point) + 1 ); if (drive->volume) set_mount_point_id( drive->volume, mount_point, strlen(mount_point) + 1 ); } @@ -400,7 +411,7 @@ static BOOL set_unix_mount_point( struct dos_drive *drive, const char *mount_poi /* create devices for mapped drives */ static void create_drive_devices(void) { - char *path, *p, *link; + char *path, *p, *link, *device; struct dos_drive *drive; unsigned int i; HKEY drives_key; @@ -415,6 +426,8 @@ static void create_drive_devices(void) p[0] = 'a' + i; p[2] = 0; if (!(link = read_symlink( path ))) continue; + p[2] = ':'; + device = read_symlink( path ); drive_type = i < 2 ? DRIVE_REMOVABLE : DRIVE_FIXED; if (drives_key) @@ -438,9 +451,14 @@ static void create_drive_devices(void) if (!create_disk_device( NULL, drive_type, &drive )) { drive->unix_mount = link; + drive->unix_device = device; set_drive_letter( drive, i ); } - else RtlFreeHeap( GetProcessHeap(), 0, link ); + else + { + RtlFreeHeap( GetProcessHeap(), 0, link ); + RtlFreeHeap( GetProcessHeap(), 0, device ); + } } RegCloseKey( drives_key ); RtlFreeHeap( GetProcessHeap(), 0, path ); @@ -467,6 +485,8 @@ BOOL add_dos_device( const char *udi, const char *device, const char *mount_poin if (create_disk_device( udi, type, &drive )) return FALSE; found: + RtlFreeHeap( GetProcessHeap(), 0, drive->unix_device ); + drive->unix_device = strdupA( device ); set_drive_letter( drive, letter ); set_unix_mount_point( drive, mount_point );

1 0

Alexandre Julliard : mountmgr: Allow to specify the drive letter explicitly when creating/ removing a drive.
by Alexandre Julliard 24 Oct '08

24 Oct '08

Module: wine Branch: master Commit: b2ebab9f6fc20ec0ddee172d4d4925887aca351b URL: http://source.winehq.org/git/wine.git/?a=commit;h=b2ebab9f6fc20ec0ddee172d4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Oct 23 15:44:18 2008 +0200 mountmgr: Allow to specify the drive letter explicitly when creating/removing a drive. --- dlls/mountmgr.sys/device.c | 44 ++++++++++++++++++++++++++++++----------- dlls/mountmgr.sys/diskarb.c | 4 +- dlls/mountmgr.sys/hal.c | 4 +- dlls/mountmgr.sys/mountmgr.h | 5 ++- 4 files changed, 39 insertions(+), 18 deletions(-) diff --git a/dlls/mountmgr.sys/device.c b/dlls/mountmgr.sys/device.c index 3ed689f..9036654 100644 --- a/dlls/mountmgr.sys/device.c +++ b/dlls/mountmgr.sys/device.c @@ -464,16 +464,30 @@ static void create_drive_devices(void) RtlFreeHeap( GetProcessHeap(), 0, path ); } -BOOL add_dos_device( const char *udi, const char *device, const char *mount_point, DWORD type ) +/* create a new dos drive */ +NTSTATUS add_dos_device( int letter, const char *udi, const char *device, + const char *mount_point, DWORD type ) { struct dos_drive *drive, *next; - int letter = add_drive( device, type ); - if (letter == -1) return FALSE; + if (letter == -1) /* auto-assign a letter */ + { + letter = add_drive( device, type ); + if (letter == -1) return STATUS_OBJECT_NAME_COLLISION; + } + else /* simply reset the device symlink */ + { + char *path, *p; + + if (!(path = get_dosdevices_path( &p ))) return STATUS_NO_MEMORY; + *p = 'a' + letter; + unlink( path ); + if (device) symlink( device, path ); + } LIST_FOR_EACH_ENTRY_SAFE( drive, next, &drives_list, struct dos_drive, entry ) { - if (drive->udi && !strcmp( udi, drive->udi )) + if (udi && drive->udi && !strcmp( udi, drive->udi )) { if (type == drive->type) goto found; delete_disk_device( drive ); @@ -482,7 +496,7 @@ BOOL add_dos_device( const char *udi, const char *device, const char *mount_poin if (drive->drive == letter) delete_disk_device( drive ); } - if (create_disk_device( udi, type, &drive )) return FALSE; + if (create_disk_device( udi, type, &drive )) return STATUS_NO_MEMORY; found: RtlFreeHeap( GetProcessHeap(), 0, drive->unix_device ); @@ -513,19 +527,25 @@ found: RegCloseKey( hkey ); } - send_notify( drive->drive, DBT_DEVICEARRIVAL ); + if (udi) send_notify( drive->drive, DBT_DEVICEARRIVAL ); } - return TRUE; + return STATUS_SUCCESS; } -BOOL remove_dos_device( const char *udi ) +/* remove an existing dos drive, by letter or udi */ +NTSTATUS remove_dos_device( int letter, const char *udi ) { HKEY hkey; struct dos_drive *drive; LIST_FOR_EACH_ENTRY( drive, &drives_list, struct dos_drive, entry ) { - if (!drive->udi || strcmp( udi, drive->udi )) continue; + if (letter != -1 && drive->drive != letter) continue; + if (udi) + { + if (!drive->udi) continue; + if (strcmp( udi, drive->udi )) continue; + } if (drive->drive != -1) { @@ -540,12 +560,12 @@ BOOL remove_dos_device( const char *udi ) RegCloseKey( hkey ); } - if (modified) send_notify( drive->drive, DBT_DEVICEREMOVECOMPLETE ); + if (modified && udi) send_notify( drive->drive, DBT_DEVICEREMOVECOMPLETE ); } delete_disk_device( drive ); - return TRUE; + return STATUS_SUCCESS; } - return FALSE; + return STATUS_NO_SUCH_DEVICE; } /* handler for ioctls on the harddisk device */ diff --git a/dlls/mountmgr.sys/diskarb.c b/dlls/mountmgr.sys/diskarb.c index 90ced5e..a469ee6 100644 --- a/dlls/mountmgr.sys/diskarb.c +++ b/dlls/mountmgr.sys/diskarb.c @@ -69,7 +69,7 @@ static void appeared_callback( DADiskRef disk, void *context ) TRACE( "got mount notification for '%s' on '%s'\n", device, mount_point ); - add_dos_device( device, device, mount_point, type ); + add_dos_device( -1, device, device, mount_point, type ); done: CFRelease( dict ); } @@ -98,7 +98,7 @@ static void disappeared_callback( DADiskRef disk, void *context ) TRACE( "got unmount notification for '%s'\n", device ); - remove_dos_device( device ); + remove_dos_device( -1, device ); done: CFRelease( dict ); } diff --git a/dlls/mountmgr.sys/hal.c b/dlls/mountmgr.sys/hal.c index 0f21e99..e1d3da1 100644 --- a/dlls/mountmgr.sys/hal.c +++ b/dlls/mountmgr.sys/hal.c @@ -135,7 +135,7 @@ static void new_device( LibHalContext *ctx, const char *udi ) if (type && !strcmp( type, "cdrom" )) drive_type = DRIVE_CDROM; else drive_type = DRIVE_REMOVABLE; /* FIXME: default to removable */ - add_dos_device( udi, device, mount_point, drive_type ); + add_dos_device( -1, udi, device, mount_point, drive_type ); /* add property watch for mount point */ p_libhal_device_add_property_watch( ctx, udi, &error ); @@ -155,7 +155,7 @@ static void removed_device( LibHalContext *ctx, const char *udi ) TRACE( "removed %s\n", wine_dbgstr_a(udi) ); - if (remove_dos_device( udi )) + if (!remove_dos_device( -1, udi )) { p_dbus_error_init( &error ); p_libhal_device_remove_property_watch( ctx, udi, &error ); diff --git a/dlls/mountmgr.sys/mountmgr.h b/dlls/mountmgr.sys/mountmgr.h index ac163db..57c7b38 100644 --- a/dlls/mountmgr.sys/mountmgr.h +++ b/dlls/mountmgr.sys/mountmgr.h @@ -39,8 +39,9 @@ extern void initialize_diskarbitration(void); /* device functions */ -extern BOOL add_dos_device( const char *udi, const char *device, const char *mount_point, DWORD type ); -extern BOOL remove_dos_device( const char *udi ); +extern NTSTATUS add_dos_device( int letter, const char *udi, const char *device, + const char *mount_point, DWORD type ); +extern NTSTATUS remove_dos_device( int letter, const char *udi ); extern NTSTATUS WINAPI harddisk_driver_entry( DRIVER_OBJECT *driver, UNICODE_STRING *path ); /* mount point functions */

1 0

Dylan Smith : richedit: Fixed the call to the EditWordBreakProc.
by Alexandre Julliard 23 Oct '08

23 Oct '08

Module: wine Branch: master Commit: 503972980e150f9eda348962f437484d7c9daa43 URL: http://source.winehq.org/git/wine.git/?a=commit;h=503972980e150f9eda348962f… Author: Dylan Smith <dylan.ah.smith(a)gmail.com> Date: Thu Oct 23 01:07:31 2008 -0400 richedit: Fixed the call to the EditWordBreakProc. --- dlls/riched20/string.c | 24 +++++++++++++++++++----- dlls/riched20/tests/editor.c | 4 ++-- 2 files changed, 21 insertions(+), 7 deletions(-) diff --git a/dlls/riched20/string.c b/dlls/riched20/string.c index 97b5a67..7da521c 100644 --- a/dlls/riched20/string.c +++ b/dlls/riched20/string.c @@ -304,6 +304,8 @@ ME_WordBreakProc(LPWSTR s, INT start, INT len, INT code) /* FIXME: Native also knows about punctuation */ TRACE("s==%s, start==%d, len==%d, code==%d\n", debugstr_wn(s, len), start, len, code); + /* convert number of bytes to number of characters. */ + len /= sizeof(WCHAR); switch (code) { case WB_ISDELIMITER: @@ -330,11 +332,23 @@ ME_WordBreakProc(LPWSTR s, INT start, INT len, INT code) int ME_CallWordBreakProc(ME_TextEditor *editor, ME_String *str, INT start, INT code) { - /* FIXME: ANSIfy the string when bEmulateVersion10 is TRUE */ - if (!editor->pfnWordBreak) - return ME_WordBreakProc(str->szData, start, str->nLen, code); - else - return editor->pfnWordBreak(str->szData, start, str->nLen, code); + if (!editor->pfnWordBreak) { + return ME_WordBreakProc(str->szData, start, str->nLen*sizeof(WCHAR), code); + } else if (!editor->bEmulateVersion10) { + /* MSDN lied about the third parameter for EditWordBreakProc being the number + * of characters, it is actually the number of bytes of the string. */ + return editor->pfnWordBreak(str->szData, start, str->nLen*sizeof(WCHAR), code); + } else { + int result; + int buffer_size = WideCharToMultiByte(CP_ACP, 0, str->szData, str->nLen, + NULL, 0, NULL, NULL); + char *buffer = (char*)heap_alloc(buffer_size); + WideCharToMultiByte(CP_ACP, 0, str->szData, str->nLen, + buffer, buffer_size, NULL, NULL); + result = editor->pfnWordBreak(str->szData, start, str->nLen, code); + heap_free(buffer); + return result; + } } LPWSTR ME_ToUnicode(BOOL unicode, LPVOID psz) diff --git a/dlls/riched20/tests/editor.c b/dlls/riched20/tests/editor.c index 9c0a21d..8946ed3 100644 --- a/dlls/riched20/tests/editor.c +++ b/dlls/riched20/tests/editor.c @@ -5652,7 +5652,7 @@ static void test_word_movement(void) /* one twoX|three */ SendMessage(hwnd, EM_GETSEL, (WPARAM)&sel_start, (LPARAM)&sel_end); ok(sel_start == sel_end, "Selection should be empty\n"); - todo_wine ok(sel_start == 8, "Cursor is at %d instead of %d\n", sel_start, 8); + ok(sel_start == 8, "Cursor is at %d instead of %d\n", sel_start, 8); DestroyWindow(hwnd); @@ -5671,7 +5671,7 @@ static void test_word_movement(void) /* one twoX|three */ SendMessageW(hwnd, EM_GETSEL, (WPARAM)&sel_start, (LPARAM)&sel_end); ok(sel_start == sel_end, "Selection should be empty\n"); - todo_wine ok(sel_start == 8, "Cursor is at %d instead of %d\n", sel_start, 8); + ok(sel_start == 8, "Cursor is at %d instead of %d\n", sel_start, 8); DestroyWindow(hwnd); }

1 0

Dylan Smith : richedit: Added test for correct behaviour for calling EditWordBreakProc.
by Alexandre Julliard 23 Oct '08

23 Oct '08

Module: wine Branch: master Commit: 00fd6b62d3ca794b8e2dac7357a4fa95febb1e08 URL: http://source.winehq.org/git/wine.git/?a=commit;h=00fd6b62d3ca794b8e2dac735… Author: Dylan Smith <dylan.ah.smith(a)gmail.com> Date: Thu Oct 23 01:07:11 2008 -0400 richedit: Added test for correct behaviour for calling EditWordBreakProc. EditWordBreakProc documented the third parameter as being the number of unicode characters in the string for richedit 2.0 and up. It turns out that it should actually be the number of bytes in the string. --- dlls/riched20/tests/editor.c | 68 ++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 68 insertions(+), 0 deletions(-) diff --git a/dlls/riched20/tests/editor.c b/dlls/riched20/tests/editor.c index 72e57c3..9c0a21d 100644 --- a/dlls/riched20/tests/editor.c +++ b/dlls/riched20/tests/editor.c @@ -5548,6 +5548,43 @@ static void test_undo_coalescing(void) DestroyWindow(hwnd); } +LONG CALLBACK customWordBreakProc(WCHAR *text, int pos, int bytes, int code) +{ + int length; + + /* MSDN lied, length is actually the number of bytes. */ + length = bytes / sizeof(WCHAR); + switch(code) + { + case WB_ISDELIMITER: + return text[pos] == 'X'; + case WB_LEFT: + case WB_MOVEWORDLEFT: + if (customWordBreakProc(text, pos, bytes, WB_ISDELIMITER)) + return pos-1; + return min(customWordBreakProc(text, pos, bytes, WB_LEFTBREAK)-1, 0); + case WB_LEFTBREAK: + pos--; + while (pos > 0 && !customWordBreakProc(text, pos, bytes, WB_ISDELIMITER)) + pos--; + return pos; + case WB_RIGHT: + case WB_MOVEWORDRIGHT: + if (customWordBreakProc(text, pos, bytes, WB_ISDELIMITER)) + return pos+1; + return min(customWordBreakProc(text, pos, bytes, WB_RIGHTBREAK)+1, length); + case WB_RIGHTBREAK: + pos++; + while (pos < length && !customWordBreakProc(text, pos, bytes, WB_ISDELIMITER)) + pos++; + return pos; + default: + ok(FALSE, "Unexpected code %d\n", code); + break; + } + return 0; +} + #define SEND_CTRL_LEFT(hwnd) pressKeyWithModifier(hwnd, VK_CONTROL, VK_LEFT) #define SEND_CTRL_RIGHT(hwnd) pressKeyWithModifier(hwnd, VK_CONTROL, VK_RIGHT) @@ -5556,6 +5593,7 @@ static void test_word_movement(void) HWND hwnd; int result; int sel_start, sel_end; + const WCHAR textW[] = {'o','n','e',' ','t','w','o','X','t','h','r','e','e',0}; /* multi-line control inserts CR normally */ hwnd = new_richedit(NULL); @@ -5605,6 +5643,36 @@ static void test_word_movement(void) ok(sel_start == sel_end, "Selection should be empty\n"); ok(sel_start == 9, "Cursor is at %d instead of %d\n", sel_start, 9); + /* Test with a custom word break procedure that uses X as the delimiter. */ + result = SendMessageA(hwnd, WM_SETTEXT, 0, (LPARAM)"one twoXthree"); + ok (result == TRUE, "Failed to clear the text.\n"); + SendMessage(hwnd, EM_SETWORDBREAKPROC, 0, (LPARAM)customWordBreakProc); + /* |one twoXthree */ + SEND_CTRL_RIGHT(hwnd); + /* one twoX|three */ + SendMessage(hwnd, EM_GETSEL, (WPARAM)&sel_start, (LPARAM)&sel_end); + ok(sel_start == sel_end, "Selection should be empty\n"); + todo_wine ok(sel_start == 8, "Cursor is at %d instead of %d\n", sel_start, 8); + + DestroyWindow(hwnd); + + /* Make sure the behaviour is the same with a unicode richedit window, + * and using unicode functions. */ + hwnd = CreateWindowW(RICHEDIT_CLASS20W, NULL, + ES_MULTILINE|WS_POPUP|WS_HSCROLL|WS_VSCROLL|WS_VISIBLE, + 0, 0, 200, 60, NULL, NULL, hmoduleRichEdit, NULL); + + /* Test with a custom word break procedure that uses X as the delimiter. */ + result = SendMessageW(hwnd, WM_SETTEXT, 0, (LPARAM)textW); + ok (result == TRUE, "Failed to clear the text.\n"); + SendMessageW(hwnd, EM_SETWORDBREAKPROC, 0, (LPARAM)customWordBreakProc); + /* |one twoXthree */ + SEND_CTRL_RIGHT(hwnd); + /* one twoX|three */ + SendMessageW(hwnd, EM_GETSEL, (WPARAM)&sel_start, (LPARAM)&sel_end); + ok(sel_start == sel_end, "Selection should be empty\n"); + todo_wine ok(sel_start == 8, "Cursor is at %d instead of %d\n", sel_start, 8); + DestroyWindow(hwnd); }

1 0

Aric Stewart : ntdll/tests: handle info behavior for vista and 2008 properly.
by Alexandre Julliard 23 Oct '08

23 Oct '08

Module: wine Branch: master Commit: 589d7eaa93710e9bb42a419e1917c509e1bd7bf0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=589d7eaa93710e9bb42a419e1… Author: Aric Stewart <aric(a)codeweavers.com> Date: Thu Oct 23 07:49:04 2008 -0500 ntdll/tests: handle info behavior for vista and 2008 properly. --- dlls/ntdll/tests/info.c | 15 ++++++++++++--- 1 files changed, 12 insertions(+), 3 deletions(-) diff --git a/dlls/ntdll/tests/info.c b/dlls/ntdll/tests/info.c index d93e0bc..f61196e 100644 --- a/dlls/ntdll/tests/info.c +++ b/dlls/ntdll/tests/info.c @@ -376,9 +376,18 @@ static void test_query_procperf(void) ok( status == STATUS_SUCCESS || status == STATUS_INFO_LENGTH_MISMATCH /* vista */, "Expected STATUS_SUCCESS or STATUS_INFO_LENGTH_MISMATCH, got %08x\n", status); ok( NeededLength == ReturnLength, "Inconsistent length (%d) <-> (%d)\n", NeededLength, ReturnLength); - ok (sppi->KernelTime.QuadPart != 0xdeaddead, "KernelTime unchanged\n"); - ok (sppi->UserTime.QuadPart != 0xdeaddead, "UserTime unchanged\n"); - ok (sppi->IdleTime.QuadPart != 0xdeaddead, "IdleTime unchanged\n"); + if (status == STATUS_SUCCESS) + { + ok (sppi->KernelTime.QuadPart != 0xdeaddead, "KernelTime unchanged\n"); + ok (sppi->UserTime.QuadPart != 0xdeaddead, "UserTime unchanged\n"); + ok (sppi->IdleTime.QuadPart != 0xdeaddead, "IdleTime unchanged\n"); + } + else /* vista and 2008 */ + { + ok (sppi->KernelTime.QuadPart == 0xdeaddead, "KernelTime changed\n"); + ok (sppi->UserTime.QuadPart == 0xdeaddead, "UserTime changed\n"); + ok (sppi->IdleTime.QuadPart == 0xdeaddead, "IdleTime changed\n"); + } HeapFree( GetProcessHeap(), 0, sppi); }

1 0

Sylvain Petreolle : ole32: Stub implementation for CoGetCallerTID.
by Alexandre Julliard 23 Oct '08

23 Oct '08

Module: wine Branch: master Commit: e7d951320acb7cb395b4bc2b71acf5b3b2a71c8a URL: http://source.winehq.org/git/wine.git/?a=commit;h=e7d951320acb7cb395b4bc2b7… Author: Sylvain Petreolle <spetreolle(a)yahoo.fr> Date: Thu Oct 23 09:48:25 2008 +0000 ole32: Stub implementation for CoGetCallerTID. --- dlls/ole32/ole2stubs.c | 9 +++++++++ dlls/ole32/ole32.spec | 2 +- 2 files changed, 10 insertions(+), 1 deletions(-) diff --git a/dlls/ole32/ole2stubs.c b/dlls/ole32/ole2stubs.c index e339651..7c45704 100644 --- a/dlls/ole32/ole2stubs.c +++ b/dlls/ole32/ole2stubs.c @@ -119,3 +119,12 @@ HRESULT WINAPI OleRegEnumFormatEtc ( return E_NOTIMPL; } + +/*********************************************************************** + * CoGetCallerTID [OLE32.@] + */ +HRESULT WINAPI CoGetCallerTID(LPDWORD lpdwTID) +{ + FIXME("stub!\n"); + return E_NOTIMPL; +} diff --git a/dlls/ole32/ole32.spec b/dlls/ole32/ole32.spec index 7561156..e5e90b3 100644 --- a/dlls/ole32/ole32.spec +++ b/dlls/ole32/ole32.spec @@ -22,7 +22,7 @@ @ stdcall CoFreeUnusedLibraries() @ stdcall CoFreeUnusedLibrariesEx(long long) @ stdcall CoGetCallContext(ptr ptr) -@ stub CoGetCallerTID +@ stdcall CoGetCallerTID(ptr) @ stdcall CoGetClassObject(ptr long ptr ptr ptr) @ stdcall CoGetContextToken(ptr) @ stub CoGetCurrentLogicalThreadId

1 0

Dmitry Timoshkov : gdi32: Add EMR_GDICOMMENT to the list of records producing output.
by Alexandre Julliard 23 Oct '08

23 Oct '08

Module: wine Branch: master Commit: ccba8578f5c81ea0650cad277683cbaff6306caf URL: http://source.winehq.org/git/wine.git/?a=commit;h=ccba8578f5c81ea0650cad277… Author: Dmitry Timoshkov <dmitry(a)codeweavers.com> Date: Thu Oct 23 21:32:43 2008 +0900 gdi32: Add EMR_GDICOMMENT to the list of records producing output. This completely fixes the print previews generated by MS Access in a Win9x mode. Access has a custom handler for an EMR_GDICOMMENT record, and GDI APIs it calls from it don't work properly due to wrong DC mapping. --- dlls/gdi32/enhmetafile.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/gdi32/enhmetafile.c b/dlls/gdi32/enhmetafile.c index e998c60..22451d2 100644 --- a/dlls/gdi32/enhmetafile.c +++ b/dlls/gdi32/enhmetafile.c @@ -692,6 +692,7 @@ static BOOL emr_produces_output(int type) case EMR_LINETO: case EMR_ARCTO: case EMR_POLYDRAW: + case EMR_GDICOMMENT: case EMR_FILLRGN: case EMR_FRAMERGN: case EMR_INVERTRGN:

1 0

Alexandre Julliard : msiexec: Fix initial state of the argument parsing state machine.
by Alexandre Julliard 23 Oct '08

23 Oct '08

Module: wine Branch: master Commit: 8d7fc617177e0593fd6d0461fc6c9e74be41fe00 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8d7fc617177e0593fd6d0461f… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Oct 23 12:53:47 2008 +0200 msiexec: Fix initial state of the argument parsing state machine. --- programs/msiexec/msiexec.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/programs/msiexec/msiexec.c b/programs/msiexec/msiexec.c index 9649d82..7b00eb7 100644 --- a/programs/msiexec/msiexec.c +++ b/programs/msiexec/msiexec.c @@ -388,9 +388,9 @@ enum chomp_state static int chomp( WCHAR *str ) { - enum chomp_state state = cs_whitespace; + enum chomp_state state = cs_token; WCHAR *p, *out; - int count = 0, ignore; + int count = 1, ignore; for( p = str, out = str; *p; p++ ) {

1 0

Huw Davies : ole32: Call the appropriate storage function when we run the object.
by Alexandre Julliard 23 Oct '08

23 Oct '08

Module: wine Branch: master Commit: 92202feff3c46585c552cc439c5584074cf58a1b URL: http://source.winehq.org/git/wine.git/?a=commit;h=92202feff3c46585c552cc439… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Oct 23 10:25:24 2008 +0100 ole32: Call the appropriate storage function when we run the object. --- dlls/ole32/defaulthandler.c | 51 ++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 50 insertions(+), 1 deletions(-) diff --git a/dlls/ole32/defaulthandler.c b/dlls/ole32/defaulthandler.c index ce44aec..49bb733 100644 --- a/dlls/ole32/defaulthandler.c +++ b/dlls/ole32/defaulthandler.c @@ -64,6 +64,13 @@ WINE_DEFAULT_DEBUG_CHANNEL(ole); +enum storage_state +{ + storage_state_uninitialised, + storage_state_initialised, + storage_state_loaded +}; + /**************************************************************************** * DefaultHandler * @@ -119,6 +126,10 @@ struct DefaultHandler /* connection cookie for the advise on the delegate OLE object */ DWORD dwAdvConn; + + /* storage passed to Load or InitNew */ + IStorage *storage; + enum storage_state storage_state; }; typedef struct DefaultHandler DefaultHandler; @@ -1302,7 +1313,12 @@ static HRESULT WINAPI DefaultHandler_Run( IOleObject_QueryInterface(This->pOleDelegate, &IID_IPersistStorage, (void **)&This->pPSDelegate); if (This->pPSDelegate) - hr = IPersistStorage_InitNew(This->pPSDelegate, NULL); + { + if(This->storage_state == storage_state_initialised) + hr = IPersistStorage_InitNew(This->pPSDelegate, This->storage); + else if(This->storage_state == storage_state_loaded) + hr = IPersistStorage_Load(This->pPSDelegate, This->storage); + } } if (SUCCEEDED(hr) && This->containerApp) @@ -1548,6 +1564,13 @@ static HRESULT WINAPI DefaultHandler_IPersistStorage_InitNew( if(SUCCEEDED(hr) && object_is_running(This)) hr = IPersistStorage_InitNew(This->pPSDelegate, pStg); + if(SUCCEEDED(hr)) + { + IStorage_AddRef(pStg); + This->storage = pStg; + This->storage_state = storage_state_initialised; + } + return hr; } @@ -1570,6 +1593,12 @@ static HRESULT WINAPI DefaultHandler_IPersistStorage_Load( if(SUCCEEDED(hr) && object_is_running(This)) hr = IPersistStorage_Load(This->pPSDelegate, pStg); + if(SUCCEEDED(hr)) + { + IStorage_AddRef(pStg); + This->storage = pStg; + This->storage_state = storage_state_loaded; + } return hr; } @@ -1614,6 +1643,14 @@ static HRESULT WINAPI DefaultHandler_IPersistStorage_SaveCompleted( if(SUCCEEDED(hr) && object_is_running(This)) hr = IPersistStorage_SaveCompleted(This->pPSDelegate, pStgNew); + if(pStgNew) + { + IStorage_AddRef(pStgNew); + if(This->storage) IStorage_Release(This->storage); + This->storage = pStgNew; + This->storage_state = storage_state_loaded; + } + return hr; } @@ -1635,6 +1672,10 @@ static HRESULT WINAPI DefaultHandler_IPersistStorage_HandsOffStorage( if(SUCCEEDED(hr) && object_is_running(This)) hr = IPersistStorage_HandsOffStorage(This->pPSDelegate); + if(This->storage) IStorage_Release(This->storage); + This->storage = NULL; + This->storage_state = storage_state_uninitialised; + return hr; } @@ -1795,6 +1836,8 @@ static DefaultHandler* DefaultHandler_Construct( This->pDataDelegate = NULL; This->dwAdvConn = 0; + This->storage = NULL; + This->storage_state = storage_state_uninitialised; return This; } @@ -1836,6 +1879,12 @@ static void DefaultHandler_Destroy( This->dataAdviseHolder = NULL; } + if (This->storage) + { + IStorage_Release(This->storage); + This->storage = NULL; + } + HeapFree(GetProcessHeap(), 0, This); }

1 0