April 2011 - Wine-commits - list.winehq.org

Piotr Caban : msvcrt: Added sprintf_p_l implementation.
by Alexandre Julliard 27 Apr '11

27 Apr '11

Module: wine Branch: master Commit: 00182ca6d914dd716f9fd76201a7bc857091bae0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=00182ca6d914dd716f9fd7620… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Apr 27 12:09:01 2011 +0200 msvcrt: Added sprintf_p_l implementation. --- dlls/msvcr100/msvcr100.spec | 8 ++-- dlls/msvcr80/msvcr80.spec | 8 ++-- dlls/msvcr90/msvcr90.spec | 8 ++-- dlls/msvcrt/msvcrt.h | 2 + dlls/msvcrt/msvcrt.spec | 10 ++-- dlls/msvcrt/printf.h | 73 +++++++++++++++++++++++++++++ dlls/msvcrt/tests/printf.c | 44 +++++++++++++++++ dlls/msvcrt/wcs.c | 108 +++++++++++++++++++++++++++++++++++++++++++ include/msvcrt/crtdefs.h | 2 + 9 files changed, 246 insertions(+), 17 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=00182ca6d914dd716f9fd…

1 0

Piotr Caban : msvcrt: Print all integers using pf_integer_conv function.
by Alexandre Julliard 27 Apr '11

27 Apr '11

Module: wine Branch: master Commit: 35cc03d54cce08a1b621a6a0628ccaee93d27a1c URL: http://source.winehq.org/git/wine.git/?a=commit;h=35cc03d54cce08a1b621a6a06… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Apr 27 12:09:17 2011 +0200 msvcrt: Print all integers using pf_integer_conv function. --- dlls/msvcrt/printf.h | 25 ++++++++++++++----------- 1 files changed, 14 insertions(+), 11 deletions(-) diff --git a/dlls/msvcrt/printf.h b/dlls/msvcrt/printf.h index 104dd9c..7513605 100644 --- a/dlls/msvcrt/printf.h +++ b/dlls/msvcrt/printf.h @@ -493,7 +493,7 @@ int FUNC_NAME(pf_printf)(FUNC_NAME(puts_clbk) pf_puts, void *puts_ctx, const API int *used = pf_args(args_ctx, pos, VT_PTR, &valist).get_ptr; *used = written; i = 0; - } else if(flags.IntegerDouble && flags.Format && strchr("diouxX", flags.Format)) { + } else if(flags.Format && strchr("diouxX", flags.Format)) { char *tmp = buf; int max_len = (flags.FieldLength>flags.Precision ? flags.FieldLength : flags.Precision) + 10; @@ -502,13 +502,20 @@ int FUNC_NAME(pf_printf)(FUNC_NAME(puts_clbk) pf_puts, void *puts_ctx, const API if(!tmp) return -1; - FUNC_NAME(pf_integer_conv)(tmp, max_len, &flags, - pf_args(args_ctx, pos, VT_I8, &valist).get_longlong); + if(flags.IntegerDouble) + FUNC_NAME(pf_integer_conv)(tmp, max_len, &flags, pf_args(args_ctx, pos, + VT_I8, &valist).get_longlong); + else if(flags.Format=='d' || flags.Format=='i') + FUNC_NAME(pf_integer_conv)(tmp, max_len, &flags, pf_args(args_ctx, pos, + VT_INT, &valist).get_int); + else + FUNC_NAME(pf_integer_conv)(tmp, max_len, &flags, (unsigned)pf_args( + args_ctx, pos, VT_INT, &valist).get_int); i = FUNC_NAME(pf_output_format_str)(pf_puts, puts_ctx, tmp, -1, &flags, locale); if(tmp != buf) HeapFree(GetProcessHeap(), 0, tmp); - } else if(flags.Format && strchr("acCdeEfgGinouxX", flags.Format)) { + } else if(flags.Format && strchr("aeEfgG", flags.Format)) { char fmt[20], *tmp = buf, *decimal_point; int max_len = (flags.FieldLength>flags.Precision ? flags.FieldLength : flags.Precision) + 10; @@ -519,13 +526,9 @@ int FUNC_NAME(pf_printf)(FUNC_NAME(puts_clbk) pf_puts, void *puts_ctx, const API FUNC_NAME(pf_rebuild_format_string)(fmt, &flags); - if(flags.Format && strchr("aeEfgG", flags.Format)) { - sprintf(tmp, fmt, pf_args(args_ctx, pos, VT_R8, &valist).get_double); - if(toupper(flags.Format)=='E' || toupper(flags.Format)=='G') - FUNC_NAME(pf_fixup_exponent)(tmp); - } - else - sprintf(tmp, fmt, pf_args(args_ctx, pos, VT_INT, &valist).get_int); + sprintf(tmp, fmt, pf_args(args_ctx, pos, VT_R8, &valist).get_double); + if(toupper(flags.Format)=='E' || toupper(flags.Format)=='G') + FUNC_NAME(pf_fixup_exponent)(tmp); decimal_point = strchr(tmp, '.'); if(decimal_point)

