wine-commits August 2007

wine-commits@winehq.org

3 participants
1085 discussions

Roderick Colenbrander : wined3d: Pass the proper DeviceName to ChangeDisplaySettings.
by Alexandre Julliard 14 Aug '07

14 Aug '07

Module: wine Branch: master Commit: 6b177c44e10d7c1495de2de2ca605e03a8a11d05 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6b177c44e10d7c1495de2de2c… Author: Roderick Colenbrander <thunderbird2k(a)gmx.net> Date: Mon Aug 13 16:47:17 2007 +0200 wined3d: Pass the proper DeviceName to ChangeDisplaySettings. --- dlls/wined3d/device.c | 3 +-- dlls/wined3d/directx.c | 8 ++++++++ dlls/wined3d/wined3d_private.h | 1 + 3 files changed, 10 insertions(+), 2 deletions(-) diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index b0c0b20..11679af 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -1343,8 +1343,7 @@ static HRESULT WINAPI IWineD3DDeviceImpl_CreateAdditionalSwapChain(IWineD3DDevic devmode.dmBitsPerPel = (bpp >= 24) ? 32 : bpp; /* Stupid XVidMode cannot change bpp */ devmode.dmPelsWidth = pPresentationParameters->BackBufferWidth; devmode.dmPelsHeight = pPresentationParameters->BackBufferHeight; - MultiByteToWideChar(CP_ACP, 0, "Gamers CG", -1, devmode.dmDeviceName, CCHDEVICENAME); - ChangeDisplaySettingsExW(devmode.dmDeviceName, &devmode, object->win_handle, CDS_FULLSCREEN, NULL); + ChangeDisplaySettingsExW(This->adapter->DeviceName, &devmode, NULL, CDS_FULLSCREEN, NULL); /* For GetDisplayMode */ This->ddraw_width = devmode.dmPelsWidth; diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c index 13fcf51..563c714 100644 --- a/dlls/wined3d/directx.c +++ b/dlls/wined3d/directx.c @@ -2479,6 +2479,8 @@ BOOL InitAdapters(void) { /* For now only one default adapter */ { int attribute; + DISPLAY_DEVICEW DisplayDevice; + TRACE("Initializing default adapter\n"); Adapters[0].monitorPoint.x = -1; Adapters[0].monitorPoint.y = -1; @@ -2499,6 +2501,12 @@ BOOL InitAdapters(void) { Adapters[0].driver = "Display"; Adapters[0].description = "Direct3D HAL"; + /* Initialize the Adapter's DeviceName which is required for ChangeDisplaySettings and friends */ + DisplayDevice.cb = sizeof(DisplayDevice); + EnumDisplayDevicesW(NULL, 0 /* Adapter 0 = iDevNum 0 */, &DisplayDevice, 0); + TRACE("DeviceName: %s\n", debugstr_w(DisplayDevice.DeviceName)); + strcpyW(Adapters[0].DeviceName, DisplayDevice.DeviceName); + if (WineD3D_CreateFakeGLContext()) { int iPixelFormat; int attribs[8]; diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 3ceb767..159844e 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -588,6 +588,7 @@ struct WineD3DAdapter WineD3D_GL_Info gl_info; const char *driver; const char *description; + WCHAR DeviceName[CCHDEVICENAME]; /* DeviceName for use with e.g. ChangeDisplaySettings */ int nCfgs; WineD3D_PixelFormat *cfgs; };

1 0

Detlef Riekenberg : winspool/tests: Fix 2 failing tests for restricted users.
by Alexandre Julliard 14 Aug '07

14 Aug '07

Module: wine Branch: master Commit: 1c8a466cca0a7d6ad3c1b062ae43d69541de24d6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1c8a466cca0a7d6ad3c1b062a… Author: Detlef Riekenberg <wine.dev(a)web.de> Date: Mon Aug 13 12:07:57 2007 +0200 winspool/tests: Fix 2 failing tests for restricted users. --- dlls/winspool.drv/tests/info.c | 48 ++++++++++++++++++++++----------------- 1 files changed, 27 insertions(+), 21 deletions(-) diff --git a/dlls/winspool.drv/tests/info.c b/dlls/winspool.drv/tests/info.c index 6006db3..65d0cf4 100644 --- a/dlls/winspool.drv/tests/info.c +++ b/dlls/winspool.drv/tests/info.c @@ -87,17 +87,29 @@ static LPWSTR tempfileW = NULL; /* ################################ */ /* report common behavior only once */ -static DWORD report_deactivated_spooler = 1; +static DWORD deactivated_spooler_reported = 0; #define RETURN_ON_DEACTIVATED_SPOOLER(res) \ - if((res == 0) && (GetLastError() == RPC_S_SERVER_UNAVAILABLE)) \ + if ((res == 0) && (GetLastError() == RPC_S_SERVER_UNAVAILABLE)) \ { \ - if(report_deactivated_spooler > 0) { \ - report_deactivated_spooler--; \ + if (!deactivated_spooler_reported) { \ + deactivated_spooler_reported++; \ skip("The Service 'Spooler' is required for many test\n"); \ } \ return; \ } +static DWORD access_denied_reported = 0; +#define RETURN_ON_ACCESS_DENIED(res) \ + if ((res == 0) && (GetLastError() == ERROR_ACCESS_DENIED)) \ + { \ + if (!access_denied_reported) { \ + access_denied_reported++; \ + skip("More Access-Rights are required for many test\n"); \ + } \ + return; \ + } + +/* ################################ */ static void find_default_printer(VOID) { @@ -313,11 +325,7 @@ static void test_AddMonitor(void) SetLastError(MAGIC_DEAD); res = AddMonitorA(NULL, 2, (LPBYTE) &mi2a); RETURN_ON_DEACTIVATED_SPOOLER(res) - - if (!res && (GetLastError() == ERROR_ACCESS_DENIED)) { - skip("(ACCESS_DENIED)\n"); - return; - } + RETURN_ON_ACCESS_DENIED(res) /* NT: ERROR_INVALID_PARAMETER, 9x: ERROR_INVALID_ENVIRONMENT */ ok(!res && ((GetLastError() == ERROR_INVALID_PARAMETER) || @@ -440,10 +448,8 @@ static void test_AddPort(void) SetLastError(0xdeadbeef); res = AddPortA(NULL, 0, empty); /* Allowed only for (Printer-)Administrators */ - if (!res && (GetLastError() == ERROR_ACCESS_DENIED)) { - skip("(ACCESS_DENIED)\n"); - return; - } + RETURN_ON_ACCESS_DENIED(res) + /* XP: ERROR_NOT_SUPPORTED, NT351 and 9x: ERROR_INVALID_PARAMETER */ ok( !res && ((GetLastError() == ERROR_NOT_SUPPORTED) || (GetLastError() == ERROR_INVALID_PARAMETER)), @@ -593,10 +599,8 @@ static void test_ConfigurePort(void) SetLastError(0xdeadbeef); res = ConfigurePortA(NULL, 0, empty); /* Allowed only for (Printer-)Administrators */ - if (!res && (GetLastError() == ERROR_ACCESS_DENIED)) { - skip("(ACCESS_DENIED)\n"); - return; - } + RETURN_ON_ACCESS_DENIED(res) + /* XP: ERROR_NOT_SUPPORTED, NT351 and 9x: ERROR_INVALID_PARAMETER */ ok( !res && ((GetLastError() == ERROR_NOT_SUPPORTED) || (GetLastError() == ERROR_INVALID_PARAMETER)), @@ -734,10 +738,8 @@ static void test_DeletePort(void) SetLastError(0xdeadbeef); res = DeletePortA(NULL, 0, empty); /* Allowed only for (Printer-)Administrators */ - if (!res && (GetLastError() == ERROR_ACCESS_DENIED)) { - skip("(ACCESS_DENIED)\n"); - return; - } + RETURN_ON_ACCESS_DENIED(res) + /* XP: ERROR_NOT_SUPPORTED, NT351 and 9x: ERROR_INVALID_PARAMETER */ ok( !res && ((GetLastError() == ERROR_NOT_SUPPORTED) || (GetLastError() == ERROR_INVALID_PARAMETER)), @@ -1810,6 +1812,8 @@ static void test_XcvDataW_MonitorUI(void) SetLastError(0xdeadbeef); res = OpenPrinter(xcv_localport, &hXcv, &pd); RETURN_ON_DEACTIVATED_SPOOLER(res) + RETURN_ON_ACCESS_DENIED(res) + ok(res, "returned %d with %u and handle %p (expected '!= 0')\n", res, GetLastError(), hXcv); if (!res) return; @@ -1927,6 +1931,8 @@ static void test_XcvDataW_PortIsValid(void) res = OpenPrinter(xcv_localport, &hXcv, &pd); RETURN_ON_DEACTIVATED_SPOOLER(res) + RETURN_ON_ACCESS_DENIED(res) + ok(res, "returned %d with %u and handle %p (expected '!= 0')\n", res, GetLastError(), hXcv); if (!res) return;

1 0

Misha Koshelev : urlmon: Fix timing issue with FLAG_REQUEST_COMPLETE and InternetQueryDataAvailable in HttpProtocol_Continue .
by Alexandre Julliard 14 Aug '07

14 Aug '07

Module: wine Branch: master Commit: 4f405b228d94c79cebc506053c7d892b7e18a7fa URL: http://source.winehq.org/git/wine.git/?a=commit;h=4f405b228d94c79cebc506053… Author: Misha Koshelev <mk144210(a)bcm.edu> Date: Mon Aug 13 00:24:57 2007 -0500 urlmon: Fix timing issue with FLAG_REQUEST_COMPLETE and InternetQueryDataAvailable in HttpProtocol_Continue. --- dlls/urlmon/http.c | 12 +++++++----- 1 files changed, 7 insertions(+), 5 deletions(-) diff --git a/dlls/urlmon/http.c b/dlls/urlmon/http.c index 08d6419..2d1929d 100644 --- a/dlls/urlmon/http.c +++ b/dlls/urlmon/http.c @@ -661,20 +661,22 @@ static HRESULT WINAPI HttpProtocol_Continue(IInternetProtocol *iface, PROTOCOLDA if (pProtocolData->pData >= (LPVOID)BINDSTATUS_DOWNLOADINGDATA) { + /* InternetQueryDataAvailable may immediately fork and perform its asynchronous + * read, so clear the flag _before_ calling so it does not incorrectly get cleared + * after the status callback is called */ + This->flags &= ~FLAG_REQUEST_COMPLETE; if (!InternetQueryDataAvailable(This->request, &This->available_bytes, 0, 0)) { - if (GetLastError() == ERROR_IO_PENDING) - { - This->flags &= ~FLAG_REQUEST_COMPLETE; - } - else + if (GetLastError() != ERROR_IO_PENDING) { + This->flags |= FLAG_REQUEST_COMPLETE; WARN("InternetQueryDataAvailable failed: %d\n", GetLastError()); HTTPPROTOCOL_ReportResult(This, INET_E_DATA_NOT_AVAILABLE); } } else { + This->flags |= FLAG_REQUEST_COMPLETE; HTTPPROTOCOL_ReportData(This); } }

1 0

Misha Koshelev : winebrowser: Add support for XTYP_REQUEST in DDE callback.
by Alexandre Julliard 14 Aug '07

14 Aug '07

Module: wine Branch: master Commit: 70edf2d2234159f021465f52de52cb4cfcf6e1ab URL: http://source.winehq.org/git/wine.git/?a=commit;h=70edf2d2234159f021465f52d… Author: Misha Koshelev <mk144210(a)bcm.edu> Date: Sun Aug 12 20:08:01 2007 -0500 winebrowser: Add support for XTYP_REQUEST in DDE callback. --- programs/winebrowser/main.c | 54 ++++++++++++++++++++++++++++++------------ 1 files changed, 38 insertions(+), 16 deletions(-) diff --git a/programs/winebrowser/main.c b/programs/winebrowser/main.c index 976e465..ee7b947 100644 --- a/programs/winebrowser/main.c +++ b/programs/winebrowser/main.c @@ -125,15 +125,16 @@ static int open_mailto_url( const char *url ) * DDE helper functions. */ -static char *ddeExec = NULL; -static HSZ hszTopic = 0; +static char *ddeString = NULL; +static HSZ hszTopic = 0, hszReturn = 0; +static DWORD ddeInst = 0; -/* Dde callback, save the execute string for processing */ +/* Dde callback, save the execute or request string for processing */ static HDDEDATA CALLBACK ddeCb(UINT uType, UINT uFmt, HCONV hConv, HSZ hsz1, HSZ hsz2, HDDEDATA hData, ULONG_PTR dwData1, ULONG_PTR dwData2) { - DWORD size = 0; + DWORD size = 0, ret = 0; WINE_TRACE("dde_cb: %04x, %04x, %p, %p, %p, %p, %08lx, %08lx\n", uType, uFmt, hConv, hsz1, hsz2, hData, dwData1, dwData2); @@ -148,13 +149,26 @@ static HDDEDATA CALLBACK ddeCb(UINT uType, UINT uFmt, HCONV hConv, case XTYP_EXECUTE: if (!(size = DdeGetData(hData, NULL, 0, 0))) WINE_ERR("DdeGetData returned zero size of execute string\n"); - else if (!(ddeExec = HeapAlloc(GetProcessHeap(), 0, size))) + else if (!(ddeString = HeapAlloc(GetProcessHeap(), 0, size))) WINE_ERR("Out of memory\n"); - else if (DdeGetData(hData, (LPBYTE)ddeExec, size, 0) != size) + else if (DdeGetData(hData, (LPBYTE)ddeString, size, 0) != size) WINE_WARN("DdeGetData did not return %d bytes\n", size); DdeFreeDataHandle(hData); return (HDDEDATA)DDE_FACK; + case XTYP_REQUEST: + ret = -3; /* error */ + if (!(size = DdeQueryString(ddeInst, hsz2, NULL, 0, CP_WINANSI))) + WINE_ERR("DdeQueryString returned zero size of request string\n"); + else if (!(ddeString = HeapAlloc(GetProcessHeap(), 0, size+1))) + WINE_ERR("Out of memory\n"); + else if (DdeQueryString(ddeInst, hsz2, ddeString, size+1, CP_WINANSI) != size) + WINE_WARN("DdeQueryString did not return %d bytes\n", size); + else + ret = -2; /* acknowledgment */ + return DdeCreateDataHandle(ddeInst, (LPBYTE)&ret, sizeof(ret), 0, + hszReturn, CF_TEXT, 0); + default: return NULL; } @@ -164,15 +178,15 @@ static char *get_url_from_dde(void) { static const char szApplication[] = "IExplore"; static const char szTopic[] = "WWW_OpenURL"; + static const char szReturn[] = "Return"; HSZ hszApplication = 0; - DWORD ddeInst = 0; UINT_PTR timer = 0; int rc; char *ret = NULL; rc = DdeInitializeA(&ddeInst, ddeCb, CBF_SKIP_ALLNOTIFICATIONS | CBF_FAIL_ADVISES | - CBF_FAIL_POKES | CBF_FAIL_REQUESTS, 0); + CBF_FAIL_POKES, 0); if (rc != DMLERR_NO_ERROR) { WINE_ERR("Unable to initialize DDE, DdeInitialize returned %d\n", rc); @@ -193,6 +207,13 @@ static char *get_url_from_dde(void) goto done; } + hszReturn = DdeCreateStringHandleA(ddeInst, szReturn, CP_WINANSI); + if (!hszReturn) + { + WINE_ERR("Unable to initialize DDE, DdeCreateStringHandle failed\n"); + goto done; + } + if (!DdeNameService(ddeInst, hszApplication, 0, DNS_REGISTER)) { WINE_ERR("Unable to initialize DDE, DdeNameService failed\n"); @@ -206,7 +227,7 @@ static char *get_url_from_dde(void) goto done; } - while (!ddeExec) + while (!ddeString) { MSG msg; if (!GetMessage(&msg, NULL, 0, 0)) break; @@ -214,21 +235,21 @@ static char *get_url_from_dde(void) DispatchMessage(&msg); } - if (ddeExec) + if (ddeString) { - if (*ddeExec == '"') + if (*ddeString == '"') { - char *endquote = strchr(ddeExec+1, '"'); + char *endquote = strchr(ddeString+1, '"'); if (!endquote) { - WINE_ERR("Unabled to retrieve URL from string '%s'\n", ddeExec); + WINE_ERR("Unabled to retrieve URL from string '%s'\n", ddeString); goto done; } *endquote = 0; - ret = ddeExec+1; + ret = ddeString+1; } else - ret = ddeExec; + ret = ddeString; } done: @@ -236,6 +257,7 @@ done: if (ddeInst) { if (hszTopic && hszApplication) DdeNameService(ddeInst, hszApplication, 0, DNS_UNREGISTER); + if (hszReturn) DdeFreeStringHandle(ddeInst, hszReturn); if (hszTopic) DdeFreeStringHandle(ddeInst, hszTopic); if (hszApplication) DdeFreeStringHandle(ddeInst, hszApplication); DdeUninitialize(ddeInst); @@ -332,6 +354,6 @@ int main(int argc, char *argv[]) ret = open_http_url( url ); done: - HeapFree(GetProcessHeap(), 0, ddeExec); + HeapFree(GetProcessHeap(), 0, ddeString); return ret; }

1 0

Jason Edmeades : setupapi: Remove duplicate backslashes on dirids.
by Alexandre Julliard 13 Aug '07

13 Aug '07

Module: wine Branch: master Commit: 7f901f41f05556307b8ed445591d68e264c61375 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7f901f41f05556307b8ed4455… Author: Jason Edmeades <jason.edmeades(a)googlemail.com> Date: Fri Aug 10 22:28:35 2007 +0100 setupapi: Remove duplicate backslashes on dirids. --- dlls/advpack/tests/advpack.c | 10 ++++------ dlls/setupapi/parser.c | 6 ++++-- dlls/setupapi/tests/parser.c | 12 ++++++++++++ 3 files changed, 20 insertions(+), 8 deletions(-) diff --git a/dlls/advpack/tests/advpack.c b/dlls/advpack/tests/advpack.c index 577f274..4c49e3f 100644 --- a/dlls/advpack/tests/advpack.c +++ b/dlls/advpack/tests/advpack.c @@ -419,12 +419,10 @@ static void translateinfstringex_test(void) hr = pTranslateInfStringEx(hinf, inf_file, "Options.NTx86", "Result1", buffer, size, &size, NULL); ok(hr == S_OK, "Expected S_OK, got %08x\n", hr); - todo_wine { /* Wine returns C:\\Program Files, not C:\Program Files */ - ok(!lstrcmpi(buffer, PROG_FILES_ROOT), - "Expected %s, got %s\n", PROG_FILES_ROOT, buffer); - ok(size == lstrlenA(PROG_FILES_ROOT)+1, "Expected size %d, got %d\n", - lstrlenA(PROG_FILES_ROOT)+1, size); - } + ok(!lstrcmpi(buffer, PROG_FILES_ROOT), + "Expected %s, got %s\n", PROG_FILES_ROOT, buffer); + ok(size == lstrlenA(PROG_FILES_ROOT)+1, "Expected size %d, got %d\n", + lstrlenA(PROG_FILES_ROOT)+1, size); memset(buffer, 'a', PROG_FILES_LEN); buffer[PROG_FILES_LEN - 1] = '\0'; diff --git a/dlls/setupapi/parser.c b/dlls/setupapi/parser.c index ea410f8..d5b7643 100644 --- a/dlls/setupapi/parser.c +++ b/dlls/setupapi/parser.c @@ -316,7 +316,8 @@ static const WCHAR *get_dirid_subst( struct inf_file *file, int dirid, unsigned /* retrieve the string substitution for a given string, or NULL if not found */ /* if found, len is set to the substitution length */ -static const WCHAR *get_string_subst( struct inf_file *file, const WCHAR *str, unsigned int *len ) +static const WCHAR *get_string_subst( struct inf_file *file, const WCHAR *str, unsigned int *len, + BOOL no_trailing_slash ) { static const WCHAR percent = '%'; @@ -353,6 +354,7 @@ static const WCHAR *get_string_subst( struct inf_file *file, const WCHAR *str, u dirid_str[*len] = 0; dirid = strtolW( dirid_str, &end, 10 ); if (!*end) ret = get_dirid_subst( file, dirid, len ); + if (no_trailing_slash && ret && *len && ret[*len - 1] == '\\') *len -= 1; HeapFree( GetProcessHeap(), 0, dirid_str ); return ret; } @@ -387,7 +389,7 @@ unsigned int PARSER_string_substW( struct inf_file *file, const WCHAR *text, WCH else /* end of the %xx% string, find substitution */ { len = p - start - 1; - subst = get_string_subst( file, start + 1, &len ); + subst = get_string_subst( file, start + 1, &len, p[1] == '\\' ); if (!subst) { subst = start; diff --git a/dlls/setupapi/tests/parser.c b/dlls/setupapi/tests/parser.c index 858c889..f11abb8 100644 --- a/dlls/setupapi/tests/parser.c +++ b/dlls/setupapi/tests/parser.c @@ -60,6 +60,7 @@ static const char tmpfilename[] = ".\\tmp.inf"; #define STR_SECTION "[Strings]\nfoo=aaa\nbar=bbb\nloop=%loop2%\nloop2=%loop%\n" \ "per%%cent=abcd\nper=1\ncent=2\n22=foo\n" \ "big=" A400 "\n" \ + "mydrive=\"C:\\\"\n" \ "verybig=" A400 A400 A400 "\n" /* create a new file with specified contents and open it */ @@ -284,6 +285,7 @@ static const struct { "ab=cd\",\"ef", "ab", { "cd,ef" } }, { "ab=cd\",ef", "ab", { "cd,ef" } }, { "ab=cd\",ef\\\nab", "ab", { "cd,ef\\" } }, + /* single quotes (unhandled)*/ { "HKLM,A,B,'C',D", NULL, { "HKLM", "A","B","'C'","D" } }, /* spaces */ @@ -317,6 +319,16 @@ static const struct { "a=%big%%big%%big%%big%\n" STR_SECTION, "a", { A400 A400 A400 A400 } }, { "a=%big%%big%%big%%big%%big%%big%%big%%big%%big%\n" STR_SECTION, "a", { A400 A400 A400 A400 A400 A400 A400 A400 A400 } }, { "a=%big%%big%%big%%big%%big%%big%%big%%big%%big%%big%%big%\n" STR_SECTION, "a", { A4097 /*MAX_INF_STRING_LENGTH+1*/ } }, + + /* Prove expansion of system entries removes extra \'s and string + replacements doesnt */ + { "ab=\"%24%\"\n" STR_SECTION, "ab", { "C:\\" } }, + { "ab=\"%mydrive%\"\n" STR_SECTION, "ab", { "C:\\" } }, + { "ab=\"%24%\\fred\"\n" STR_SECTION, "ab", { "C:\\fred" } }, + { "ab=\"%mydrive%\\fred\"\n" STR_SECTION,"ab", { "C:\\\\fred" } }, + /* Confirm duplicate \'s kept */ + { "ab=\"%24%\\\\fred\"", "ab", { "C:\\\\fred" } }, + { "ab=C:\\\\FRED", "ab", { "C:\\\\FRED" } }, }; /* check the key of a certain line */

1 0

Roderick Colenbrander : wgl: Re-enable GLX_ATI_render_texture support.
by Alexandre Julliard 13 Aug '07

13 Aug '07

Module: wine Branch: master Commit: 04d383381ff090c9dace841f5f15d6429ae34334 URL: http://source.winehq.org/git/wine.git/?a=commit;h=04d383381ff090c9dace841f5… Author: Roderick Colenbrander <thunderbird2k(a)gmx.net> Date: Sun Aug 12 17:31:25 2007 +0200 wgl: Re-enable GLX_ATI_render_texture support. --- dlls/winex11.drv/opengl.c | 75 +++++++++++++++++++++++++++++++++++++-------- 1 files changed, 62 insertions(+), 13 deletions(-) diff --git a/dlls/winex11.drv/opengl.c b/dlls/winex11.drv/opengl.c index ca379d0..521b9e7 100644 --- a/dlls/winex11.drv/opengl.c +++ b/dlls/winex11.drv/opengl.c @@ -139,7 +139,7 @@ typedef struct wine_glpbuffer { static Wine_GLContext *context_list; static struct WineGLInfo WineGLInfo = { 0 }; static int use_render_texture_emulation = 1; -static int use_render_texture_ati = 0; +static int use_render_texture_ati = 1; static int swap_interval = 1; #define MAX_EXTENSIONS 16 @@ -241,11 +241,13 @@ MAKE_FUNCPTR(glXGetCurrentReadDrawable) /* GLX Extensions */ static void* (*pglXGetProcAddressARB)(const GLubyte *); -static BOOL (*pglXBindTexImageARB)(Display *dpy, GLXPbuffer pbuffer, int buffer); -static BOOL (*pglXReleaseTexImageARB)(Display *dpy, GLXPbuffer pbuffer, int buffer); -static BOOL (*pglXDrawableAttribARB)(Display *dpy, GLXDrawable draw, const int *attribList); static int (*pglXSwapIntervalSGI)(int); +/* ATI GLX Extensions */ +static BOOL (*pglXBindTexImageATI)(Display *dpy, GLXPbuffer pbuffer, int buffer); +static BOOL (*pglXReleaseTexImageATI)(Display *dpy, GLXPbuffer pbuffer, int buffer); +static BOOL (*pglXDrawableAttribATI)(Display *dpy, GLXDrawable draw, const int *attribList); + /* NV GLX Extension */ static void* (*pglXAllocateMemoryNV)(GLsizei size, GLfloat readfreq, GLfloat writefreq, GLfloat priority); static void (*pglXFreeMemoryNV)(GLvoid *pointer); @@ -502,9 +504,9 @@ LOAD_FUNCPTR(glXFreeMemoryNV) } if(glxRequireExtension("GLX_ATI_render_texture")) { - pglXBindTexImageARB = (void*)pglXGetProcAddressARB((const GLubyte *) "glXBindTexImageARB"); - pglXReleaseTexImageARB = (void*)pglXGetProcAddressARB((const GLubyte *) "glXReleaseTexImageARB"); - pglXDrawableAttribARB = (void*)pglXGetProcAddressARB((const GLubyte *) "glXDrawableAttribARB"); + pglXBindTexImageATI = (void*)pglXGetProcAddressARB((const GLubyte *) "glXBindTexImageATI"); + pglXReleaseTexImageATI = (void*)pglXGetProcAddressARB((const GLubyte *) "glXReleaseTexImageATI"); + pglXDrawableAttribATI = (void*)pglXGetProcAddressARB((const GLubyte *) "glXDrawableAttribATI"); } X11DRV_WineGL_LoadExtensions(); @@ -2490,11 +2492,11 @@ static GLboolean WINAPI X11DRV_wglSetPbufferAttribARB(HPBUFFERARB hPbuffer, cons if (!use_render_texture_ati && 1 == use_render_texture_emulation) { return GL_TRUE; } - if (NULL != pglXDrawableAttribARB) { + if (NULL != pglXDrawableAttribATI) { if (use_render_texture_ati) { FIXME("Need conversion for GLX_ATI_render_texture\n"); } - return pglXDrawableAttribARB(object->display, object->drawable, piAttribList); + return pglXDrawableAttribATI(object->display, object->drawable, piAttribList); } return GL_FALSE; } @@ -2893,8 +2895,35 @@ static GLboolean WINAPI X11DRV_wglBindTexImageARB(HPBUFFERARB hPbuffer, int iBuf return GL_TRUE; } - if (NULL != pglXBindTexImageARB) { - return pglXBindTexImageARB(object->display, object->drawable, iBuffer); + if (NULL != pglXBindTexImageATI) { + int buffer; + + switch(iBuffer) + { + case WGL_FRONT_LEFT_ARB: + buffer = GLX_FRONT_LEFT_ATI; + break; + case WGL_FRONT_RIGHT_ARB: + buffer = GLX_FRONT_RIGHT_ATI; + break; + case WGL_BACK_LEFT_ARB: + buffer = GLX_BACK_LEFT_ATI; + break; + case WGL_BACK_RIGHT_ARB: + buffer = GLX_BACK_RIGHT_ATI; + break; + default: + ERR("Unknown iBuffer=%#x\n", iBuffer); + return FALSE; + } + + /* In the sample 'ogl_offscreen_rendering_3' from codesampler.net I get garbage on the screen. + * I'm not sure if that's a bug in the ATI extension or in the program. I think that the program + * expected a single buffering format since it didn't ask for double buffering. A buffer swap + * fixed the program. I don't know what the correct behavior is. On the other hand that demo + * works fine using our pbuffer emulation path. + */ + return pglXBindTexImageATI(object->display, object->drawable, buffer); } return GL_FALSE; } @@ -2919,8 +2948,28 @@ static GLboolean WINAPI X11DRV_wglReleaseTexImageARB(HPBUFFERARB hPbuffer, int i if (!use_render_texture_ati && 1 == use_render_texture_emulation) { return GL_TRUE; } - if (NULL != pglXReleaseTexImageARB) { - return pglXReleaseTexImageARB(object->display, object->drawable, iBuffer); + if (NULL != pglXReleaseTexImageATI) { + int buffer; + + switch(iBuffer) + { + case WGL_FRONT_LEFT_ARB: + buffer = GLX_FRONT_LEFT_ATI; + break; + case WGL_FRONT_RIGHT_ARB: + buffer = GLX_FRONT_RIGHT_ATI; + break; + case WGL_BACK_LEFT_ARB: + buffer = GLX_BACK_LEFT_ATI; + break; + case WGL_BACK_RIGHT_ARB: + buffer = GLX_BACK_RIGHT_ATI; + break; + default: + ERR("Unknown iBuffer=%#x\n", iBuffer); + return FALSE; + } + return pglXReleaseTexImageATI(object->display, object->drawable, buffer); } return GL_FALSE; }

1 0

Chris Robinson : winex11: Set proper pixel format flags.
by Alexandre Julliard 13 Aug '07

13 Aug '07

Module: wine Branch: master Commit: 308da6313ec8dcf0352f312fc1192290cf717f04 URL: http://source.winehq.org/git/wine.git/?a=commit;h=308da6313ec8dcf0352f312fc… Author: Chris Robinson <chris.kcat(a)gmail.com> Date: Sun Aug 12 13:57:02 2007 -0700 winex11: Set proper pixel format flags. --- dlls/winex11.drv/opengl.c | 12 +++++++++++- 1 files changed, 11 insertions(+), 1 deletions(-) diff --git a/dlls/winex11.drv/opengl.c b/dlls/winex11.drv/opengl.c index 521b9e7..ebed630 100644 --- a/dlls/winex11.drv/opengl.c +++ b/dlls/winex11.drv/opengl.c @@ -1233,11 +1233,21 @@ int X11DRV_DescribePixelFormat(X11DRV_PDEVICE *physDev, ppfd->nVersion = 1; /* These flags are always the same... */ - ppfd->dwFlags = PFD_DRAW_TO_WINDOW | PFD_SUPPORT_OPENGL; + ppfd->dwFlags = PFD_SUPPORT_OPENGL; /* Now the flags extracted from the Visual */ wine_tsx11_lock(); + pglXGetFBConfigAttrib(gdi_display, fmt->fbconfig, GLX_X_RENDERABLE, &value); + if(value) + ppfd->dwFlags |= PFD_SUPPORT_GDI; + + pglXGetFBConfigAttrib(gdi_display, fmt->fbconfig, GLX_DRAWABLE_TYPE, &value); + if(value & GLX_WINDOW_BIT) + ppfd->dwFlags |= PFD_DRAW_TO_WINDOW; + if(value & GLX_PIXMAP_BIT) + ppfd->dwFlags |= PFD_DRAW_TO_BITMAP; + pglXGetFBConfigAttrib(gdi_display, fmt->fbconfig, GLX_CONFIG_CAVEAT, &value); if(value == GLX_SLOW_CONFIG) ppfd->dwFlags |= PFD_GENERIC_ACCELERATED;

1 0

Roderick Colenbrander : wined3d: Check the results of SetPixelFormat.
by Alexandre Julliard 13 Aug '07

13 Aug '07

Module: wine Branch: master Commit: 1dc0298a55b6050f19cf2df298268dcda4b85081 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1dc0298a55b6050f19cf2df29… Author: Roderick Colenbrander <thunderbird2k(a)gmx.net> Date: Sat Aug 11 17:04:26 2007 +0200 wined3d: Check the results of SetPixelFormat. --- dlls/wined3d/context.c | 16 +++++++++++++++- 1 files changed, 15 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index 9edbd73..0727aee 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -208,6 +208,7 @@ WineD3DContext *CreateContext(IWineD3DDeviceImpl *This, IWineD3DSurfaceImpl *tar short red, green, blue, alpha; short colorBits; short depthBits, stencilBits; + int res; hdc = GetDC(win_handle); if(hdc == NULL) { @@ -249,7 +250,20 @@ WineD3DContext *CreateContext(IWineD3DDeviceImpl *This, IWineD3DSurfaceImpl *tar } DescribePixelFormat(hdc, iPixelFormat, sizeof(pfd), &pfd); - SetPixelFormat(hdc, iPixelFormat, NULL); + res = SetPixelFormat(hdc, iPixelFormat, NULL); + if(!res) { + int oldPixelFormat = GetPixelFormat(hdc); + + if(oldPixelFormat) { + /* OpenGL doesn't allow pixel format adjustments. Print an error and continue using the old format. + * There's a big chance that the old format works although with a performance hit and perhaps rendering errors. */ + ERR("HDC=%p is already set to iPixelFormat=%d and OpenGL doesn't allow changes!\n", hdc, oldPixelFormat); + } + else { + ERR("SetPixelFormat failed on HDC=%p for iPixelFormat=%d\n", hdc, iPixelFormat); + return FALSE; + } + } } ctx = wglCreateContext(hdc);

1 0

Roderick Colenbrander : wined3d: Proper CheckDeviceType / CheckDepthStencilMatch support.
by Alexandre Julliard 13 Aug '07

13 Aug '07

Module: wine Branch: master Commit: d391c11d434fb799aceedf07b1ce5058bfacc8c2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d391c11d434fb799aceedf07b… Author: Roderick Colenbrander <thunderbird2k(a)gmx.net> Date: Sat Aug 11 16:25:58 2007 +0200 wined3d: Proper CheckDeviceType / CheckDepthStencilMatch support. --- dlls/wined3d/directx.c | 232 +++++++++++++++++----------------------- dlls/wined3d/wined3d_private.h | 8 ++ 2 files changed, 104 insertions(+), 136 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=d391c11d434fb799aceed…

1 0

Roderick Colenbrander : wined3d: Add more pixelformats to getColorBits.
by Alexandre Julliard 13 Aug '07

13 Aug '07

Module: wine Branch: master Commit: 4cd5298a57993794adcb6c9bb4af6e1d755275c9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4cd5298a57993794adcb6c9bb… Author: Roderick Colenbrander <thunderbird2k(a)gmx.net> Date: Sat Aug 11 16:47:47 2007 +0200 wined3d: Add more pixelformats to getColorBits. --- dlls/wined3d/utils.c | 10 +++++++++- 1 files changed, 9 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/utils.c b/dlls/wined3d/utils.c index c66c381..84af4ee 100644 --- a/dlls/wined3d/utils.c +++ b/dlls/wined3d/utils.c @@ -2486,8 +2486,16 @@ BOOL getColorBits(WINED3DFORMAT fmt, short *redSize, short *greenSize, short *bl TRACE("fmt: %s\n", debug_d3dformat(fmt)); switch(fmt) { - case WINED3DFMT_R5G6B5: case WINED3DFMT_X8R8G8B8: + case WINED3DFMT_R8G8B8: + case WINED3DFMT_A8R8G8B8: + case WINED3DFMT_A2R10G10B10: + case WINED3DFMT_X1R5G5B5: + case WINED3DFMT_A1R5G5B5: + case WINED3DFMT_R5G6B5: + case WINED3DFMT_R3G3B2: + case WINED3DFMT_A8P8: + case WINED3DFMT_P8: break; default: ERR("Unsupported format: %s\n", debug_d3dformat(fmt));

1 0

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 August 2007