wine-commits June 2012

wine-commits@winehq.org

3 participants
827 discussions

Roderick Colenbrander : opengl32: Disable wglGetProcAddress for core GL 1.0 /1.1 functions.
by Alexandre Julliard 26 Jun '12

26 Jun '12

Module: wine Branch: master Commit: 71faef0f503c248c4d9c94c2be2fcbc3c9c93741 URL: http://source.winehq.org/git/wine.git/?a=commit;h=71faef0f503c248c4d9c94c2b… Author: Roderick Colenbrander <thunderbird2k(a)gmail.com> Date: Mon Jun 25 22:37:30 2012 -0700 opengl32: Disable wglGetProcAddress for core GL 1.0/1.1 functions. --- dlls/opengl32/tests/opengl.c | 10 ++++++++++ dlls/opengl32/wgl.c | 8 +------- 2 files changed, 11 insertions(+), 7 deletions(-) diff --git a/dlls/opengl32/tests/opengl.c b/dlls/opengl32/tests/opengl.c index c18b0ad..a2aa358 100644 --- a/dlls/opengl32/tests/opengl.c +++ b/dlls/opengl32/tests/opengl.c @@ -779,6 +779,16 @@ static void test_getprocaddress(HDC hdc) return; } + /* Core GL 1.0/1.1 functions should not be loadable through wglGetProcAddress. + * Try to load the function with and without a context. + */ + func = wglGetProcAddress("glEnable"); + ok(func == NULL, "Lookup of function glEnable with a context passed, expected a failure\n"); + wglMakeCurrent(hdc, NULL); + func = wglGetProcAddress("glEnable"); + ok(func == NULL, "Lookup of function glEnable without a context passed, expected a failure\n"); + wglMakeCurrent(hdc, ctx); + /* The goal of the test will be to test behavior of wglGetProcAddress when * no WGL context is active. Before the test we pick an extension (GL_ARB_multitexture) * which any GL >=1.2.1 implementation supports. Unfortunately the GDI renderer doesn't diff --git a/dlls/opengl32/wgl.c b/dlls/opengl32/wgl.c index 79aefcd..eca42e2 100644 --- a/dlls/opengl32/wgl.c +++ b/dlls/opengl32/wgl.c @@ -467,13 +467,7 @@ PROC WINAPI wglGetProcAddress(LPCSTR lpszProc) { return NULL; } - /* First, look if it's not already defined in the 'standard' OpenGL functions */ - if ((local_func = GetProcAddress(opengl32_handle, lpszProc)) != NULL) { - TRACE(" found function in 'standard' OpenGL functions (%p)\n", local_func); - return local_func; - } - - /* After that, search in the thunks to find the real name of the extension */ + /* Search in the thunks to find the real name of the extension */ ext.name = lpszProc; ext_ret = bsearch(&ext, extension_registry, extension_registry_size, sizeof(OpenGL_extension), compar);

1 0

Chris Robinson : quartz: Remove the seek position table from the MPEG splitter.
by Alexandre Julliard 26 Jun '12

26 Jun '12