1 0

Piotr Caban : msvcrt: Don't use additional buffer in pf_integer_conv.
by Alexandre Julliard 27 Apr '11

27 Apr '11

Module: wine Branch: master Commit: 0f9f0a16135a8a98ba71d8c62e3ffff01b2cff2e URL: http://source.winehq.org/git/wine.git/?a=commit;h=0f9f0a16135a8a98ba71d8c62… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Apr 27 12:09:27 2011 +0200 msvcrt: Don't use additional buffer in pf_integer_conv. --- dlls/msvcrt/printf.h | 32 +++++++++++++++----------------- 1 files changed, 15 insertions(+), 17 deletions(-) diff --git a/dlls/msvcrt/printf.h b/dlls/msvcrt/printf.h index 7513605..c3884ab 100644 --- a/dlls/msvcrt/printf.h +++ b/dlls/msvcrt/printf.h @@ -254,12 +254,9 @@ static inline void FUNC_NAME(pf_integer_conv)(char *buf, int buf_len, { unsigned int base; const char *digits; + char tmp; int i, j, k; - char number[40], *tmp = number; - - if(buf_len > sizeof number) - tmp = HeapAlloc(GetProcessHeap(), 0, buf_len); if(flags->Format == 'o') base = 8; @@ -280,35 +277,36 @@ static inline void FUNC_NAME(pf_integer_conv)(char *buf, int buf_len, i = 0; if(x==0 && flags->Precision) - tmp[i++] = '0'; + buf[i++] = '0'; else { while(x != 0) { j = (ULONGLONG)x%base; x = (ULONGLONG)x/base; - tmp[i++] = digits[j]; + buf[i++] = digits[j]; } } k = flags->Precision-i; while(k-- > 0) - tmp[i++] = '0'; + buf[i++] = '0'; if(flags->Alternate) { if(base == 16) { - tmp[i++] = digits[16]; - tmp[i++] = '0'; - } else if(base==8 && tmp[i-1]!='0') - tmp[i++] = '0'; + buf[i++] = digits[16]; + buf[i++] = '0'; + } else if(base==8 && buf[i-1]!='0') + buf[i++] = '0'; } + buf[i] = '\0'; j = 0; - while(i-- > 0) - buf[j++] = tmp[i]; - buf[j] = '\0'; + while(i-- > j) { + tmp = buf[j]; + buf[j] = buf[i]; + buf[i] = tmp; + j++; + } /* Adjust precision so pf_fill won't truncate the number later */ flags->Precision = strlen(buf); - - if(tmp != number) - HeapFree(GetProcessHeap(), 0, tmp); } static inline void FUNC_NAME(pf_fixup_exponent)(char *buf)

1 0

Piotr Caban : msvcrt: Use pf_integer_conv when displaying pointers.
by Alexandre Julliard 27 Apr '11

27 Apr '11

