October 2011 - Wine-Commits

Piotr Caban : msvcrt: Fix __pioinfo block size.
by Alexandre Julliard Oct. 6, 2011

Oct. 6, 2011

Module: wine Branch: master Commit: 190c67c70e0cd3700c04d9c2c42f1389bbd2e691 URL: http://source.winehq.org/git/wine.git/?a=commit;h=190c67c70e0cd3700c04d9c2c… Author: Piotr Caban <piotr(a)codeweavers.com> Date: Thu Oct 6 20:13:16 2011 +0200 msvcrt: Fix __pioinfo block size. --- dlls/msvcrt/file.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/msvcrt/file.c b/dlls/msvcrt/file.c index f1af1b7..ecbec34 100644 --- a/dlls/msvcrt/file.c +++ b/dlls/msvcrt/file.c @@ -69,7 +69,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(msvcrt); /* FIXME: this should be allocated dynamically */ #define MSVCRT_MAX_FILES 2048 -#define MSVCRT_FD_BLOCK_SIZE 64 +#define MSVCRT_FD_BLOCK_SIZE 32 /* ioinfo structure size is different in msvcrXX.dll's */ typedef struct { @@ -82,7 +82,7 @@ typedef struct { /********************************************************************* * __pioinfo (MSVCRT.@) - * array of pointers to ioinfo arrays [64] + * array of pointers to ioinfo arrays [32] */ ioinfo * MSVCRT___pioinfo[MSVCRT_MAX_FILES/MSVCRT_FD_BLOCK_SIZE] = { 0 }; @@ -229,7 +229,7 @@ static HANDLE msvcrt_fdtoh(int fd) return INVALID_HANDLE_VALUE; } if (msvcrt_get_ioinfo(fd)->handle == INVALID_HANDLE_VALUE) - FIXME("returning INVALID_HANDLE_VALUE for %d\n", fd); + WARN("returning INVALID_HANDLE_VALUE for %d\n", fd); return msvcrt_get_ioinfo(fd)->handle; }

1 0

Damjan Jovanovic : windowscodecs: Read PNGs sequentially.
by Alexandre Julliard Oct. 6, 2011

Oct. 6, 2011