Module: wine Branch: master Commit: 2ccc5bd2c7b90009a967ba63ab3babc6e6d92386 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2ccc5bd2c7b90009a967ba63a… Author: Chris Robinson <chris.kcat(a)gmail.com> Date: Mon Jun 25 19:41:13 2012 -0700 quartz: Remove the seek position table from the MPEG splitter. It takes a good 0.5 seconds for MPEGSplitter_pre_connect to iterate over the file and build the table for a standard 2 or 3 minute MP3. This causes unsightly hiccups when an app tries to play a new file, particularly in games, as the pre-connect needs to happen synchronously to tell whether it can succeed or not. --- dlls/quartz/mpegsplit.c | 109 ++++++++--------------------------------------- 1 files changed, 18 insertions(+), 91 deletions(-) diff --git a/dlls/quartz/mpegsplit.c b/dlls/quartz/mpegsplit.c index 934238e..de03d17 100644 --- a/dlls/quartz/mpegsplit.c +++ b/dlls/quartz/mpegsplit.c @@ -52,28 +52,16 @@ WINE_DEFAULT_DEBUG_CHANNEL(quartz); #define MPEG_AUDIO_HEADER 1 #define MPEG_NO_HEADER 0 -#define SEEK_INTERVAL (ULONGLONG)(10 * 10000000) /* Add an entry every 10 seconds */ - -struct seek_entry { - ULONGLONG bytepos; - ULONGLONG timepos; -}; - typedef struct MPEGSplitterImpl { ParserImpl Parser; LONGLONG EndOfFile; - LONGLONG duration; LONGLONG position; DWORD begin_offset; BYTE header[4]; /* Whether we just seeked (or started playing) */ BOOL seek; - - /* Seeking cache */ - ULONG seek_entries; - struct seek_entry *seektable; } MPEGSplitterImpl; static inline MPEGSplitterImpl *impl_from_IMediaSeeking( IMediaSeeking *iface ) @@ -545,22 +533,17 @@ static HRESULT MPEGSplitter_pre_connect(IPin *iface, IPin *pConnectPin, ALLOCATO This->begin_offset = pos; memcpy(This->header, header, 4); - This->seektable[0].bytepos = pos; - This->seektable[0].timepos = 0; - switch(streamtype) { case MPEG_AUDIO_HEADER: { LONGLONG duration = 0; - DWORD last_entry = 0; - - DWORD ticks = GetTickCount(); + WAVEFORMATEX *format; hr = MPEGSplitter_init_audio(This, header, &piOutput, &amt); if (SUCCEEDED(hr)) { - WAVEFORMATEX *format = (WAVEFORMATEX*)amt.pbFormat; + format = (WAVEFORMATEX*)amt.pbFormat; props->cbAlign = 1; props->cbPrefix = 0; @@ -580,58 +563,16 @@ static HRESULT MPEGSplitter_pre_connect(IPin *iface, IPin *pConnectPin, ALLOCATO } /* Check for idv1 tag, and remove it from stream if found */ - hr = IAsyncReader_SyncRead(pPin->pReader, This->EndOfFile-128, 3, header+4); + hr = IAsyncReader_SyncRead(pPin->pReader, This->EndOfFile-128, 3, header); if (FAILED(hr)) break; - if (!strncmp((char*)header+4, "TAG", 3)) + if (!strncmp((char*)header, "TAG", 3)) This->EndOfFile -= 128; This->Parser.pInputPin->rtStop = MEDIATIME_FROM_BYTES(This->EndOfFile); This->Parser.pInputPin->rtStart = This->Parser.pInputPin->rtCurrent = MEDIATIME_FROM_BYTES(This->begin_offset); - /* http://mpgedit.org/mpgedit/mpeg_format/mpeghdr.htm has a whole read up on audio headers */ - while (pos + 3 < This->EndOfFile) - { - LONGLONG length = 0; - hr = IAsyncReader_SyncRead(pPin->pReader, pos, 4, header); - if (hr != S_OK) - break; - while ((hr=parse_header(header, &length, &duration)) != S_OK && - pos + 4 < This->EndOfFile) - { - /* No valid header yet; shift by a byte and check again */ - memmove(header, header+1, 3); - hr = IAsyncReader_SyncRead(pPin->pReader, ++pos + 3, 1, header + 3); - if (hr != S_OK) - break; - } - if (hr != S_OK) - break; - pos += length; - - if (This->seektable && (duration / SEEK_INTERVAL) > last_entry) - { - if (last_entry + 1 > duration / SEEK_INTERVAL) - { - ERR("Somehow skipped %d interval lengths instead of 1\n", (DWORD)(duration/SEEK_INTERVAL) - (last_entry + 1)); - } - ++last_entry; - - TRACE("Entry: %u\n", last_entry); - if (last_entry >= This->seek_entries) - { - This->seek_entries += 64; - This->seektable = CoTaskMemRealloc(This->seektable, (This->seek_entries)*sizeof(struct seek_entry)); - } - This->seektable[last_entry].bytepos = pos; - This->seektable[last_entry].timepos = duration; - } - - TRACE("Pos: %x%08x/%x%08x\n", (DWORD)(pos >> 32), (DWORD)pos, (DWORD)(This->EndOfFile>>32), (DWORD)This->EndOfFile); - } - hr = S_OK; + duration = (This->EndOfFile-This->begin_offset) * 10000000 / format->nAvgBytesPerSec; TRACE("Duration: %d seconds\n", (DWORD)(duration / 10000000)); - TRACE("Parsing took %u ms\n", GetTickCount() - ticks); - This->duration = duration; This->Parser.sourceSeeking.llCurrent = 0; This->Parser.sourceSeeking.llDuration = duration; @@ -669,45 +610,39 @@ static HRESULT WINAPI MPEGSplitter_seek(IMediaSeeking *iface) MPEGSplitterImpl *This = impl_from_IMediaSeeking(iface); PullPin *pPin = This->Parser.pInputPin; LONGLONG newpos, timepos, bytepos; - HRESULT hr = S_OK; + HRESULT hr = E_INVALIDARG; BYTE header[4]; newpos = This->Parser.sourceSeeking.llCurrent; - - if (newpos > This->duration) - { - WARN("Requesting position %x%08x beyond end of stream %x%08x\n", (DWORD)(newpos>>32), (DWORD)newpos, (DWORD)(This->duration>>32), (DWORD)This->duration); - return E_INVALIDARG; - } - if (This->position/1000000 == newpos/1000000) { TRACE("Requesting position %x%08x same as current position %x%08x\n", (DWORD)(newpos>>32), (DWORD)newpos, (DWORD)(This->position>>32), (DWORD)This->position); return S_OK; } - /* Position, cached */ - bytepos = This->seektable[newpos / SEEK_INTERVAL].bytepos; - timepos = This->seektable[newpos / SEEK_INTERVAL].timepos; - - hr = IAsyncReader_SyncRead(pPin->pReader, bytepos, 4, header); + bytepos = This->begin_offset; + timepos = 0; + /* http://mpgedit.org/mpgedit/mpeg_format/mpeghdr.htm has a whole read up on audio headers */ while (bytepos + 3 < This->EndOfFile) { + LONGLONG duration = timepos; LONGLONG length = 0; hr = IAsyncReader_SyncRead(pPin->pReader, bytepos, 4, header); - if (hr != S_OK || timepos >= newpos) + if (hr != S_OK) break; - - while (parse_header(header, &length, &timepos) && bytepos + 4 < This->EndOfFile) + while ((hr=parse_header(header, &length, &duration)) != S_OK && + bytepos + 4 < This->EndOfFile) { /* No valid header yet; shift by a byte and check again */ memmove(header, header+1, 3); hr = IAsyncReader_SyncRead(pPin->pReader, ++bytepos + 3, 1, header + 3); if (hr != S_OK) break; - } - bytepos += length; - TRACE("Pos: %x%08x/%x%08x\n", (DWORD)(bytepos >> 32), (DWORD)bytepos, (DWORD)(This->EndOfFile>>32), (DWORD)This->EndOfFile); + } + if (hr != S_OK || duration > newpos) + break; + bytepos += length; + timepos = duration; } if (SUCCEEDED(hr)) @@ -832,14 +767,6 @@ HRESULT MPEGSplitter_create(IUnknown * pUnkOuter, LPVOID * ppv) return E_OUTOFMEMORY; ZeroMemory(This, sizeof(MPEGSplitterImpl)); - This->seektable = CoTaskMemAlloc(sizeof(struct seek_entry) * 64); - if (!This->seektable) - { - CoTaskMemFree(This); - return E_OUTOFMEMORY; - } - This->seek_entries = 64; - hr = Parser_Create(&(This->Parser), &MPEGSplitter_Vtbl, &CLSID_MPEG1Splitter, MPEGSplitter_process_sample, MPEGSplitter_query_accept, MPEGSplitter_pre_connect, MPEGSplitter_cleanup, MPEGSplitter_disconnect, MPEGSplitter_first_request, NULL, NULL, MPEGSplitter_seek, NULL); if (FAILED(hr)) {