Module: wine Branch: master Commit: 8e58cbd8cb312dd22efd578d2b4ca856042f2d54 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8e58cbd8cb312dd22efd578d2… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Apr 27 12:09:37 2011 +0200 msvcrt: Use pf_integer_conv when displaying pointers. --- dlls/msvcrt/printf.h | 13 +++++++------ 1 files changed, 7 insertions(+), 6 deletions(-) diff --git a/dlls/msvcrt/printf.h b/dlls/msvcrt/printf.h index c3884ab..93a8d43 100644 --- a/dlls/msvcrt/printf.h +++ b/dlls/msvcrt/printf.h @@ -478,13 +478,14 @@ int FUNC_NAME(pf_printf)(FUNC_NAME(puts_clbk) pf_puts, void *puts_ctx, const API i = FUNC_NAME(pf_handle_string)(pf_puts, puts_ctx, &ch, 1, &flags, locale); } else if(flags.Format == 'p') { - void *ptr = pf_args(args_ctx, pos, VT_PTR, &valist).get_ptr; - + flags.Format = 'X'; + flags.PadZero = '0'; + i = flags.Precision; + flags.Precision = 2*sizeof(void*); + FUNC_NAME(pf_integer_conv)(buf, sizeof(buf), &flags, + pf_args(args_ctx, pos, VT_INT, &valist).get_int); flags.PadZero = 0; - if(flags.Alternate) - sprintf(buf, "0X%0*lX", 2*(int)sizeof(ptr), (ULONG_PTR)ptr); - else - sprintf(buf, "%0*lX", 2*(int)sizeof(ptr), (ULONG_PTR)ptr); + flags.Precision = i; i = FUNC_NAME(pf_output_format_str)(pf_puts, puts_ctx, buf, -1, &flags, locale); } else if(flags.Format == 'n') {

1 0

Piotr Caban : msvcrt: Remove wchar_t/ char conversions needed for pf_integer_conv.
by Alexandre Julliard 27 Apr '11

27 Apr '11

Module: wine Branch: master Commit: 10901e17b4107eacd5cc06e3a8296feb4ed51d27 URL: http://source.winehq.org/git/wine.git/?a=commit;h=10901e17b4107eacd5cc06e3a… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Wed Apr 27 12:09:51 2011 +0200 msvcrt: Remove wchar_t/char conversions needed for pf_integer_conv. --- dlls/msvcrt/printf.h | 36 +++++++++++++++++++++--------------- 1 files changed, 21 insertions(+), 15 deletions(-) diff --git a/dlls/msvcrt/printf.h b/dlls/msvcrt/printf.h index 93a8d43..0198756 100644 --- a/dlls/msvcrt/printf.h +++ b/dlls/msvcrt/printf.h @@ -249,13 +249,11 @@ static inline void FUNC_NAME(pf_rebuild_format_string)(char *p, FUNC_NAME(pf_fla /* pf_integer_conv: prints x to buf, including alternate formats and additional precision digits, but not field characters or the sign */ -static inline void FUNC_NAME(pf_integer_conv)(char *buf, int buf_len, +static inline void FUNC_NAME(pf_integer_conv)(APICHAR *buf, int buf_len, FUNC_NAME(pf_flags) *flags, LONGLONG x) { unsigned int base; const char *digits; - char tmp; - int i, j, k; if(flags->Format == 'o') @@ -296,17 +294,17 @@ static inline void FUNC_NAME(pf_integer_conv)(char *buf, int buf_len, buf[i++] = '0'; } + /* Adjust precision so pf_fill won't truncate the number later */ + flags->Precision = i; + buf[i] = '\0'; j = 0; - while(i-- > j) { - tmp = buf[j]; + while(--i > j) { + APICHAR tmp = buf[j]; buf[j] = buf[i]; buf[i] = tmp; j++; } - - /* Adjust precision so pf_fill won't truncate the number later */ - flags->Precision = strlen(buf); } static inline void FUNC_NAME(pf_fixup_exponent)(char *buf) @@ -342,9 +340,9 @@ int FUNC_NAME(pf_printf)(FUNC_NAME(puts_clbk) pf_puts, void *puts_ctx, const API args_clbk pf_args, void *args_ctx, __ms_va_list valist) { const APICHAR *q, *p = fmt; + APICHAR buf[32]; int written = 0, pos, i; FUNC_NAME(pf_flags) flags; - char buf[32]; TRACE("Format is: %s\n", FUNC_NAME(debugstr)(fmt)); @@ -482,21 +480,25 @@ int FUNC_NAME(pf_printf)(FUNC_NAME(puts_clbk) pf_puts, void *puts_ctx, const API flags.PadZero = '0'; i = flags.Precision; flags.Precision = 2*sizeof(void*); - FUNC_NAME(pf_integer_conv)(buf, sizeof(buf), &flags, + FUNC_NAME(pf_integer_conv)(buf, sizeof(buf)/sizeof(APICHAR), &flags, pf_args(args_ctx, pos, VT_INT, &valist).get_int); flags.PadZero = 0; flags.Precision = i; +#ifdef PRINTF_WIDE + i = FUNC_NAME(pf_output_format_wstr)(pf_puts, puts_ctx, buf, -1, &flags, locale); +#else i = FUNC_NAME(pf_output_format_str)(pf_puts, puts_ctx, buf, -1, &flags, locale); +#endif } else if(flags.Format == 'n') { int *used = pf_args(args_ctx, pos, VT_PTR, &valist).get_ptr; *used = written; i = 0; } else if(flags.Format && strchr("diouxX", flags.Format)) { - char *tmp = buf; + APICHAR *tmp = buf; int max_len = (flags.FieldLength>flags.Precision ? flags.FieldLength : flags.Precision) + 10; - if(max_len > sizeof(buf)) + if(max_len > sizeof(buf)/sizeof(APICHAR)) tmp = HeapAlloc(GetProcessHeap(), 0, max_len); if(!tmp) return -1; @@ -511,14 +513,18 @@ int FUNC_NAME(pf_printf)(FUNC_NAME(puts_clbk) pf_puts, void *puts_ctx, const API FUNC_NAME(pf_integer_conv)(tmp, max_len, &flags, (unsigned)pf_args( args_ctx, pos, VT_INT, &valist).get_int); +#ifdef PRINTF_WIDE + i = FUNC_NAME(pf_output_format_wstr)(pf_puts, puts_ctx, tmp, -1, &flags, locale); +#else i = FUNC_NAME(pf_output_format_str)(pf_puts, puts_ctx, tmp, -1, &flags, locale); +#endif if(tmp != buf) HeapFree(GetProcessHeap(), 0, tmp); } else if(flags.Format && strchr("aeEfgG", flags.Format)) { - char fmt[20], *tmp = buf, *decimal_point; + char fmt[20], buf_a[32], *tmp = buf_a, *decimal_point; int max_len = (flags.FieldLength>flags.Precision ? flags.FieldLength : flags.Precision) + 10; - if(max_len > sizeof(buf)) + if(max_len > sizeof(buf_a)) tmp = HeapAlloc(GetProcessHeap(), 0, max_len); if(!tmp) return -1; @@ -534,7 +540,7 @@ int FUNC_NAME(pf_printf)(FUNC_NAME(puts_clbk) pf_puts, void *puts_ctx, const API *decimal_point = *locale->locinfo->lconv->decimal_point; i = FUNC_NAME(pf_output_str)(pf_puts, puts_ctx, tmp, strlen(tmp), locale); - if(tmp != buf) + if(tmp != buf_a) HeapFree(GetProcessHeap(), 0, tmp); } else { if(invoke_invalid_param_handler) {

1 0

Alexandre Julliard : winex11: Ignore MotionNotify events if we have pending RawMotion events.
by Alexandre Julliard 27 Apr '11

27 Apr '11

Module: wine Branch: master Commit: da9922b40da5bc581815a4357340f3f128e8c5cb URL: http://source.winehq.org/git/wine.git/?a=commit;h=da9922b40da5bc581815a4357… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Apr 27 16:00:02 2011 +0200 winex11: Ignore MotionNotify events if we have pending RawMotion events. --- dlls/winex11.drv/event.c | 26 +++++++++++++++++++++++++- dlls/winex11.drv/mouse.c | 1 - dlls/winex11.drv/x11drv.h | 1 + 3 files changed, 26 insertions(+), 2 deletions(-) diff --git a/dlls/winex11.drv/event.c b/dlls/winex11.drv/event.c index 0233a61..30d8553 100644 --- a/dlls/winex11.drv/event.c +++ b/dlls/winex11.drv/event.c @@ -32,6 +32,9 @@ #include <X11/Xlib.h> #include <X11/Xresource.h> #include <X11/Xutil.h> +#ifdef HAVE_X11_EXTENSIONS_XINPUT2_H +#include <X11/extensions/XInput2.h> +#endif #include <assert.h> #include <stdarg.h> @@ -154,6 +157,8 @@ static const char * event_names[MAX_EVENT_HANDLERS] = "SelectionNotify", "ColormapNotify", "ClientMessage", "MappingNotify", "GenericEvent" }; +int xinput2_opcode = 0; + /* return the name of an X event */ static const char *dbgstr_event( int type ) { @@ -236,7 +241,8 @@ enum event_merge_action { MERGE_DISCARD, /* discard the old event */ MERGE_HANDLE, /* handle the old event */ - MERGE_KEEP /* keep the old event for future merging */ + MERGE_KEEP, /* keep the old event for future merging */ + MERGE_IGNORE /* ignore the new event, keep the old one */ }; /*********************************************************************** @@ -270,6 +276,22 @@ static enum event_merge_action merge_events( XEvent *prev, XEvent *next ) return MERGE_DISCARD; } break; +#ifdef HAVE_X11_EXTENSIONS_XINPUT2_H + case GenericEvent: + if (prev->xcookie.extension != xinput2_opcode) break; + if (prev->xcookie.evtype != XI_RawMotion) break; + switch (next->type) + { + case MotionNotify: + if (next->xany.window == x11drv_thread_data()->clip_window) + { + TRACE( "ignoring MotionNotify for clip window\n" ); + return MERGE_IGNORE; + } + break; + } + break; +#endif } return MERGE_HANDLE; } @@ -363,6 +385,8 @@ static int process_events( Display *display, Bool (*filter)(Display*, XEvent*,XP break; case MERGE_KEEP: /* handle new, keep prev for future merging */ call_event_handler( display, &event ); + /* fall through */ + case MERGE_IGNORE: /* ignore new, keep prev for future merging */ free_event_data( &event ); break; } diff --git a/dlls/winex11.drv/mouse.c b/dlls/winex11.drv/mouse.c index c4e0b59..7f45a2a 100644 --- a/dlls/winex11.drv/mouse.c +++ b/dlls/winex11.drv/mouse.c @@ -130,7 +130,6 @@ static Cursor create_cursor( HANDLE handle ); #ifdef HAVE_X11_EXTENSIONS_XINPUT2_H static BOOL xinput2_available; -static int xinput2_opcode; static int xinput2_core_pointer; #define MAKE_FUNCPTR(f) static typeof(f) * p##f MAKE_FUNCPTR(XIFreeDeviceInfo); diff --git a/dlls/winex11.drv/x11drv.h b/dlls/winex11.drv/x11drv.h index cfe2509..7113a02 100644 --- a/dlls/winex11.drv/x11drv.h +++ b/dlls/winex11.drv/x11drv.h @@ -733,6 +733,7 @@ extern void X11DRV_SelectionClear( HWND hWnd, XEvent *event ); extern void X11DRV_MappingNotify( HWND hWnd, XEvent *event ); extern void X11DRV_GenericEvent( HWND hwnd, XEvent *event ); +extern int xinput2_opcode; extern Bool (*pXGetEventData)( Display *display, XEvent /*XGenericEventCookie*/ *event ); extern void (*pXFreeEventData)( Display *display, XEvent /*XGenericEventCookie*/ *event );

1 0

Alexandre Julliard : winex11: Merge consecutive RawMotion events.
by Alexandre Julliard 27 Apr '11

27 Apr '11

Module: wine Branch: master Commit: bab850389ae88681f9eaced94fe97469e1fc1d44 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bab850389ae88681f9eaced94… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Apr 27 16:00:50 2011 +0200 winex11: Merge consecutive RawMotion events. --- dlls/winex11.drv/event.c | 44 ++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 44 insertions(+), 0 deletions(-) diff --git a/dlls/winex11.drv/event.c b/dlls/winex11.drv/event.c index 30d8553..acaf49c 100644 --- a/dlls/winex11.drv/event.c +++ b/dlls/winex11.drv/event.c @@ -246,6 +246,46 @@ enum event_merge_action }; /*********************************************************************** + * merge_raw_motion_events + */ +#ifdef HAVE_X11_EXTENSIONS_XINPUT2_H +static enum event_merge_action merge_raw_motion_events( XIRawEvent *prev, XIRawEvent *next ) +{ + int i, j, k; + unsigned char mask; + + if (!prev->valuators.mask_len) return MERGE_HANDLE; + if (!next->valuators.mask_len) return MERGE_HANDLE; + + mask = prev->valuators.mask[0] | next->valuators.mask[0]; + if (mask == next->valuators.mask[0]) /* keep next */ + { + for (i = j = k = 0; i < 8; i++) + { + if (XIMaskIsSet( prev->valuators.mask, i )) + next->valuators.values[j] += prev->valuators.values[k++]; + if (XIMaskIsSet( next->valuators.mask, i )) j++; + } + TRACE( "merging duplicate GenericEvent\n" ); + return MERGE_DISCARD; + } + if (mask == prev->valuators.mask[0]) /* keep prev */ + { + for (i = j = k = 0; i < 8; i++) + { + if (XIMaskIsSet( next->valuators.mask, i )) + prev->valuators.values[j] += next->valuators.values[k++]; + if (XIMaskIsSet( prev->valuators.mask, i )) j++; + } + TRACE( "merging duplicate GenericEvent\n" ); + return MERGE_IGNORE; + } + /* can't merge events with disjoint masks */ + return MERGE_HANDLE; +} +#endif + +/*********************************************************************** * merge_events * * Try to merge 2 consecutive events. @@ -289,6 +329,10 @@ static enum event_merge_action merge_events( XEvent *prev, XEvent *next ) return MERGE_IGNORE; } break; + case GenericEvent: + if (next->xcookie.extension != xinput2_opcode) break; + if (next->xcookie.evtype != XI_RawMotion) break; + return merge_raw_motion_events( prev->xcookie.data, next->xcookie.data ); } break; #endif