Module: wine Branch: master Commit: ea35386b2b11f4f74f8dfbae5f614ee49689e6af URL: http://source.winehq.org/git/wine.git/?a=commit;h=ea35386b2b11f4f74f8dfbae5… Author: Damjan Jovanovic <damjan.jov(a)gmail.com> Date: Tue Oct 4 16:44:46 2011 +0200 windowscodecs: Read PNGs sequentially. --- dlls/windowscodecs/pngformat.c | 185 +++++++++++++++++++--------------------- 1 files changed, 87 insertions(+), 98 deletions(-) diff --git a/dlls/windowscodecs/pngformat.c b/dlls/windowscodecs/pngformat.c index f5f6fdb..b3d92fd 100644 --- a/dlls/windowscodecs/pngformat.c +++ b/dlls/windowscodecs/pngformat.c @@ -56,11 +56,8 @@ MAKE_FUNCPTR(png_get_image_height); MAKE_FUNCPTR(png_get_image_width); MAKE_FUNCPTR(png_get_io_ptr); MAKE_FUNCPTR(png_get_pHYs); -MAKE_FUNCPTR(png_get_progressive_ptr); MAKE_FUNCPTR(png_get_PLTE); MAKE_FUNCPTR(png_get_tRNS); -MAKE_FUNCPTR(png_process_data); -MAKE_FUNCPTR(png_progressive_combine_row); MAKE_FUNCPTR(png_set_bgr); MAKE_FUNCPTR(png_set_error_fn); #if HAVE_PNG_SET_EXPAND_GRAY_1_2_4_TO_8 @@ -72,7 +69,6 @@ MAKE_FUNCPTR(png_set_filler); MAKE_FUNCPTR(png_set_gray_to_rgb); MAKE_FUNCPTR(png_set_IHDR); MAKE_FUNCPTR(png_set_pHYs); -MAKE_FUNCPTR(png_set_progressive_read_fn); MAKE_FUNCPTR(png_set_read_fn); MAKE_FUNCPTR(png_set_strip_16); MAKE_FUNCPTR(png_set_tRNS_to_alpha); @@ -80,7 +76,6 @@ MAKE_FUNCPTR(png_set_write_fn); MAKE_FUNCPTR(png_read_end); MAKE_FUNCPTR(png_read_image); MAKE_FUNCPTR(png_read_info); -MAKE_FUNCPTR(png_read_update_info); MAKE_FUNCPTR(png_write_end); MAKE_FUNCPTR(png_write_info); MAKE_FUNCPTR(png_write_rows); @@ -108,11 +103,8 @@ static void *load_libpng(void) LOAD_FUNCPTR(png_get_image_width); LOAD_FUNCPTR(png_get_io_ptr); LOAD_FUNCPTR(png_get_pHYs); - LOAD_FUNCPTR(png_get_progressive_ptr); LOAD_FUNCPTR(png_get_PLTE); LOAD_FUNCPTR(png_get_tRNS); - LOAD_FUNCPTR(png_process_data); - LOAD_FUNCPTR(png_progressive_combine_row); LOAD_FUNCPTR(png_set_bgr); LOAD_FUNCPTR(png_set_error_fn); #if HAVE_PNG_SET_EXPAND_GRAY_1_2_4_TO_8 @@ -124,7 +116,6 @@ static void *load_libpng(void) LOAD_FUNCPTR(png_set_gray_to_rgb); LOAD_FUNCPTR(png_set_IHDR); LOAD_FUNCPTR(png_set_pHYs); - LOAD_FUNCPTR(png_set_progressive_read_fn); LOAD_FUNCPTR(png_set_read_fn); LOAD_FUNCPTR(png_set_strip_16); LOAD_FUNCPTR(png_set_tRNS_to_alpha); @@ -132,7 +123,6 @@ static void *load_libpng(void) LOAD_FUNCPTR(png_read_end); LOAD_FUNCPTR(png_read_image); LOAD_FUNCPTR(png_read_info); - LOAD_FUNCPTR(png_read_update_info); LOAD_FUNCPTR(png_write_end); LOAD_FUNCPTR(png_write_info); LOAD_FUNCPTR(png_write_rows); @@ -165,6 +155,7 @@ typedef struct { LONG ref; png_structp png_ptr; png_infop info_ptr; + png_infop end_info; BOOL initialized; int bpp; int width, height; @@ -228,7 +219,7 @@ static ULONG WINAPI PngDecoder_Release(IWICBitmapDecoder *iface) if (ref == 0) { if (This->png_ptr) - ppng_destroy_read_struct(&This->png_ptr, &This->info_ptr, NULL); + ppng_destroy_read_struct(&This->png_ptr, &This->info_ptr, &This->end_info); This->lock.DebugInfo->Spare[0] = 0; DeleteCriticalSection(&This->lock); HeapFree(GetProcessHeap(), 0, This->image_bits); @@ -245,18 +236,86 @@ static HRESULT WINAPI PngDecoder_QueryCapability(IWICBitmapDecoder *iface, IStre return E_NOTIMPL; } -static void info_callback(png_structp png_ptr, png_infop info) +static void user_read_data(png_structp png_ptr, png_bytep data, png_size_t length) { + IStream *stream = ppng_get_io_ptr(png_ptr); + HRESULT hr; + ULONG bytesread; + + hr = IStream_Read(stream, data, length, &bytesread); + if (FAILED(hr) || bytesread != length) + { + ppng_error(png_ptr, "failed reading data"); + } +} + +static HRESULT WINAPI PngDecoder_Initialize(IWICBitmapDecoder *iface, IStream *pIStream, + WICDecodeOptions cacheOptions) +{ + PngDecoder *This = impl_from_IWICBitmapDecoder(iface); + LARGE_INTEGER seek; + HRESULT hr=S_OK; + png_bytep *row_pointers=NULL; + UINT image_size; + UINT i; int color_type, bit_depth; png_bytep trans; int num_trans; png_uint_32 transparency; png_color_16p trans_values; - UINT image_size; - HRESULT hr = S_OK; - PngDecoder *This; + jmp_buf jmpbuf; + + TRACE("(%p,%p,%x)\n", iface, pIStream, cacheOptions); + + EnterCriticalSection(&This->lock); + + /* initialize libpng */ + This->png_ptr = ppng_create_read_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL); + if (!This->png_ptr) + { + hr = E_FAIL; + goto end; + } - This = (PngDecoder*) ppng_get_progressive_ptr(png_ptr); + This->info_ptr = ppng_create_info_struct(This->png_ptr); + if (!This->info_ptr) + { + ppng_destroy_read_struct(&This->png_ptr, NULL, NULL); + This->png_ptr = NULL; + hr = E_FAIL; + goto end; + } + + This->end_info = ppng_create_info_struct(This->png_ptr); + if (!This->info_ptr) + { + ppng_destroy_read_struct(&This->png_ptr, &This->info_ptr, NULL); + This->png_ptr = NULL; + hr = E_FAIL; + goto end; + } + + /* set up setjmp/longjmp error handling */ + if (setjmp(jmpbuf)) + { + ppng_destroy_read_struct(&This->png_ptr, &This->info_ptr, &This->end_info); + HeapFree(GetProcessHeap(), 0, row_pointers); + This->png_ptr = NULL; + hr = E_FAIL; + goto end; + } + ppng_set_error_fn(This->png_ptr, &jmpbuf, user_error_fn, user_warning_fn); + + /* seek to the start of the stream */ + seek.QuadPart = 0; + hr = IStream_Seek(pIStream, seek, STREAM_SEEK_SET, NULL); + if (FAILED(hr)) goto end; + + /* set up custom i/o handling */ + ppng_set_read_fn(This->png_ptr, pIStream, user_read_data); + + /* read the header */ + ppng_read_info(This->png_ptr, This->info_ptr); /* choose a pixel format */ color_type = ppng_get_color_type(This->png_ptr, This->info_ptr); @@ -355,6 +414,7 @@ static void info_callback(png_structp png_ptr, png_infop info) goto end; } + /* read the image data */ This->width = ppng_get_image_width(This->png_ptr, This->info_ptr); This->height = ppng_get_image_height(This->png_ptr, This->info_ptr); This->stride = This->width * This->bpp; @@ -367,96 +427,24 @@ static void info_callback(png_structp png_ptr, png_infop info) goto end; } - ppng_read_update_info(This->png_ptr, This->info_ptr); - This->initialized = TRUE; - -end: - if (FAILED(hr)) - ERR("PNG info parsing failed, hr=0x%08X\n", hr); -} - -static void row_callback(png_structp png_ptr, png_bytep new_row, - png_uint_32 row_num, int pass) -{ - png_bytep old_row; - PngDecoder *This; - - This = (PngDecoder*) ppng_get_progressive_ptr(png_ptr); - if (!This->initialized) - return; - - old_row = ((png_bytep)This->image_bits) + row_num*This->stride; - ppng_progressive_combine_row(png_ptr, old_row, new_row); -} - -static void end_callback(png_structp png_ptr, png_infop info) -{ -} - -static HRESULT WINAPI PngDecoder_Initialize(IWICBitmapDecoder *iface, IStream *pIStream, - WICDecodeOptions cacheOptions) -{ - PngDecoder *This = impl_from_IWICBitmapDecoder(iface); - LARGE_INTEGER seek; - HRESULT hr=S_OK; - png_bytep *row_pointers=NULL; - jmp_buf jmpbuf; - BYTE buffer[8096]; - ULONG bytesRead = 0; - - TRACE("(%p,%p,%x)\n", iface, pIStream, cacheOptions); - - EnterCriticalSection(&This->lock); - - /* initialize libpng */ - This->png_ptr = ppng_create_read_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL); - if (!This->png_ptr) + row_pointers = HeapAlloc(GetProcessHeap(), 0, sizeof(png_bytep)*This->height); + if (!row_pointers) { - hr = E_FAIL; + hr = E_OUTOFMEMORY; goto end; } - This->info_ptr = ppng_create_info_struct(This->png_ptr); - if (!This->info_ptr) - { - ppng_destroy_read_struct(&This->png_ptr, NULL, NULL); - This->png_ptr = NULL; - hr = E_FAIL; - goto end; - } + for (i=0; i<This->height; i++) + row_pointers[i] = This->image_bits + i * This->stride; - /* set up setjmp/longjmp error handling */ - if (setjmp(jmpbuf)) - { - ppng_destroy_read_struct(&This->png_ptr, &This->info_ptr, NULL); - HeapFree(GetProcessHeap(), 0, row_pointers); - This->png_ptr = NULL; - hr = E_FAIL; - goto end; - } - ppng_set_error_fn(This->png_ptr, &jmpbuf, user_error_fn, user_warning_fn); + ppng_read_image(This->png_ptr, row_pointers); - /* seek to the start of the stream */ - seek.QuadPart = 0; - hr = IStream_Seek(pIStream, seek, STREAM_SEEK_SET, NULL); - if (FAILED(hr)) goto end; + HeapFree(GetProcessHeap(), 0, row_pointers); + row_pointers = NULL; - /* set up progressive loading */ - ppng_set_progressive_read_fn(This->png_ptr, (void*)This, - info_callback, row_callback, end_callback); + ppng_read_end(This->png_ptr, This->end_info); - do - { - hr = IStream_Read(pIStream, buffer, sizeof(buffer), &bytesRead); - if (SUCCEEDED(hr)) - ppng_process_data(This->png_ptr, This->info_ptr, buffer, bytesRead); - } while (bytesRead == sizeof(buffer)); - if (hr == S_FALSE) - hr = S_OK; - if (FAILED(hr)) - ERR("reading PNG file failed, error 0x%08X\n", hr); - if (!This->initialized) - hr = E_FAIL; + This->initialized = TRUE; end: @@ -780,6 +768,7 @@ HRESULT PngDecoder_CreateInstance(IUnknown *pUnkOuter, REFIID iid, void** ppv) This->ref = 1; This->png_ptr = NULL; This->info_ptr = NULL; + This->end_info = NULL; This->initialized = FALSE; This->image_bits = NULL; InitializeCriticalSection(&This->lock);