1 0

Michael Stefaniuc : usp10: Remove unreachable break after return (Smatch).
by Alexandre Julliard 26 Jun '12

26 Jun '12

Module: wine Branch: master Commit: f596c747ef6dda58bb69bcdce335de5bcc7206f1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f596c747ef6dda58bb69bcdce… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Jun 25 23:11:51 2012 +0200 usp10: Remove unreachable break after return (Smatch). --- dlls/usp10/shape.c | 114 ++++++++++++++++++++++++++-------------------------- 1 files changed, 57 insertions(+), 57 deletions(-) diff --git a/dlls/usp10/shape.c b/dlls/usp10/shape.c index a99614e..b574a85 100644 --- a/dlls/usp10/shape.c +++ b/dlls/usp10/shape.c @@ -898,11 +898,11 @@ static int combining_lexical_Arabic(WCHAR c) case 0x0652: case 0x0657: case 0x0658: - case 0x06E1: return Arab_DIAC1; break; + case 0x06E1: return Arab_DIAC1; case 0x064D: case 0x0650: - case 0x0656: return Arab_DIAC2; break; - case 0x0651: return Arab_DIAC3; break; + case 0x0656: return Arab_DIAC2; + case 0x0651: return Arab_DIAC3; case 0x0610: case 0x0611: case 0x0612: @@ -918,14 +918,14 @@ static int combining_lexical_Arabic(WCHAR c) case 0x06E7: case 0x06E8: case 0x06EB: - case 0x06EC: return Arab_DIAC4; break; + case 0x06EC: return Arab_DIAC4; case 0x06E3: case 0x06EA: - case 0x06ED: return Arab_DIAC5; break; - case 0x0670: return Arab_DIAC6; break; - case 0x0653: return Arab_DIAC7; break; + case 0x06ED: return Arab_DIAC5; + case 0x0670: return Arab_DIAC6; + case 0x0653: return Arab_DIAC7; case 0x0655: - case 0x0654: return Arab_DIAC8; break; + case 0x0654: return Arab_DIAC8; default: return Arab_Norm; } } @@ -1037,46 +1037,46 @@ static int combining_lexical_Hebrew(WCHAR c) case 0x05B4: case 0x05B5: case 0x05B6: - case 0x05BB: return Hebr_DIAC; break; + case 0x05BB: return Hebr_DIAC; case 0x0599: case 0x05A1: case 0x05A9: - case 0x05AE: return Hebr_CANT1; break; + case 0x05AE: return Hebr_CANT1; case 0x0597: case 0x05A8: - case 0x05AC: return Hebr_CANT2; break; + case 0x05AC: return Hebr_CANT2; case 0x0592: case 0x0593: case 0x0594: case 0x0595: case 0x05A7: - case 0x05AB: return Hebr_CANT3; break; + case 0x05AB: return Hebr_CANT3; case 0x0598: case 0x059C: case 0x059E: - case 0x059F: return Hebr_CANT4; break; + case 0x059F: return Hebr_CANT4; case 0x059D: - case 0x05A0: return Hebr_CANT5; break; + case 0x05A0: return Hebr_CANT5; case 0x059B: - case 0x05A5: return Hebr_CANT6; break; + case 0x05A5: return Hebr_CANT6; case 0x0591: case 0x05A3: - case 0x05A6: return Hebr_CANT7; break; + case 0x05A6: return Hebr_CANT7; case 0x0596: case 0x05A4: - case 0x05AA: return Hebr_CANT8; break; + case 0x05AA: return Hebr_CANT8; case 0x059A: - case 0x05AD: return Hebr_CANT9; break; - case 0x05AF: return Hebr_CANT10; break; - case 0x05BC: return Hebr_DAGESH; break; - case 0x05C4: return Hebr_DOTABV; break; - case 0x05B9: return Hebr_HOLAM; break; - case 0x05BD: return Hebr_METEG; break; - case 0x05B7: return Hebr_PATAH; break; - case 0x05B8: return Hebr_QAMATS; break; - case 0x05BF: return Hebr_RAFE; break; + case 0x05AD: return Hebr_CANT9; + case 0x05AF: return Hebr_CANT10; + case 0x05BC: return Hebr_DAGESH; + case 0x05C4: return Hebr_DOTABV; + case 0x05B9: return Hebr_HOLAM; + case 0x05BD: return Hebr_METEG; + case 0x05B7: return Hebr_PATAH; + case 0x05B8: return Hebr_QAMATS; + case 0x05BF: return Hebr_RAFE; case 0x05C1: - case 0x05C2: return Hebr_SHINSIN; break; + case 0x05C2: return Hebr_SHINSIN; default: return Hebr_Norm; } } @@ -1113,40 +1113,40 @@ static int combining_lexical_Syriac(WCHAR c) case 0x733: case 0x736: case 0x73A: - case 0x73D: return Syriac_DIAC1; break; + case 0x73D: return Syriac_DIAC1; case 0x731: case 0x734: case 0x737: case 0x73B: - case 0x73E: return Syriac_DIAC2; break; + case 0x73E: return Syriac_DIAC2; case 0x740: case 0x749: - case 0x74A: return Syriac_DIAC3; break; + case 0x74A: return Syriac_DIAC3; case 0x732: case 0x735: - case 0x73F: return Syriac_DIAC4; break; + case 0x73F: return Syriac_DIAC4; case 0x738: case 0x739: - case 0x73C: return Syriac_DIAC5; break; + case 0x73C: return Syriac_DIAC5; case 0x741: - case 0x30A: return Syriac_DIAC6; break; + case 0x30A: return Syriac_DIAC6; case 0x742: - case 0x325: return Syriac_DIAC7; break; + case 0x325: return Syriac_DIAC7; case 0x747: - case 0x303: return Syriac_DIAC8; break; + case 0x303: return Syriac_DIAC8; case 0x748: case 0x32D: case 0x32E: case 0x330: - case 0x331: return Syriac_DIAC9; break; - case 0x308: return Syriac_DIAC10; break; - case 0x304: return Syriac_DIAC11; break; - case 0x307: return Syriac_DIAC12; break; - case 0x323: return Syriac_DIAC13; break; - case 0x743: return Syriac_DIAC14; break; - case 0x744: return Syriac_DIAC15; break; - case 0x745: return Syriac_DIAC16; break; - case 0x746: return Syriac_DIAC17; break; + case 0x331: return Syriac_DIAC9; + case 0x308: return Syriac_DIAC10; + case 0x304: return Syriac_DIAC11; + case 0x307: return Syriac_DIAC12; + case 0x323: return Syriac_DIAC13; + case 0x743: return Syriac_DIAC14; + case 0x744: return Syriac_DIAC15; + case 0x745: return Syriac_DIAC16; + case 0x746: return Syriac_DIAC17; default: return Syriac_Norm; } } @@ -1260,7 +1260,7 @@ static int combining_lexical_Thaana(WCHAR c) case 0x7AC: case 0x7AD: case 0x7AE: - case 0x7AF: return Thaana_FILI; break; + case 0x7AF: return Thaana_FILI; default: return Thaana_Norm; } } @@ -1378,19 +1378,19 @@ static int combining_lexical_Thai(WCHAR c) case 0xE34: case 0xE35: case 0xE36: - case 0xE37: return Thai_ABOVE1; break; + case 0xE37: return Thai_ABOVE1; case 0xE47: - case 0xE4D: return Thai_ABOVE2; break; + case 0xE4D: return Thai_ABOVE2; case 0xE48: case 0xE49: case 0xE4A: - case 0xE4B: return Thai_ABOVE3; break; + case 0xE4B: return Thai_ABOVE3; case 0xE4C: - case 0xE4E: return Thai_ABOVE4; break; + case 0xE4E: return Thai_ABOVE4; case 0xE38: - case 0xE39: return Thai_BELOW1; break; - case 0xE3A: return Thai_BELOW2; break; - case 0xE33: return Thai_AM; break; + case 0xE39: return Thai_BELOW1; + case 0xE3A: return Thai_BELOW2; + case 0xE33: return Thai_AM; default: return Thai_Norm; } } @@ -1425,16 +1425,16 @@ static int combining_lexical_Lao(WCHAR c) case 0xEB6: case 0xEB7: case 0xEBB: - case 0xECD: return Lao_ABOVE1; break; + case 0xECD: return Lao_ABOVE1; case 0xEC8: case 0xEC9: case 0xECA: case 0xECB: - case 0xECC: return Lao_ABOVE2; break; - case 0xEBC: return Lao_BELOW1; break; + case 0xECC: return Lao_ABOVE2; + case 0xEBC: return Lao_BELOW1; case 0xEB8: - case 0xEB9: return Lao_BELOW2; break; - case 0xEB3: return Lao_AM; break; + case 0xEB9: return Lao_BELOW2; + case 0xEB3: return Lao_AM; default: return Lao_Norm; } }