1 0

André Hentschel : winedbg: Copy the x86 implementation of fetch_integer for ARM.
by Alexandre Julliard 27 Apr '11

27 Apr '11

Module: wine Branch: master Commit: ac3ed42601f24e9bc79629f34ee8204cb7684738 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ac3ed42601f24e9bc79629f34… Author: André Hentschel <nerv(a)dawncrow.de> Date: Tue Apr 26 23:11:06 2011 +0200 winedbg: Copy the x86 implementation of fetch_integer for ARM. --- programs/winedbg/be_arm.c | 17 +++++++++++++++-- 1 files changed, 15 insertions(+), 2 deletions(-) diff --git a/programs/winedbg/be_arm.c b/programs/winedbg/be_arm.c index 0379bd9..f34ecc7 100644 --- a/programs/winedbg/be_arm.c +++ b/programs/winedbg/be_arm.c @@ -191,8 +191,21 @@ static int be_arm_adjust_pc_for_break(CONTEXT* ctx, BOOL way) static int be_arm_fetch_integer(const struct dbg_lvalue* lvalue, unsigned size, unsigned ext_sign, LONGLONG* ret) { - dbg_printf("be_arm_fetch_integer: not done\n"); - return FALSE; + if (size != 1 && size != 2 && size != 4 && size != 8) return FALSE; + + memset(ret, 0, sizeof(*ret)); /* clear unread bytes */ + /* FIXME: this assumes that debuggee and debugger use the same + * integral representation + */ + if (!memory_read_value(lvalue, size, ret)) return FALSE; + + /* propagate sign information */ + if (ext_sign && size < 8 && (*ret >> (size * 8 - 1)) != 0) + { + ULONGLONG neg = -1; + *ret |= neg << (size * 8); + } + return TRUE; } static int be_arm_fetch_float(const struct dbg_lvalue* lvalue, unsigned size,

1 0

Michael Stefaniuc : shell32/tests: COM cleanup in shlfolder.c.
by Alexandre Julliard 27 Apr '11

27 Apr '11

Module: wine Branch: master Commit: 1da5c9d17f1157f501d20d4f3375bedda0687493 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1da5c9d17f1157f501d20d4f3… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Tue Apr 26 23:45:13 2011 +0200 shell32/tests: COM cleanup in shlfolder.c. --- dlls/shell32/tests/shlfolder.c | 13 +++++++++---- 1 files changed, 9 insertions(+), 4 deletions(-) diff --git a/dlls/shell32/tests/shlfolder.c b/dlls/shell32/tests/shlfolder.c index cc68280..681a28f 100644 --- a/dlls/shell32/tests/shlfolder.c +++ b/dlls/shell32/tests/shlfolder.c @@ -2965,7 +2965,7 @@ cleanup: /**************************************************************/ /* IUnknown implementation for counting QueryInterface calls. */ typedef struct { - const IUnknownVtbl *lpVtbl; + IUnknown IUnknown_iface; struct if_count { REFIID id; LONG count; @@ -2973,9 +2973,14 @@ typedef struct { LONG unknown; } IUnknownImpl; +static inline IUnknownImpl *impl_from_IUnknown(IUnknown *iface) +{ + return CONTAINING_RECORD(iface, IUnknownImpl, IUnknown_iface); +} + static HRESULT WINAPI unk_fnQueryInterface(IUnknown *iunk, REFIID riid, void** punk) { - IUnknownImpl *This = (IUnknownImpl*)iunk; + IUnknownImpl *This = impl_from_IUnknown(iunk); UINT i, found; for(i = found = 0; This->ifaces[i].id != NULL; i++) { @@ -3042,7 +3047,7 @@ static void test_SHGetIDListFromObject(void) ok(hres == E_NOINTERFACE, "Got %x\n", hres); punkimpl = HeapAlloc(GetProcessHeap(), 0, sizeof(IUnknownImpl)); - punkimpl->lpVtbl = &vt_IUnknown; + punkimpl->IUnknown_iface.lpVtbl = &vt_IUnknown; punkimpl->ifaces = ifaces; punkimpl->unknown = 0; @@ -3213,7 +3218,7 @@ static void test_SHGetItemFromObject(void) ok(hres == E_NOINTERFACE, "Got 0x%08x\n", hres); punkimpl = HeapAlloc(GetProcessHeap(), 0, sizeof(IUnknownImpl)); - punkimpl->lpVtbl = &vt_IUnknown; + punkimpl->IUnknown_iface.lpVtbl = &vt_IUnknown; punkimpl->ifaces = ifaces; punkimpl->unknown = 0;

1 0

Michael Stefaniuc : msxml3: COM cleanup in domdoc.c.
by Alexandre Julliard 27 Apr '11

27 Apr '11

Module: wine Branch: master Commit: 31057e4b75931360009d26c66e61922a77ee7a78 URL: http://source.winehq.org/git/wine.git/?a=commit;h=31057e4b75931360009d26c66… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Tue Apr 26 23:47:48 2011 +0200 msxml3: COM cleanup in domdoc.c. --- dlls/msxml3/domdoc.c | 97 +++++++++++++++++++++++++------------------------ 1 files changed, 49 insertions(+), 48 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=31057e4b75931360009d2…

1 0