October 2008 - Wine-commits

Damjan Jovanovic : ws2_32: Only enable FD_WRITE on short sends.
by Alexandre Julliard 16 Oct '08

16 Oct '08

Module: wine Branch: stable Commit: 65aa98826db11f5f8d648ecabff134e4995cb9c9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=65aa98826db11f5f8d648ecab… Author: Damjan Jovanovic <damjan.jov(a)gmail.com> Date: Sun Sep 7 12:41:19 2008 +0200 ws2_32: Only enable FD_WRITE on short sends. (cherry picked from commit b904dd783cd2b7368e11882e37b90e7da23b73be) --- dlls/ws2_32/socket.c | 12 +++- dlls/ws2_32/tests/sock.c | 140 +++++++++++++++++++++++++++++++++------------ 2 files changed, 112 insertions(+), 40 deletions(-) diff --git a/dlls/ws2_32/socket.c b/dlls/ws2_32/socket.c index 989af4a..53514e7 100644 --- a/dlls/ws2_32/socket.c +++ b/dlls/ws2_32/socket.c @@ -1250,8 +1250,13 @@ static NTSTATUS WS2_async_send(void* user, IO_STATUS_BLOCK* iosb, NTSTATUS statu if (result >= 0) { + int totalLength = 0; + int i; status = STATUS_SUCCESS; - _enable_event( wsa->hSocket, FD_WRITE, 0, 0 ); + for (i = 0; i < wsa->n_iovecs; i++) + totalLength += wsa->iovec[i].iov_len; + if (result < totalLength) + _enable_event( wsa->hSocket, FD_WRITE, 0, 0 ); } else { @@ -2667,6 +2672,7 @@ INT WINAPI WSASendTo( SOCKET s, LPWSABUF lpBuffers, DWORD dwBufferCount, unsigned int i, options; int n, fd, err; struct iovec iovec[WS_MSG_MAXIOVLEN]; + int totalLength = 0; ULONG_PTR cvalue = (lpOverlapped && ((ULONG_PTR)lpOverlapped->hEvent & 1) == 0) ? (ULONG_PTR)lpOverlapped : 0; TRACE("socket %04lx, wsabuf %p, nbufs %d, flags %d, to %p, tolen %d, ovl %p, func %p\n", @@ -2694,6 +2700,7 @@ INT WINAPI WSASendTo( SOCKET s, LPWSABUF lpBuffers, DWORD dwBufferCount, { iovec[i].iov_base = lpBuffers[i].buf; iovec[i].iov_len = lpBuffers[i].len; + totalLength += lpBuffers[i].len; } for (;;) @@ -2819,7 +2826,8 @@ INT WINAPI WSASendTo( SOCKET s, LPWSABUF lpBuffers, DWORD dwBufferCount, } else /* non-blocking */ { - _enable_event(SOCKET2HANDLE(s), FD_WRITE, 0, 0); + if (n < totalLength) + _enable_event(SOCKET2HANDLE(s), FD_WRITE, 0, 0); if (n == -1) { err = WSAEWOULDBLOCK; diff --git a/dlls/ws2_32/tests/sock.c b/dlls/ws2_32/tests/sock.c index c2c56ae..d497b44 100644 --- a/dlls/ws2_32/tests/sock.c +++ b/dlls/ws2_32/tests/sock.c @@ -1992,9 +1992,23 @@ static DWORD WINAPI drain_socket_thread(LPVOID arg) { char buffer[1024]; SOCKET sock = *(SOCKET*)arg; + int ret; - while (recv(sock, buffer, sizeof(buffer), 0) > 0) - ; + while ((ret = recv(sock, buffer, sizeof(buffer), 0)) != 0) + { + if (ret < 0) + { + if (WSAGetLastError() == WSAEWOULDBLOCK) + { + fd_set readset; + FD_ZERO(&readset); + FD_SET(sock, &readset); + select(0, &readset, NULL, NULL, NULL); + } + else + break; + } + } return 0; } @@ -2050,10 +2064,13 @@ static void test_write_events(void) SOCKET dst = INVALID_SOCKET; HANDLE hThread = NULL; HANDLE hEvent = INVALID_HANDLE_VALUE; - int len; + char *buffer = NULL; + int bufferSize = 1024*1024; u_long one = 1; int ret; DWORD id; + WSANETWORKEVENTS netEvents; + DWORD dwRet; if (tcp_socketpair(&src, &dst) != 0) { @@ -2061,6 +2078,29 @@ static void test_write_events(void) return; } + /* On Windows it seems when a non-blocking socket sends to a + blocking socket on the same host, the send() is BLOCKING, + so make both sockets non-blocking */ + ret = ioctlsocket(src, FIONBIO, &one); + if (ret) + { + ok(0, "ioctlsocket failed, error %d\n", WSAGetLastError()); + goto end; + } + ret = ioctlsocket(dst, FIONBIO, &one); + if (ret) + { + ok(0, "ioctlsocket failed, error %d\n", WSAGetLastError()); + goto end; + } + + buffer = HeapAlloc(GetProcessHeap(), 0, bufferSize); + if (buffer == NULL) + { + ok(0, "could not allocate memory for test\n"); + goto end; + } + hThread = CreateThread(NULL, 0, drain_socket_thread, &dst, 0, &id); if (hThread == NULL) { @@ -2075,55 +2115,79 @@ static void test_write_events(void) goto end; } - ret = ioctlsocket(src, FIONBIO, &one); + ret = WSAEventSelect(src, hEvent, FD_WRITE | FD_CLOSE); if (ret) { - ok(0, "ioctlsocket failed, error %d\n", WSAGetLastError()); + ok(0, "WSAEventSelect failed, error %d\n", ret); goto end; } - ret = WSAEventSelect(src, hEvent, FD_WRITE | FD_CLOSE); + /* FD_WRITE should be set initially, and allow us to send at least 1 byte */ + dwRet = WaitForSingleObject(hEvent, 5000); + if (dwRet != WAIT_OBJECT_0) + { + ok(0, "Initial WaitForSingleObject failed, error %d\n", dwRet); + goto end; + } + ret = WSAEnumNetworkEvents(src, NULL, &netEvents); if (ret) { - ok(0, "WSAEventSelect failed, error %d\n", ret); + ok(0, "WSAEnumNetworkEvents failed, error %d\n", ret); + goto end; + } + if (netEvents.lNetworkEvents & FD_WRITE) + { + ret = send(src, "a", 1, 0); + ok(ret == 1, "sending 1 byte failed, error %d\n", WSAGetLastError()); + if (ret != 1) + goto end; + } + else + { + ok(0, "FD_WRITE not among initial events\n"); goto end; } - for (len = 100; len > 0; --len) + /* Now FD_WRITE should not be set, because the socket send buffer isn't full yet */ + dwRet = WaitForSingleObject(hEvent, 2000); + if (dwRet == WAIT_OBJECT_0) + { + ok(0, "WaitForSingleObject should have timed out, but succeeded!\n"); + goto end; + } + + /* Now if we send a tonne of data, the socket send buffer will only take some of it, + and we will get a short write, which will trigger another FD_WRITE event + as soon as data is sent and more space becomes available, but not any earlier. */ + do + { + ret = send(src, buffer, bufferSize, 0); + } while (ret == bufferSize); + if (ret >= 0 || WSAGetLastError() == WSAEWOULDBLOCK) { - WSANETWORKEVENTS netEvents; - DWORD dwRet = WaitForSingleObject(hEvent, 5000); - if (dwRet != WAIT_OBJECT_0) - { - ok(0, "WaitForSingleObject failed, error %d\n", dwRet); - goto end; - } - - ret = WSAEnumNetworkEvents(src, NULL, &netEvents); - if (ret) - { - ok(0, "WSAEnumNetworkEvents failed, error %d\n", ret); - goto end; - } - - if (netEvents.lNetworkEvents & FD_WRITE) - { - ret = send(src, "a", 1, 0); - if (ret < 0 && WSAGetLastError() != WSAEWOULDBLOCK) - { - ok(0, "send failed, error %d\n", WSAGetLastError()); - goto end; - } - } - - if (netEvents.lNetworkEvents & FD_CLOSE) - { - ok(0, "unexpected close\n"); - goto end; - } + dwRet = WaitForSingleObject(hEvent, 5000); + ok(dwRet == WAIT_OBJECT_0, "Waiting failed with %d\n", dwRet); + if (dwRet == WAIT_OBJECT_0) + { + ret = WSAEnumNetworkEvents(src, NULL, &netEvents); + ok(ret == 0, "WSAEnumNetworkEvents failed, error %d\n", ret); + if (ret == 0) + goto end; + ok(netEvents.lNetworkEvents & FD_WRITE, + "FD_WRITE event not set as expected, events are 0x%x\n", netEvents.lNetworkEvents); + } + else + goto end; + } + else + { + ok(0, "sending a lot of data failed with error %d\n", WSAGetLastError()); + goto end; } end: + if (buffer != NULL) + HeapFree(GetProcessHeap(), 0, buffer); if (src != INVALID_SOCKET) closesocket(src); if (dst != INVALID_SOCKET)

1 0

Alexandre Julliard : ws2_32: Allow arbitrary-sized iovecs again in WSASendTo and WSARecvFrom.
by Alexandre Julliard 16 Oct '08

16 Oct '08

Module: wine Branch: stable Commit: b20836e9097890a350a3a5f8f1c20e8b630c4b6b URL: http://source.winehq.org/git/wine.git/?a=commit;h=b20836e9097890a350a3a5f8f… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Oct 9 12:57:33 2008 +0200 ws2_32: Allow arbitrary-sized iovecs again in WSASendTo and WSARecvFrom. (cherry picked from commit fe7c5d86e7d9e06872c2fb6ddf7ebe8d92525d9c) --- dlls/ws2_32/socket.c | 159 +++++++++++++++++++++---------------------------- 1 files changed, 68 insertions(+), 91 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=b20836e9097890a350a3a…

1 0

Tobias Jakobi : d3d8: Fix IDirect3DDevice8::CreateImageSurface pool type.
by Alexandre Julliard 16 Oct '08

16 Oct '08

Module: wine Branch: stable Commit: b7091494c75794c7cc7f9b776bfd9e391fc1eff9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b7091494c75794c7cc7f9b776… Author: Tobias Jakobi <liquid.acid(a)gmx.net> Date: Thu Oct 16 14:23:39 2008 +0200 d3d8: Fix IDirect3DDevice8::CreateImageSurface pool type. (cherry picked from commit 9ba3d03e7848ec7881ffc3cfdcf01a76d1aa1cc5) --- dlls/d3d8/device.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/dlls/d3d8/device.c b/dlls/d3d8/device.c index cf864cc..476abe9 100644 --- a/dlls/d3d8/device.c +++ b/dlls/d3d8/device.c @@ -697,12 +697,14 @@ static HRESULT WINAPI IDirect3DDevice8Impl_CreateDepthStencilSurface(LPDIRECT3DD return hr; } +/* IDirect3DDevice8Impl::CreateImageSurface returns surface with pool type SYSTEMMEM */ static HRESULT WINAPI IDirect3DDevice8Impl_CreateImageSurface(LPDIRECT3DDEVICE8 iface, UINT Width, UINT Height, D3DFORMAT Format, IDirect3DSurface8** ppSurface) { HRESULT hr; TRACE("Relay\n"); EnterCriticalSection(&d3d8_cs); - hr = IDirect3DDevice8Impl_CreateSurface(iface, Width, Height, Format, TRUE /* Loackable */ , FALSE /*Discard*/ , 0 /* Level */ , ppSurface, D3DRTYPE_SURFACE, 0 /* Usage (undefined/none) */ , D3DPOOL_SCRATCH, D3DMULTISAMPLE_NONE, 0 /* MultisampleQuality */); + hr = IDirect3DDevice8Impl_CreateSurface(iface, Width, Height, Format, TRUE /* Loackable */ , FALSE /*Discard*/ , 0 /* Level */ , ppSurface, + D3DRTYPE_SURFACE, 0 /* Usage (undefined/none) */ , D3DPOOL_SYSTEMMEM, D3DMULTISAMPLE_NONE, 0 /* MultisampleQuality */); LeaveCriticalSection(&d3d8_cs); return hr; }

1 0

Juan Lang : crypt32: Fix length calculation for decoding CRL_DIST_POINT_NAME.
by Alexandre Julliard 16 Oct '08

16 Oct '08

Module: wine Branch: stable Commit: 187630cd262c6cacde80d2b3a7de494ed18912b5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=187630cd262c6cacde80d2b3a… Author: Juan Lang <juan.lang(a)gmail.com> Date: Fri Aug 15 16:19:11 2008 -0700 crypt32: Fix length calculation for decoding CRL_DIST_POINT_NAME. (cherry picked from commit 9423f836ac76b8242a5870dbabe73b8f947252a0) --- dlls/crypt32/decode.c | 12 ++++++++---- 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/dlls/crypt32/decode.c b/dlls/crypt32/decode.c index 47b0ba6..8c16d31 100644 --- a/dlls/crypt32/decode.c +++ b/dlls/crypt32/decode.c @@ -3913,15 +3913,19 @@ static BOOL CRYPT_AsnDecodeDistPointName(const BYTE *pbEncoded, sizeof(CERT_ALT_NAME_ENTRY), TRUE, offsetof(CERT_ALT_NAME_ENTRY, u.pwszURL) }; BYTE lenBytes = GET_LEN_BYTES(pbEncoded[1]); + DWORD nameLen; if (dataLen) { - DWORD nameLen; - ret = CRYPT_AsnDecodeArray(&arrayDesc, pbEncoded + 1 + lenBytes, cbEncoded - 1 - lenBytes, 0, NULL, NULL, &nameLen, NULL, NULL); - bytesNeeded = sizeof(CRL_DIST_POINT_NAME) + nameLen; + /* The CERT_ALT_NAME_INFO's size is included by CRYPT_AsnDecodeArray + * as the sizeof(struct GenericArray), so don't include it in the + * total bytes needed. + */ + bytesNeeded = sizeof(CRL_DIST_POINT_NAME) + nameLen - + sizeof(CERT_ALT_NAME_INFO); } else bytesNeeded = sizeof(CRL_DIST_POINT_NAME); @@ -3944,7 +3948,7 @@ static BOOL CRYPT_AsnDecodeDistPointName(const BYTE *pbEncoded, name->dwDistPointNameChoice = CRL_DIST_POINT_FULL_NAME; ret = CRYPT_AsnDecodeArray(&arrayDesc, pbEncoded + 1 + lenBytes, cbEncoded - 1 - lenBytes, - 0, NULL, &name->u.FullName, pcbStructInfo, NULL, + 0, NULL, &name->u.FullName, &nameLen, NULL, name->u.FullName.rgAltEntry); } else

1 0

Alexander Morozov : ntoskrnl.exe: Initialize some IRP fields to prevent using bad pointers by IofCompleteRequest .
by Alexandre Julliard 16 Oct '08

16 Oct '08

Module: wine Branch: master Commit: e99451561dc431d3b22125ec235c6fafaba191c9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e99451561dc431d3b22125ec2… Author: Alexander Morozov <amorozov(a)etersoft.ru> Date: Thu Oct 16 12:06:23 2008 +0400 ntoskrnl.exe: Initialize some IRP fields to prevent using bad pointers by IofCompleteRequest. --- dlls/ntoskrnl.exe/ntoskrnl.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/ntoskrnl.exe/ntoskrnl.c b/dlls/ntoskrnl.exe/ntoskrnl.c index 1ed3040..4a20d5e 100644 --- a/dlls/ntoskrnl.exe/ntoskrnl.c +++ b/dlls/ntoskrnl.exe/ntoskrnl.c @@ -153,6 +153,7 @@ static NTSTATUS process_ioctl( DEVICE_OBJECT *device, ULONG code, void *in_buff, irp.UserBuffer = out_buff; irp.MdlAddress = &mdl; irp.Tail.Overlay.s.u.CurrentStackLocation = &irpsp; + irp.UserIosb = NULL; irpsp.MajorFunction = IRP_MJ_DEVICE_CONTROL; irpsp.Parameters.DeviceIoControl.OutputBufferLength = *out_size; @@ -160,6 +161,7 @@ static NTSTATUS process_ioctl( DEVICE_OBJECT *device, ULONG code, void *in_buff, irpsp.Parameters.DeviceIoControl.IoControlCode = code; irpsp.Parameters.DeviceIoControl.Type3InputBuffer = in_buff; irpsp.DeviceObject = device; + irpsp.CompletionRoutine = NULL; mdl.Next = NULL; mdl.Size = 0;

1 0

Kirill K. Smirnov : oleaut32: Remove superfluous return statement.
by Alexandre Julliard 16 Oct '08

16 Oct '08

Module: wine Branch: master Commit: c47d0715ab046d1e111b947b81803d7e9631b849 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c47d0715ab046d1e111b947b8… Author: Kirill K. Smirnov <lich(a)math.spbu.ru> Date: Thu Oct 16 13:20:59 2008 +0400 oleaut32: Remove superfluous return statement. --- dlls/oleaut32/olepicture.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/dlls/oleaut32/olepicture.c b/dlls/oleaut32/olepicture.c index 4b45779..93b932f 100644 --- a/dlls/oleaut32/olepicture.c +++ b/dlls/oleaut32/olepicture.c @@ -509,7 +509,6 @@ static void OLEPicture_SendNotify(OLEPictureImpl* this, DISPID dispID) IUnknown_Release(CD.pUnk); } IEnumConnections_Release(pEnum); - return; } /************************************************************************

1 0

Kirill K. Smirnov : oleaut32: Return correct error code.
by Alexandre Julliard 16 Oct '08

16 Oct '08

Module: wine Branch: master Commit: fd6d2b4d0c2012d4ac281069df1ddf17fa816fa2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fd6d2b4d0c2012d4ac281069d… Author: Kirill K. Smirnov <lich(a)math.spbu.ru> Date: Thu Oct 16 13:20:54 2008 +0400 oleaut32: Return correct error code. --- dlls/oleaut32/olepicture.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/oleaut32/olepicture.c b/dlls/oleaut32/olepicture.c index ce6ad16..4b45779 100644 --- a/dlls/oleaut32/olepicture.c +++ b/dlls/oleaut32/olepicture.c @@ -1802,7 +1802,7 @@ static HRESULT WINAPI OLEPictureImpl_Load(IPersistStream* iface,IStream*pStm) { hr=IStream_Read(pStm,header,8,&xread); if (hr || xread!=8) { ERR("Failure while reading picture header (hr is %x, nread is %d).\n",hr,xread); - return hr; + return (hr?hr:E_FAIL); } headerread += xread; xread = 0;

1 0

Henri Verbeet : wined3d: Modifying GL_TEXTURE_MIN_FILTER modifies both the MINFILTER and MIPFILTER sampler states .
by Alexandre Julliard 16 Oct '08

16 Oct '08

Module: wine Branch: master Commit: 723f3656e54c5840956c56c047bc04fad070b405 URL: http://source.winehq.org/git/wine.git/?a=commit;h=723f3656e54c5840956c56c04… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Oct 16 11:52:54 2008 +0200 wined3d: Modifying GL_TEXTURE_MIN_FILTER modifies both the MINFILTER and MIPFILTER sampler states. --- dlls/wined3d/context.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index bf77f12..d8ac3ea 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -83,9 +83,11 @@ static void context_apply_attachment_filter_states(IWineD3DDevice *iface, IWineD /* Update base texture states array */ if (SUCCEEDED(IWineD3DSurface_GetContainer(surface, &IID_IWineD3DBaseTexture, (void **)&texture_impl))) { - if (texture_impl->baseTexture.states[WINED3DTEXSTA_MINFILTER] != WINED3DTEXF_POINT) + if (texture_impl->baseTexture.states[WINED3DTEXSTA_MINFILTER] != WINED3DTEXF_POINT + || texture_impl->baseTexture.states[WINED3DTEXSTA_MIPFILTER] != WINED3DTEXF_NONE) { texture_impl->baseTexture.states[WINED3DTEXSTA_MINFILTER] = WINED3DTEXF_POINT; + texture_impl->baseTexture.states[WINED3DTEXSTA_MIPFILTER] = WINED3DTEXF_NONE; update_minfilter = TRUE; }

1 0

Kirill K. Smirnov : oleaut32: Print error messages as ERR.
by Alexandre Julliard 16 Oct '08

16 Oct '08

Module: wine Branch: master Commit: 4dfaf90ab21422eaeffb08e114fc9aed8730c808 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4dfaf90ab21422eaeffb08e11… Author: Kirill K. Smirnov <lich(a)math.spbu.ru> Date: Thu Oct 16 13:20:45 2008 +0400 oleaut32: Print error messages as ERR. --- dlls/oleaut32/olepicture.c | 26 +++++++++++++------------- 1 files changed, 13 insertions(+), 13 deletions(-) diff --git a/dlls/oleaut32/olepicture.c b/dlls/oleaut32/olepicture.c index 09d2c32..ce6ad16 100644 --- a/dlls/oleaut32/olepicture.c +++ b/dlls/oleaut32/olepicture.c @@ -1036,7 +1036,7 @@ static int _gif_inputfunc(GifFileType *gif, GifByteType *data, int len) { struct gifdata *gd = (struct gifdata*)gif->UserData; if (len+gd->curoff > gd->len) { - FIXME("Trying to read %d bytes, but only %d available.\n",len, gd->len-gd->curoff); + ERR("Trying to read %d bytes, but only %d available.\n",len, gd->len-gd->curoff); len = gd->len - gd->curoff; } memcpy(data, gd->data+gd->curoff, len); @@ -1066,14 +1066,14 @@ static HRESULT OLEPictureImpl_LoadGif(OLEPictureImpl *This, BYTE *xbuf, ULONG xr gif = DGifOpen((void*)&gd, _gif_inputfunc); ret = DGifSlurp(gif); if (ret == GIF_ERROR) { - FIXME("Failed reading GIF using libgif.\n"); + ERR("Failed reading GIF using libgif.\n"); return E_FAIL; } TRACE("screen height %d, width %d\n", gif->SWidth, gif->SHeight); TRACE("color res %d, backgcolor %d\n", gif->SColorResolution, gif->SBackGroundColor); TRACE("imgcnt %d\n", gif->ImageCount); if (gif->ImageCount<1) { - FIXME("GIF stream does not have images inside?\n"); + ERR("GIF stream does not have images inside?\n"); return E_FAIL; } TRACE("curimage: %d x %d, on %dx%d, interlace %d\n", @@ -1259,7 +1259,7 @@ static HRESULT OLEPictureImpl_LoadJpeg(OLEPictureImpl *This, BYTE *xbuf, ULONG x if(!libjpeg_handle) { if(!load_libjpeg()) { - FIXME("Failed reading JPEG because unable to find %s\n", SONAME_LIBJPEG); + ERR("Failed reading JPEG because unable to find %s\n", SONAME_LIBJPEG); return E_FAIL; } } @@ -1298,7 +1298,7 @@ static HRESULT OLEPictureImpl_LoadJpeg(OLEPictureImpl *This, BYTE *xbuf, ULONG x while ( jd.output_scanline<jd.output_height ) { x = pjpeg_read_scanlines(&jd,&samprow,1); if (x != 1) { - FIXME("failed to read current scanline?\n"); + ERR("failed to read current scanline?\n"); break; } /* We have to convert from RGB to BGR, see MSDN/ BITMAPINFOHEADER */ @@ -1672,7 +1672,7 @@ static HRESULT OLEPictureImpl_LoadIcon(OLEPictureImpl *This, BYTE *xbuf, ULONG x 0 ); if (!hicon) { - FIXME("CreateIcon failed.\n"); + ERR("CreateIcon failed.\n"); return E_FAIL; } else { This->desc.picType = PICTYPE_ICON; @@ -1801,7 +1801,7 @@ static HRESULT WINAPI OLEPictureImpl_Load(IPersistStream* iface,IStream*pStm) { do { hr=IStream_Read(pStm,header,8,&xread); if (hr || xread!=8) { - FIXME("Failure while reading picture header (hr is %x, nread is %d).\n",hr,xread); + ERR("Failure while reading picture header (hr is %x, nread is %d).\n",hr,xread); return hr; } headerread += xread; @@ -1879,7 +1879,7 @@ static HRESULT WINAPI OLEPictureImpl_Load(IPersistStream* iface,IStream*pStm) { break; } if (xread != This->datalen) - FIXME("Could only read %d of %d bytes out of stream?\n",xread,This->datalen); + ERR("Could only read %d of %d bytes out of stream?\n",xread,This->datalen); } if (This->datalen == 0) { /* Marks the "NONE" picture */ This->desc.picType = PICTYPE_NONE; @@ -2637,7 +2637,7 @@ HRESULT WINAPI OleLoadPicture( LPSTREAM lpstream, LONG lSize, BOOL fRunmode, return hr; hr = IPicture_QueryInterface(newpic,&IID_IPersistStream, (LPVOID*)&ps); if (hr) { - FIXME("Could not get IPersistStream iface from Ole Picture?\n"); + ERR("Could not get IPersistStream iface from Ole Picture?\n"); IPicture_Release(newpic); *ppvObj = NULL; return hr; @@ -2653,7 +2653,7 @@ HRESULT WINAPI OleLoadPicture( LPSTREAM lpstream, LONG lSize, BOOL fRunmode, } hr = IPicture_QueryInterface(newpic,riid,ppvObj); if (hr) - FIXME("Failed to get interface %s from IPicture.\n",debugstr_guid(riid)); + ERR("Failed to get interface %s from IPicture.\n",debugstr_guid(riid)); IPicture_Release(newpic); return hr; } @@ -2676,7 +2676,7 @@ HRESULT WINAPI OleLoadPictureEx( LPSTREAM lpstream, LONG lSize, BOOL fRunmode, return hr; hr = IPicture_QueryInterface(newpic,&IID_IPersistStream, (LPVOID*)&ps); if (hr) { - FIXME("Could not get IPersistStream iface from Ole Picture?\n"); + ERR("Could not get IPersistStream iface from Ole Picture?\n"); IPicture_Release(newpic); *ppvObj = NULL; return hr; @@ -2692,7 +2692,7 @@ HRESULT WINAPI OleLoadPictureEx( LPSTREAM lpstream, LONG lSize, BOOL fRunmode, } hr = IPicture_QueryInterface(newpic,riid,ppvObj); if (hr) - FIXME("Failed to get interface %s from IPicture.\n",debugstr_guid(riid)); + ERR("Failed to get interface %s from IPicture.\n",debugstr_guid(riid)); IPicture_Release(newpic); return hr; } @@ -2798,7 +2798,7 @@ HRESULT WINAPI OleLoadPicturePath( LPOLESTR szURLorPath, LPUNKNOWN punkCaller, hRes = IPicture_QueryInterface(ipicture,riid,ppvRet); if (hRes) - FIXME("Failed to get interface %s from IPicture.\n",debugstr_guid(riid)); + ERR("Failed to get interface %s from IPicture.\n",debugstr_guid(riid)); IPicture_Release(ipicture); return hRes;

1 0

Henri Verbeet : wined3d: Handle offscreen rendering a bit nicer in surface_blt_to_drawable().
by Alexandre Julliard 16 Oct '08

16 Oct '08

Module: wine Branch: master Commit: f5fbfe47d7b5e3667f43708d17eedd62219c3952 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f5fbfe47d7b5e3667f43708d1… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Thu Oct 16 11:52:54 2008 +0200 wined3d: Handle offscreen rendering a bit nicer in surface_blt_to_drawable(). --- dlls/wined3d/surface.c | 15 +++++++++++---- 1 files changed, 11 insertions(+), 4 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 72f7537..9f0f5f9 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -4238,18 +4238,25 @@ static inline void surface_blt_to_drawable(IWineD3DSurfaceImpl *This, const RECT glTexParameteri(bind_target, GL_TEXTURE_MIN_FILTER, GL_NEAREST); checkGLcall("glTexParameteri"); + if (device->render_offscreen) + { + LONG tmp = rect.top; + rect.top = rect.bottom; + rect.bottom = tmp; + } + glBegin(GL_QUADS); glTexCoord3fv(&coords[0].x); - glVertex2i(rect.left, device->render_offscreen ? rect.bottom : rect.top); + glVertex2i(rect.left, rect.top); glTexCoord3fv(&coords[1].x); - glVertex2i(rect.left, device->render_offscreen ? rect.top : rect.bottom); + glVertex2i(rect.left, rect.bottom); glTexCoord3fv(&coords[2].x); - glVertex2i(rect.right, device->render_offscreen ? rect.top : rect.bottom); + glVertex2i(rect.right, rect.bottom); glTexCoord3fv(&coords[3].x); - glVertex2i(rect.right, device->render_offscreen ? rect.bottom : rect.top); + glVertex2i(rect.right, rect.top); glEnd(); checkGLcall("glEnd");

1 0