1 0

Frédéric Delanoy : cmd: Infer handle type from handle value in WCMD_fgets and WCMD_ReadAndParseLine .
by Alexandre Julliard Oct. 6, 2011

Oct. 6, 2011

Module: wine Branch: master Commit: 5e3cb1867b8a78a29ba62a306ba9f2ed5d572be4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5e3cb1867b8a78a29ba62a306… Author: Frédéric Delanoy <frederic.delanoy(a)gmail.com> Date: Thu Oct 6 18:50:31 2011 +0200 cmd: Infer handle type from handle value in WCMD_fgets and WCMD_ReadAndParseLine. --- programs/cmd/batch.c | 6 +++--- programs/cmd/builtins.c | 4 ++-- programs/cmd/wcmd.h | 9 ++++++--- programs/cmd/wcmdmain.c | 23 +++++++---------------- 4 files changed, 18 insertions(+), 24 deletions(-) diff --git a/programs/cmd/batch.c b/programs/cmd/batch.c index 77a49c9..79bb009 100644 --- a/programs/cmd/batch.c +++ b/programs/cmd/batch.c @@ -89,7 +89,7 @@ void WCMD_batch (WCHAR *file, WCHAR *command, int called, WCHAR *startLabel, HAN while (context -> skip_rest == FALSE) { CMD_LIST *toExecute = NULL; /* Commands left to be executed */ - if (!WCMD_ReadAndParseLine(NULL, &toExecute, h, FALSE)) + if (!WCMD_ReadAndParseLine(NULL, &toExecute, h)) break; WCMD_process_commands(toExecute, FALSE, NULL, NULL); WCMD_free_commands(toExecute); @@ -186,7 +186,7 @@ WCHAR *WCMD_parameter (WCHAR *s, int n, WCHAR **where, WCHAR **end) { * NULL on error or EOF */ -WCHAR *WCMD_fgets(WCHAR *buf, int noChars, HANDLE h, const BOOL is_console_handle) +WCHAR *WCMD_fgets(WCHAR *buf, int noChars, HANDLE h) { DWORD bytes, charsRead; BOOL status; @@ -196,7 +196,7 @@ WCHAR *WCMD_fgets(WCHAR *buf, int noChars, HANDLE h, const BOOL is_console_handl between DOS and Unix. Also need to lose the LF (or CRLF) from the line. */ p = buf; - if (is_console_handle) { + if (WCMD_is_console_handle(h)) { status = ReadConsoleW(h, buf, noChars, &charsRead, NULL); if (!status) return NULL; if (buf[charsRead-2] == '\r') diff --git a/programs/cmd/builtins.c b/programs/cmd/builtins.c index d9eae31..6b4d0ab 100644 --- a/programs/cmd/builtins.c +++ b/programs/cmd/builtins.c @@ -1133,7 +1133,7 @@ void WCMD_for (WCHAR *p, CMD_LIST **cmdList) { WCHAR buffer[MAXSTRING] = {'\0'}; WCHAR *where, *parm; - while (WCMD_fgets(buffer, sizeof(buffer)/sizeof(WCHAR), input, FALSE)) { + while (WCMD_fgets(buffer, sizeof(buffer)/sizeof(WCHAR), input)) { /* Skip blank lines*/ parm = WCMD_parameter (buffer, 0, &where, NULL); @@ -1393,7 +1393,7 @@ void WCMD_goto (CMD_LIST **cmdList) { if (*paramStart == ':') paramStart++; SetFilePointer (context -> h, 0, NULL, FILE_BEGIN); - while (WCMD_fgets (string, sizeof(string)/sizeof(WCHAR), context -> h, FALSE)) { + while (WCMD_fgets (string, sizeof(string)/sizeof(WCHAR), context -> h)) { str = string; while (isspaceW (*str)) str++; if (*str == ':') { diff --git a/programs/cmd/wcmd.h b/programs/cmd/wcmd.h index af75a5d..61cef8f 100644 --- a/programs/cmd/wcmd.h +++ b/programs/cmd/wcmd.h @@ -97,7 +97,11 @@ void WCMD_verify (const WCHAR *command); void WCMD_version (void); int WCMD_volume (BOOL set_label, const WCHAR *command); -WCHAR *WCMD_fgets (WCHAR *buf, int n, HANDLE stream, const BOOL is_console_handle); +static inline BOOL WCMD_is_console_handle(HANDLE h) +{ + return (((DWORD_PTR)h) & 3) == 3; +} +WCHAR *WCMD_fgets (WCHAR *buf, int n, HANDLE stream); WCHAR *WCMD_parameter (WCHAR *s, int n, WCHAR **where, WCHAR **end); WCHAR *WCMD_skip_leading_spaces (WCHAR *string); BOOL WCMD_keyword_ws_found(const WCHAR *keyword, int len, const WCHAR *ptr); @@ -110,8 +114,7 @@ WCHAR *WCMD_strdupW(const WCHAR *input); void WCMD_strsubstW(WCHAR *start, const WCHAR* next, const WCHAR* insert, int len); BOOL WCMD_ReadFile(const HANDLE hIn, WCHAR *intoBuf, const DWORD maxChars, LPDWORD charsRead); -WCHAR *WCMD_ReadAndParseLine(const WCHAR *initialcmd, CMD_LIST **output, - HANDLE readFrom, const BOOL is_console_handle); +WCHAR *WCMD_ReadAndParseLine(const WCHAR *initialcmd, CMD_LIST **output, HANDLE readFrom); CMD_LIST *WCMD_process_commands(CMD_LIST *thisCmd, BOOL oneBracket, const WCHAR *var, const WCHAR *val); void WCMD_free_commands(CMD_LIST *cmds); diff --git a/programs/cmd/wcmdmain.c b/programs/cmd/wcmdmain.c index 5611642..a4726e8 100644 --- a/programs/cmd/wcmdmain.c +++ b/programs/cmd/wcmdmain.c @@ -233,11 +233,6 @@ void WCMD_leave_paged_mode(void) pagedMessage = NULL; } -static inline BOOL is_console_handle(HANDLE h) -{ - return (((DWORD_PTR)h) & 3) == 3; -} - /*************************************************************************** * WCMD_Readfile * @@ -248,7 +243,7 @@ BOOL WCMD_ReadFile(const HANDLE hIn, WCHAR *intoBuf, const DWORD maxChars, LPDWO DWORD numRead; char *buffer; - if (is_console_handle(hIn)) + if (WCMD_is_console_handle(hIn)) /* Try to read from console as Unicode */ return ReadConsoleW(hIn, intoBuf, maxChars, charsRead, NULL); @@ -1781,8 +1776,7 @@ static BOOL WCMD_IsEndQuote(const WCHAR *quote, int quoteIndex) * - Anything else gets put into the command string (including * redirects) */ -WCHAR *WCMD_ReadAndParseLine(const WCHAR *optionalcmd, CMD_LIST **output, - HANDLE readFrom, const BOOL is_console_handle) +WCHAR *WCMD_ReadAndParseLine(const WCHAR *optionalcmd, CMD_LIST **output, HANDLE readFrom) { WCHAR *curPos; int inQuotes = 0; @@ -1827,7 +1821,7 @@ WCHAR *WCMD_ReadAndParseLine(const WCHAR *optionalcmd, CMD_LIST **output, } else if (readFrom == INVALID_HANDLE_VALUE) { WINE_FIXME("No command nor handle supplied\n"); } else { - if (!WCMD_fgets(extraSpace, MAXSTRING, readFrom, is_console_handle)) + if (!WCMD_fgets(extraSpace, MAXSTRING, readFrom)) return NULL; } curPos = extraSpace; @@ -2188,7 +2182,7 @@ WCHAR *WCMD_ReadAndParseLine(const WCHAR *optionalcmd, CMD_LIST **output, /* Read more, skipping any blank lines */ while (*extraSpace == 0x00) { if (!context) WCMD_output_asis( WCMD_LoadMessage(WCMD_MOREPROMPT)); - if (!WCMD_fgets(extraSpace, MAXSTRING, readFrom, is_console_handle)) + if (!WCMD_fgets(extraSpace, MAXSTRING, readFrom)) break; } curPos = extraSpace; @@ -2280,8 +2274,6 @@ int wmain (int argc, WCHAR *argvW[]) WCHAR string[1024]; WCHAR envvar[4]; int opt_q; - BOOL is_console; - DWORD dummy; int opt_t = 0; static const WCHAR promptW[] = {'P','R','O','M','P','T','\0'}; static const WCHAR defaultpromptW[] = {'$','P','$','G','\0'}; @@ -2497,7 +2489,7 @@ int wmain (int argc, WCHAR *argvW[]) */ /* Parse the command string, without reading any more input */ - WCMD_ReadAndParseLine(cmd, &toExecute, INVALID_HANDLE_VALUE, FALSE); + WCMD_ReadAndParseLine(cmd, &toExecute, INVALID_HANDLE_VALUE); WCMD_process_commands(toExecute, FALSE, NULL, NULL); WCMD_free_commands(toExecute); toExecute = NULL; @@ -2593,7 +2585,7 @@ int wmain (int argc, WCHAR *argvW[]) if (opt_k) { /* Parse the command string, without reading any more input */ - WCMD_ReadAndParseLine(cmd, &toExecute, INVALID_HANDLE_VALUE, FALSE); + WCMD_ReadAndParseLine(cmd, &toExecute, INVALID_HANDLE_VALUE); WCMD_process_commands(toExecute, FALSE, NULL, NULL); WCMD_free_commands(toExecute); toExecute = NULL; @@ -2606,13 +2598,12 @@ int wmain (int argc, WCHAR *argvW[]) SetEnvironmentVariableW(promptW, defaultpromptW); WCMD_version (); - is_console = !!GetConsoleMode(GetStdHandle(STD_INPUT_HANDLE), &dummy); while (TRUE) { /* Read until EOF (which for std input is never, but if redirect in place, may occur */ if (echo_mode) WCMD_show_prompt(); - if (!WCMD_ReadAndParseLine(NULL, &toExecute, GetStdHandle(STD_INPUT_HANDLE), is_console)) + if (!WCMD_ReadAndParseLine(NULL, &toExecute, GetStdHandle(STD_INPUT_HANDLE))) break; WCMD_process_commands(toExecute, FALSE, NULL, NULL); WCMD_free_commands(toExecute);

1 0

Nikolay Sivov : oleaut32/typelib: Use BSTR for filename.
by Alexandre Julliard Oct. 6, 2011

Oct. 6, 2011

Module: wine Branch: master Commit: 8899c2e59aecf824d8b3fbbdc0a5464e2485508b URL: http://source.winehq.org/git/wine.git/?a=commit;h=8899c2e59aecf824d8b3fbbdc… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Oct 6 13:55:47 2011 -0500 oleaut32/typelib: Use BSTR for filename. --- dlls/oleaut32/typelib2.c | 11 +++++------ 1 files changed, 5 insertions(+), 6 deletions(-) diff --git a/dlls/oleaut32/typelib2.c b/dlls/oleaut32/typelib2.c index f3f424a..d761491 100644 --- a/dlls/oleaut32/typelib2.c +++ b/dlls/oleaut32/typelib2.c @@ -167,7 +167,7 @@ typedef struct tagICreateTypeLib2Impl ITypeLib2 ITypeLib2_iface; LONG ref; - WCHAR *filename; + BSTR filename; MSFT_Header typelib_header; INT helpStringDll; @@ -4329,7 +4329,7 @@ static ULONG WINAPI ICreateTypeLib2_fnRelease(ICreateTypeLib2 *iface) This->typelib_segment_data[i] = NULL; } - heap_free(This->filename); + SysFreeString(This->filename); This->filename = NULL; while (This->typeinfos) { @@ -5216,22 +5216,21 @@ static const ITypeLib2Vtbl typelib2vt = ITypeLib2_fnGetAllCustData, }; -static ICreateTypeLib2 *ICreateTypeLib2_Constructor(SYSKIND syskind, LPCOLESTR szFile) +static ICreateTypeLib2 *ICreateTypeLib2_Constructor(SYSKIND syskind, LPCOLESTR filename) { ICreateTypeLib2Impl *create_tlib2; int failed = 0; - TRACE("Constructing ICreateTypeLib2 (%d, %s)\n", syskind, debugstr_w(szFile)); + TRACE("Constructing ICreateTypeLib2 (%d, %s)\n", syskind, debugstr_w(filename)); create_tlib2 = heap_alloc_zero(sizeof(ICreateTypeLib2Impl)); if (!create_tlib2) return NULL; - create_tlib2->filename = heap_alloc((strlenW(szFile) + 1) * sizeof(WCHAR)); + create_tlib2->filename = SysAllocString(filename); if (!create_tlib2->filename) { heap_free(create_tlib2); return NULL; } - strcpyW(create_tlib2->filename, szFile); ctl2_init_header(create_tlib2); ctl2_init_segdir(create_tlib2);

1 0

Huw Davies : gdi32: Keep track of the english family name if there' s a localised name as well.
by Alexandre Julliard Oct. 6, 2011

Oct. 6, 2011

Module: wine Branch: master Commit: ab1373d84710350d73131c993afdc08acbab5bdb URL: http://source.winehq.org/git/wine.git/?a=commit;h=ab1373d84710350d73131c993… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Oct 6 16:26:04 2011 -0500 gdi32: Keep track of the english family name if there's a localised name as well. --- dlls/gdi32/freetype.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c index 81c704b..ae8ca41 100644 --- a/dlls/gdi32/freetype.c +++ b/dlls/gdi32/freetype.c @@ -282,6 +282,7 @@ typedef struct tagFace { typedef struct tagFamily { struct list entry; const WCHAR *FamilyName; + const WCHAR *EnglishName; struct list faces; } Family; @@ -1378,6 +1379,7 @@ static INT AddFontToList(const char *file, void *font_data_ptr, DWORD font_data_ if(!family) { family = HeapAlloc(GetProcessHeap(), 0, sizeof(*family)); family->FamilyName = strdupW(localised_family ? localised_family : english_family); + family->EnglishName = localised_family ? strdupW(english_family) : NULL; list_init(&family->faces); list_add_tail(&font_list, &family->entry);

1 0

Huw Davies : gdi32: Cache the loaded font list properties.
by Alexandre Julliard Oct. 6, 2011

Oct. 6, 2011

Module: wine Branch: master Commit: a155292f98f62c10eaf3b9328cbd8c085b8818e9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a155292f98f62c10eaf3b9328… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Oct 6 16:26:06 2011 -0500 gdi32: Cache the loaded font list properties. --- dlls/gdi32/freetype.c | 129 ++++++++++++++++++++++++++++++++++++++++++++++--- 1 files changed, 122 insertions(+), 7 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=a155292f98f62c10eaf3b…

1 0

Nikolay Sivov : msxml3: Use a helper to get property values.
by Alexandre Julliard Oct. 6, 2011

Oct. 6, 2011

Module: wine Branch: master Commit: 35b59556064e773ee3e51c7b0cdb98caf2970c86 URL: http://source.winehq.org/git/wine.git/?a=commit;h=35b59556064e773ee3e51c7b0… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Oct 6 10:35:07 2011 -0500 msxml3: Use a helper to get property values. --- dlls/msxml3/mxwriter.c | 42 ++++++++++++------------------------------ 1 files changed, 12 insertions(+), 30 deletions(-) diff --git a/dlls/msxml3/mxwriter.c b/dlls/msxml3/mxwriter.c index 0214d80..cc75d1d 100644 --- a/dlls/msxml3/mxwriter.c +++ b/dlls/msxml3/mxwriter.c @@ -186,6 +186,13 @@ static HRESULT writer_set_property(mxwriter *writer, MXWRITER_PROPS property, VA return S_OK; } +static HRESULT writer_get_property(const mxwriter *writer, MXWRITER_PROPS property, VARIANT_BOOL *value) +{ + if (!value) return E_POINTER; + *value = writer->props[property]; + return S_OK; +} + static inline mxwriter *impl_from_IMXWriter(IMXWriter *iface) { return CONTAINING_RECORD(iface, mxwriter, IMXWriter_iface); @@ -469,12 +476,7 @@ static HRESULT WINAPI mxwriter_get_byteOrderMark(IMXWriter *iface, VARIANT_BOOL mxwriter *This = impl_from_IMXWriter( iface ); TRACE("(%p)->(%p)\n", This, value); - - if (!value) return E_POINTER; - - *value = This->props[MXWriter_BOM]; - - return S_OK; + return writer_get_property(This, MXWriter_BOM, value); } static HRESULT WINAPI mxwriter_put_indent(IMXWriter *iface, VARIANT_BOOL value) @@ -490,12 +492,7 @@ static HRESULT WINAPI mxwriter_get_indent(IMXWriter *iface, VARIANT_BOOL *value) mxwriter *This = impl_from_IMXWriter( iface ); TRACE("(%p)->(%p)\n", This, value); - - if (!value) return E_POINTER; - - *value = This->props[MXWriter_Indent]; - - return S_OK; + return writer_get_property(This, MXWriter_Indent, value); } static HRESULT WINAPI mxwriter_put_standalone(IMXWriter *iface, VARIANT_BOOL value) @@ -511,12 +508,7 @@ static HRESULT WINAPI mxwriter_get_standalone(IMXWriter *iface, VARIANT_BOOL *va mxwriter *This = impl_from_IMXWriter( iface ); TRACE("(%p)->(%p)\n", This, value); - - if (!value) return E_POINTER; - - *value = This->props[MXWriter_Standalone]; - - return S_OK; + return writer_get_property(This, MXWriter_Standalone, value); } static HRESULT WINAPI mxwriter_put_omitXMLDeclaration(IMXWriter *iface, VARIANT_BOOL value) @@ -532,12 +524,7 @@ static HRESULT WINAPI mxwriter_get_omitXMLDeclaration(IMXWriter *iface, VARIANT_ mxwriter *This = impl_from_IMXWriter( iface ); TRACE("(%p)->(%p)\n", This, value); - - if (!value) return E_POINTER; - - *value = This->props[MXWriter_OmitXmlDecl]; - - return S_OK; + return writer_get_property(This, MXWriter_OmitXmlDecl, value); } static HRESULT WINAPI mxwriter_put_version(IMXWriter *iface, BSTR version) @@ -578,12 +565,7 @@ static HRESULT WINAPI mxwriter_get_disableOutputEscaping(IMXWriter *iface, VARIA mxwriter *This = impl_from_IMXWriter( iface ); TRACE("(%p)->(%p)\n", This, value); - - if (!value) return E_POINTER; - - *value = This->props[MXWriter_DisableEscaping]; - - return S_OK; + return writer_get_property(This, MXWriter_DisableEscaping, value); } static HRESULT WINAPI mxwriter_flush(IMXWriter *iface)

1 0

Andrew Eikum : dsound: Fix fragment position calculations.
by Alexandre Julliard Oct. 6, 2011

Oct. 6, 2011

Module: wine Branch: master Commit: ecd13dcbef16ccee9ae7785cc60ec63f5772ae19 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ecd13dcbef16ccee9ae7785cc… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Thu Oct 6 09:35:07 2011 -0500 dsound: Fix fragment position calculations. --- dlls/dsound/mixer.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/dsound/mixer.c b/dlls/dsound/mixer.c index 936ea3e..8adbdff 100644 --- a/dlls/dsound/mixer.c +++ b/dlls/dsound/mixer.c @@ -788,7 +788,7 @@ static void DSOUND_WaveQueue(DirectSoundDevice *device, BOOL force) device->pwqueue += prebuf_frags; prebuf_frames = ((prebuf_frags + wave_fragpos > device->helfrags) ? - (prebuf_frags + wave_fragpos - device->helfrags) : + (device->helfrags - wave_fragpos) : (prebuf_frags)) * device->fraglen / device->pwfx->nBlockAlign; hr = IAudioRenderClient_GetBuffer(device->render, prebuf_frames, &buffer); @@ -859,8 +859,8 @@ static void DSOUND_PerformMix(DirectSoundDevice *device) return; } - pos_bytes = (clock_pos / (double)clock_freq) * device->pwfx->nSamplesPerSec * - device->pwfx->nBlockAlign; + pos_bytes = ceil(clock_pos * device->pwfx->nBlockAlign * + (clock_freq / (double)device->pwfx->nSamplesPerSec)); delta_frags = (pos_bytes - device->last_pos_bytes) / device->fraglen; if(delta_frags > 0){

1 0

Aric Stewart : usp10: Correct ScriptStringGetLogicalWidths to be based on char not glyph.
by Alexandre Julliard Oct. 6, 2011

Oct. 6, 2011

Module: wine Branch: master Commit: 63490ce03837b467e33457469ab55a26f9b4e120 URL: http://source.winehq.org/git/wine.git/?a=commit;h=63490ce03837b467e33457469… Author: Aric Stewart <aric(a)codeweavers.com> Date: Thu Oct 6 08:35:59 2011 -0500 usp10: Correct ScriptStringGetLogicalWidths to be based on char not glyph. --- dlls/usp10/usp10.c | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/usp10/usp10.c b/dlls/usp10/usp10.c index b8eee22..14f523f 100644 --- a/dlls/usp10/usp10.c +++ b/dlls/usp10/usp10.c @@ -2287,9 +2287,12 @@ HRESULT WINAPI ScriptStringGetLogicalWidths(SCRIPT_STRING_ANALYSIS ssa, int *piD for (i = 0; i < analysis->numItems; i++) { - for (j = 0; j < analysis->glyphs[i].numGlyphs; j++) + int cChar = analysis->pItem[i+1].iCharPos - analysis->pItem[i].iCharPos; + for (j = 0; j < cChar; j++) { - piDx[next] = analysis->glyphs[i].piAdvance[j]; + + int glyph = analysis->glyphs[i].pwLogClust[j]; + piDx[next] = analysis->glyphs[i].piAdvance[glyph]; next++; } }

1 0

Nikolay Sivov : advapi32/tests: Add one more QueryServiceStatusEx() test.
by Alexandre Julliard Oct. 6, 2011

Oct. 6, 2011

Module: wine Branch: master Commit: cd919f60cf11c26a37f45c749e6341e3ce6cee05 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cd919f60cf11c26a37f45c749… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Wed Oct 5 17:53:41 2011 -0500 advapi32/tests: Add one more QueryServiceStatusEx() test. --- dlls/advapi32/tests/service.c | 8 ++++++++ 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/dlls/advapi32/tests/service.c b/dlls/advapi32/tests/service.c index d594416..5bb40eb 100644 --- a/dlls/advapi32/tests/service.c +++ b/dlls/advapi32/tests/service.c @@ -1032,6 +1032,14 @@ static void test_query_svc(void) else ok(statusproc->dwProcessId == 0, "Expect no process id for this stopped service\n"); + + /* same call with null needed pointer */ + SetLastError(0xdeadbeef); + ret = pQueryServiceStatusEx(svc_handle, SC_STATUS_PROCESS_INFO, (BYTE*)statusproc, bufsize, NULL); + ok(!ret, "Expected failure\n"); + ok(broken(GetLastError() == ERROR_INVALID_PARAMETER) /* NT4 */ || + GetLastError() == ERROR_INVALID_ADDRESS, "got %d\n", GetLastError()); + HeapFree(GetProcessHeap(), 0, statusproc); CloseServiceHandle(svc_handle);

1 0