1 0

Vincent Povirk : appwiz: Update text to match button names.
by Alexandre Julliard 26 Jun '12

26 Jun '12

1 0

Vincent Povirk : gdiplus: Remove FIXME from GdipAddPathString.
by Alexandre Julliard 26 Jun '12

26 Jun '12

Module: wine Branch: master Commit: 697dd42ed2c66a9d77065f2c42cece76a69ba2c1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=697dd42ed2c66a9d77065f2c4… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Mon Jun 25 10:40:36 2012 -0500 gdiplus: Remove FIXME from GdipAddPathString. --- dlls/gdiplus/graphicspath.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/graphicspath.c b/dlls/gdiplus/graphicspath.c index 471f915..370bb1f 100644 --- a/dlls/gdiplus/graphicspath.c +++ b/dlls/gdiplus/graphicspath.c @@ -953,7 +953,7 @@ GpStatus WINGDIPAPI GdipAddPathString(GpPath* path, GDIPCONST WCHAR* string, INT RectF scaled_layout_rect; TEXTMETRICW textmetric; - FIXME("(%p, %s, %d, %p, %d, %f, %p, %p): stub\n", path, debugstr_w(string), length, family, style, emSize, layoutRect, format); + TRACE("(%p, %s, %d, %p, %d, %f, %p, %p)\n", path, debugstr_w(string), length, family, style, emSize, layoutRect, format); if (!path || !string || !family || !emSize || !layoutRect || !format) return InvalidParameter;

1 0

Dmitry Timoshkov : windowscodecs: Implement MetadataReaderInfo_GetVendorGUID.
by Alexandre Julliard 26 Jun '12

26 Jun '12

Module: wine Branch: master Commit: 58387fca692f2940f9150ab29d0fffab20c465d9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=58387fca692f2940f9150ab29… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Jun 25 12:54:21 2012 +0900 windowscodecs: Implement MetadataReaderInfo_GetVendorGUID. --- dlls/windowscodecs/info.c | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/windowscodecs/info.c b/dlls/windowscodecs/info.c index 1b650d6..980dfc4 100644 --- a/dlls/windowscodecs/info.c +++ b/dlls/windowscodecs/info.c @@ -1485,8 +1485,11 @@ static HRESULT WINAPI MetadataReaderInfo_GetAuthor(IWICMetadataReaderInfo *iface static HRESULT WINAPI MetadataReaderInfo_GetVendorGUID(IWICMetadataReaderInfo *iface, GUID *vendor) { - FIXME("(%p,%p): stub\n", iface, vendor); - return E_NOTIMPL; + MetadataReaderInfo *This = impl_from_IWICMetadataReaderInfo(iface); + + TRACE("(%p,%p)\n", iface, vendor); + + return ComponentInfo_GetGUIDValue(This->classkey, vendor_valuename, vendor); } static HRESULT WINAPI MetadataReaderInfo_GetVersion(IWICMetadataReaderInfo *iface,

1 0

Dmitry Timoshkov : windowscodecs: Implement TiffFrameDecode_GetThumbnail.
by Alexandre Julliard 26 Jun '12

26 Jun '12

Module: wine Branch: master Commit: 2d3cf48300cbb731d0be9b0f5fbb9fc3df29109a URL: http://source.winehq.org/git/wine.git/?a=commit;h=2d3cf48300cbb731d0be9b0f5… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Jun 25 12:27:22 2012 +0900 windowscodecs: Implement TiffFrameDecode_GetThumbnail. --- dlls/windowscodecs/tiffformat.c | 8 ++++++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/dlls/windowscodecs/tiffformat.c b/dlls/windowscodecs/tiffformat.c index a64d65f..5351098 100644 --- a/dlls/windowscodecs/tiffformat.c +++ b/dlls/windowscodecs/tiffformat.c @@ -1096,8 +1096,12 @@ static HRESULT WINAPI TiffFrameDecode_GetColorContexts(IWICBitmapFrameDecode *if static HRESULT WINAPI TiffFrameDecode_GetThumbnail(IWICBitmapFrameDecode *iface, IWICBitmapSource **ppIThumbnail) { - FIXME("(%p,%p): stub\n", iface, ppIThumbnail); - return E_NOTIMPL; + TRACE("(%p,%p)\n", iface, ppIThumbnail); + + if (!ppIThumbnail) return E_INVALIDARG; + + *ppIThumbnail = NULL; + return WINCODEC_ERR_CODECNOTHUMBNAIL; } static const IWICBitmapFrameDecodeVtbl TiffFrameDecode_Vtbl = {

1 0

Dmitry Timoshkov : windowscodecs: Implement TiffDecoder_GetPreview.
by Alexandre Julliard 26 Jun '12

26 Jun '12

Module: wine Branch: master Commit: a7190c7dc7c82d0ba0d0be9db6774a1f354c9e6a URL: http://source.winehq.org/git/wine.git/?a=commit;h=a7190c7dc7c82d0ba0d0be9db… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Jun 25 12:27:12 2012 +0900 windowscodecs: Implement TiffDecoder_GetPreview. --- dlls/windowscodecs/tiffformat.c | 8 ++++++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/dlls/windowscodecs/tiffformat.c b/dlls/windowscodecs/tiffformat.c index d12729f..a64d65f 100644 --- a/dlls/windowscodecs/tiffformat.c +++ b/dlls/windowscodecs/tiffformat.c @@ -628,8 +628,12 @@ static HRESULT WINAPI TiffDecoder_GetMetadataQueryReader(IWICBitmapDecoder *ifac static HRESULT WINAPI TiffDecoder_GetPreview(IWICBitmapDecoder *iface, IWICBitmapSource **ppIBitmapSource) { - FIXME("(%p,%p): stub\n", iface, ppIBitmapSource); - return E_NOTIMPL; + TRACE("(%p,%p)\n", iface, ppIBitmapSource); + + if (!ppIBitmapSource) return E_INVALIDARG; + + *ppIBitmapSource = NULL; + return WINCODEC_ERR_UNSUPPORTEDOPERATION; } static HRESULT WINAPI TiffDecoder_GetColorContexts(IWICBitmapDecoder *iface,

1 0

Dmitry Timoshkov : windowscodecs: TiffDecoder_GetThumbnail should return NULL interface.
by Alexandre Julliard 26 Jun '12

26 Jun '12

Module: wine Branch: master Commit: f07829da5750949ad9e7a9c5a370c65b0f6076f8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f07829da5750949ad9e7a9c5a… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Jun 25 12:26:56 2012 +0900 windowscodecs: TiffDecoder_GetThumbnail should return NULL interface. --- dlls/windowscodecs/tiffformat.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/dlls/windowscodecs/tiffformat.c b/dlls/windowscodecs/tiffformat.c index cd3703f..d12729f 100644 --- a/dlls/windowscodecs/tiffformat.c +++ b/dlls/windowscodecs/tiffformat.c @@ -643,6 +643,10 @@ static HRESULT WINAPI TiffDecoder_GetThumbnail(IWICBitmapDecoder *iface, IWICBitmapSource **ppIThumbnail) { TRACE("(%p,%p)\n", iface, ppIThumbnail); + + if (!ppIThumbnail) return E_INVALIDARG; + + *ppIThumbnail = NULL; return WINCODEC_ERR_CODECNOTHUMBNAIL; }

1 0

Alexandre Julliard : gdi32: Forward SwapBuffers to opengl32 which in turn calls GdiSwapBuffers.
by Alexandre Julliard 26 Jun '12

26 Jun '12

Module: wine Branch: master Commit: 67e7eda96c16417526e9659be0eeecf2b6edd4c1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=67e7eda96c16417526e9659be… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Jun 25 22:45:28 2012 +0200 gdi32: Forward SwapBuffers to opengl32 which in turn calls GdiSwapBuffers. --- dlls/gdi32/gdi32.spec | 2 +- dlls/gdi32/opengl.c | 15 +++++++++++++++ dlls/gdi32/painting.c | 12 +++--------- dlls/opengl32/wgl.c | 5 +++-- 4 files changed, 22 insertions(+), 12 deletions(-) diff --git a/dlls/gdi32/gdi32.spec b/dlls/gdi32/gdi32.spec index 98fe5c6..2a7edc3 100644 --- a/dlls/gdi32/gdi32.spec +++ b/dlls/gdi32/gdi32.spec @@ -218,7 +218,7 @@ @ stub GdiSetServerAttr # @ stub GdiStartDocEMF # @ stub GdiStartPageEMF -# @ stub GdiSwapBuffers +@ stdcall GdiSwapBuffers(long) @ stdcall GdiTransparentBlt(long long long long long long long long long long long) # @ stub GdiValidateHandle @ stub GdiWinWatchClose diff --git a/dlls/gdi32/opengl.c b/dlls/gdi32/opengl.c index 18b5fcb..d2ab2aa 100644 --- a/dlls/gdi32/opengl.c +++ b/dlls/gdi32/opengl.c @@ -43,6 +43,7 @@ static HMODULE opengl32; static INT (WINAPI *wglChoosePixelFormat)(HDC,const PIXELFORMATDESCRIPTOR *); static INT (WINAPI *wglDescribePixelFormat)(HDC,INT,UINT,PIXELFORMATDESCRIPTOR*); static BOOL (WINAPI *wglSetPixelFormat)(HDC,INT,const PIXELFORMATDESCRIPTOR*); +static BOOL (WINAPI *wglSwapBuffers)(HDC); static HDC default_hdc = 0; @@ -267,3 +268,17 @@ BOOL WINAPI SetPixelFormat( HDC hdc, INT fmt, const PIXELFORMATDESCRIPTOR *pfd ) } return wglSetPixelFormat( hdc, fmt, pfd ); } + +/****************************************************************************** + * SwapBuffers (GDI32.@) + */ +BOOL WINAPI SwapBuffers( HDC hdc ) +{ + if (!wglSwapBuffers) + { + if (!opengl32) opengl32 = LoadLibraryW( opengl32W ); + if (!(wglSwapBuffers = (void *)GetProcAddress( opengl32, "wglSwapBuffers" ))) + return 0; + } + return wglSwapBuffers( hdc ); +} diff --git a/dlls/gdi32/painting.c b/dlls/gdi32/painting.c index 4be8cb3..571a96b 100644 --- a/dlls/gdi32/painting.c +++ b/dlls/gdi32/painting.c @@ -562,17 +562,11 @@ INT WINAPI GdiDescribePixelFormat( HDC hdc, INT iPixelFormat, UINT nBytes, /****************************************************************************** - * SwapBuffers [GDI32.@] - * Exchanges front and back buffers of window + * GdiSwapBuffers [GDI32.@] * - * PARAMS - * hdc [I] Device context whose buffers get swapped - * - * RETURNS - * Success: TRUE - * Failure: FALSE + * Probably not the correct semantics, it's supposed to be an internal backend for SwapBuffers. */ -BOOL WINAPI SwapBuffers( HDC hdc ) +BOOL WINAPI GdiSwapBuffers( HDC hdc ) { INT bRet = FALSE; DC * dc = get_dc_ptr( hdc ); diff --git a/dlls/opengl32/wgl.c b/dlls/opengl32/wgl.c index 328cb63..79aefcd 100644 --- a/dlls/opengl32/wgl.c +++ b/dlls/opengl32/wgl.c @@ -90,6 +90,7 @@ const GLubyte * WINAPI wine_glGetString( GLenum name ); /* internal GDI functions */ extern INT WINAPI GdiDescribePixelFormat( HDC hdc, INT fmt, UINT size, PIXELFORMATDESCRIPTOR *pfd ); extern BOOL WINAPI GdiSetPixelFormat( HDC hdc, INT fmt, const PIXELFORMATDESCRIPTOR *pfd ); +extern BOOL WINAPI GdiSwapBuffers( HDC hdc ); /*********************************************************************** * wglSetPixelFormat(OPENGL32.@) @@ -560,7 +561,7 @@ BOOL WINAPI wglSwapLayerBuffers(HDC hdc, TRACE_(opengl)("(%p, %08x)\n", hdc, fuPlanes); if (fuPlanes & WGL_SWAP_MAIN_PLANE) { - if (!SwapBuffers(hdc)) return FALSE; + if (!GdiSwapBuffers(hdc)) return FALSE; fuPlanes &= ~WGL_SWAP_MAIN_PLANE; } @@ -1062,7 +1063,7 @@ void WINAPI wine_glGetIntegerv( GLenum pname, GLint* params ) */ BOOL WINAPI DECLSPEC_HOTPATCH wglSwapBuffers( HDC hdc ) { - return SwapBuffers(hdc); + return GdiSwapBuffers(hdc); } /* This is for brain-dead applications that use OpenGL functions before even

1 0

← Newer
1
...
16
17
18
19
20
21
22
...
83
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits June 2012