Ros-diffs

ros-diffs@reactos.org

81390 discussions

[tfaber] 63733: [WBEMPROX] - Don't omit lpBytesReturned in DeviceIoControl call ROSTESTS-120 #resolve
by tfaber＠svn.reactos.org 23 Jul '14

23 Jul '14

Author: tfaber Date: Wed Jul 23 18:50:09 2014 New Revision: 63733 URL: http://svn.reactos.org/svn/reactos?rev=63733&view=rev Log: [WBEMPROX] - Don't omit lpBytesReturned in DeviceIoControl call ROSTESTS-120 #resolve Modified: trunk/reactos/dll/win32/wbemprox/builtin.c trunk/rostests/winetests/wbemprox/query.c Modified: trunk/reactos/dll/win32/wbemprox/builtin.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/wbemprox/builtin… ============================================================================== --- trunk/reactos/dll/win32/wbemprox/builtin.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/wbemprox/builtin.c [iso-8859-1] Wed Jul 23 18:50:09 2014 @@ -1477,6 +1477,7 @@ ULARGE_INTEGER free; DISK_GEOMETRY_EX info; HANDLE handle; + DWORD bytes_returned; free.QuadPart = 512 * 1024 * 1024; GetDiskFreeSpaceExW( dir, NULL, NULL, &free ); @@ -1485,7 +1486,7 @@ handle = CreateFileW( root, GENERIC_READ, FILE_SHARE_READ|FILE_SHARE_WRITE, NULL, OPEN_EXISTING, 0, 0 ); if (handle != INVALID_HANDLE_VALUE) { - if (DeviceIoControl( handle, IOCTL_DISK_GET_DRIVE_GEOMETRY_EX, NULL, 0, &info, sizeof(info), NULL, NULL )) + if (DeviceIoControl( handle, IOCTL_DISK_GET_DRIVE_GEOMETRY_EX, NULL, 0, &info, sizeof(info), &bytes_returned, NULL )) *disksize = info.DiskSize.QuadPart; CloseHandle( handle ); } Modified: trunk/rostests/winetests/wbemprox/query.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/wbemprox/query.… ============================================================================== --- trunk/rostests/winetests/wbemprox/query.c [iso-8859-1] (original) +++ trunk/rostests/winetests/wbemprox/query.c [iso-8859-1] Wed Jul 23 18:50:09 2014 @@ -675,14 +675,7 @@ RPC_C_AUTHN_LEVEL_CALL, RPC_C_IMP_LEVEL_IMPERSONATE, NULL, EOAC_NONE ); ok( hr == S_OK, "failed to set proxy blanket %08x\n", hr ); - if (!winetest_interactive) - { - skip("test_select( services ), ROSTESTS-120\n"); - } - else - { - test_select( services ); - } + test_select( services ); test_Win32_Process( services ); test_Win32_Service( services ); test_StdRegProv( services );

1 0

[dquintana] 63732: [STOBJECT] * Create skeleton for the SysTray shell service object DLL. No tray icons are created yet. * Tranks Robert Naumann for providing the icon resources, and some file temp...
by dquintana＠svn.reactos.org 23 Jul '14

23 Jul '14

Author: dquintana Date: Wed Jul 23 18:00:36 2014 New Revision: 63732 URL: http://svn.reactos.org/svn/reactos?rev=63732&view=rev Log: [STOBJECT] * Create skeleton for the SysTray shell service object DLL. No tray icons are created yet. * Tranks Robert Naumann for providing the icon resources, and some file templates. Added: branches/shell-experiments/dll/win32/stobject/ (with props) branches/shell-experiments/dll/win32/stobject/CMakeLists.txt (with props) branches/shell-experiments/dll/win32/stobject/lang/ (with props) branches/shell-experiments/dll/win32/stobject/lang/de-DE.rc (with props) branches/shell-experiments/dll/win32/stobject/lang/en-US.rc (with props) branches/shell-experiments/dll/win32/stobject/precomp.h (with props) branches/shell-experiments/dll/win32/stobject/resource.h (with props) branches/shell-experiments/dll/win32/stobject/resources/ (with props) branches/shell-experiments/dll/win32/stobject/resources/1.ico (with props) branches/shell-experiments/dll/win32/stobject/resources/2.ico (with props) branches/shell-experiments/dll/win32/stobject/resources/3.ico (with props) branches/shell-experiments/dll/win32/stobject/resources/4.ico (with props) branches/shell-experiments/dll/win32/stobject/resources/rgs/ (with props) branches/shell-experiments/dll/win32/stobject/resources/rgs/systray.rgs (with props) branches/shell-experiments/dll/win32/stobject/stobject.cpp (with props) branches/shell-experiments/dll/win32/stobject/stobject.rc (with props) branches/shell-experiments/dll/win32/stobject/stobject.spec (with props) Modified: branches/shell-experiments/base/shell/rshell/CStartMenu.cpp branches/shell-experiments/dll/win32/CMakeLists.txt Modified: branches/shell-experiments/base/shell/rshell/CStartMenu.cpp URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/base/shell/rs… ============================================================================== --- branches/shell-experiments/base/shell/rshell/CStartMenu.cpp [iso-8859-1] (original) +++ branches/shell-experiments/base/shell/rshell/CStartMenu.cpp [iso-8859-1] Wed Jul 23 18:00:36 2014 @@ -412,6 +412,8 @@ return hr; hr = SHGetSpecialFolderLocation(NULL, CSIDL_PROGRAMS, &pidlProgramsAbsolute); + if (FAILED_UNEXPECTEDLY(hr)) + return hr; pidlPrograms = ILClone(ILFindLastID(pidlProgramsAbsolute)); ILFree(pidlProgramsAbsolute); Modified: branches/shell-experiments/dll/win32/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/CMa… ============================================================================== --- branches/shell-experiments/dll/win32/CMakeLists.txt [iso-8859-1] (original) +++ branches/shell-experiments/dll/win32/CMakeLists.txt [iso-8859-1] Wed Jul 23 18:00:36 2014 @@ -193,6 +193,7 @@ add_subdirectory(stdole2.tlb) add_subdirectory(stdole32.tlb) add_subdirectory(sti) +add_subdirectory(stobject) add_subdirectory(sxs) add_subdirectory(syssetup) add_subdirectory(t2embed) Propchange: branches/shell-experiments/dll/win32/stobject/ ------------------------------------------------------------------------------ --- bugtraq:logregex (added) +++ bugtraq:logregex Wed Jul 23 18:00:36 2014 @@ -0,0 +1,2 @@ +([Ii]ssue|[Bb]ug)s? #?(\d+)(,? ?#?(\d+))*(,? ?(and |or )?#?(\d+))? +(\d+) Propchange: branches/shell-experiments/dll/win32/stobject/ ------------------------------------------------------------------------------ bugtraq:message = See issue #%BUGID% for more details. Propchange: branches/shell-experiments/dll/win32/stobject/ ------------------------------------------------------------------------------ bugtraq:url = http://www.reactos.org/bugzilla/show_bug.cgi?id=%BUGID% Propchange: branches/shell-experiments/dll/win32/stobject/ ------------------------------------------------------------------------------ tsvn:logminsize = 10 Added: branches/shell-experiments/dll/win32/stobject/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/sto… ============================================================================== --- branches/shell-experiments/dll/win32/stobject/CMakeLists.txt (added) +++ branches/shell-experiments/dll/win32/stobject/CMakeLists.txt [iso-8859-1] Wed Jul 23 18:00:36 2014 @@ -0,0 +1,36 @@ + +project(SHELL) + +set_cpp(WITH_RUNTIME) + +if(NOT MSVC) + # HACK: this should be enabled globally! + add_compile_flags_language("-std=c++11" "CXX") +endif() + +include_directories( + ${REACTOS_SOURCE_DIR}/lib/atl + ${REACTOS_SOURCE_DIR}) + +spec2def(stobject.dll stobject.spec) + +add_library(stobject SHARED + stobject.cpp + stobject.rc + ${CMAKE_CURRENT_BINARY_DIR}/stobject.def) + +set_module_type(stobject win32dll UNICODE) +target_link_libraries(stobject uuid wine atlnew) + +add_importlibs(stobject + advapi32 + ole32 + gdi32 + user32 + comctl32 + shlwapi + msvcrt + kernel32 + ntdll) + +add_cd_file(TARGET stobject DESTINATION reactos/system32 FOR all) Propchange: branches/shell-experiments/dll/win32/stobject/CMakeLists.txt ------------------------------------------------------------------------------ svn:eol-style = native Propchange: branches/shell-experiments/dll/win32/stobject/lang/ ------------------------------------------------------------------------------ --- bugtraq:logregex (added) +++ bugtraq:logregex Wed Jul 23 18:00:36 2014 @@ -0,0 +1,2 @@ +([Ii]ssue|[Bb]ug)s? #?(\d+)(,? ?#?(\d+))*(,? ?(and |or )?#?(\d+))? +(\d+) Propchange: branches/shell-experiments/dll/win32/stobject/lang/ ------------------------------------------------------------------------------ bugtraq:message = See issue #%BUGID% for more details. Propchange: branches/shell-experiments/dll/win32/stobject/lang/ ------------------------------------------------------------------------------ bugtraq:url = http://www.reactos.org/bugzilla/show_bug.cgi?id=%BUGID% Propchange: branches/shell-experiments/dll/win32/stobject/lang/ ------------------------------------------------------------------------------ tsvn:logminsize = 10 Added: branches/shell-experiments/dll/win32/stobject/lang/de-DE.rc URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/sto… ============================================================================== --- branches/shell-experiments/dll/win32/stobject/lang/de-DE.rc (added) +++ branches/shell-experiments/dll/win32/stobject/lang/de-DE.rc [iso-8859-1] Wed Jul 23 18:00:36 2014 @@ -0,0 +1,2 @@ +LANGUAGE LANG_GERMAN, SUBLANG_NEUTRAL + Propchange: branches/shell-experiments/dll/win32/stobject/lang/de-DE.rc ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/shell-experiments/dll/win32/stobject/lang/en-US.rc URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/sto… ============================================================================== --- branches/shell-experiments/dll/win32/stobject/lang/en-US.rc (added) +++ branches/shell-experiments/dll/win32/stobject/lang/en-US.rc [iso-8859-1] Wed Jul 23 18:00:36 2014 @@ -0,0 +1 @@ +LANGUAGE LANG_ENGLISH, SUBLANG_ENGLISH_US Propchange: branches/shell-experiments/dll/win32/stobject/lang/en-US.rc ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/shell-experiments/dll/win32/stobject/precomp.h URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/sto… ============================================================================== --- branches/shell-experiments/dll/win32/stobject/precomp.h (added) +++ branches/shell-experiments/dll/win32/stobject/precomp.h [iso-8859-1] Wed Jul 23 18:00:36 2014 @@ -0,0 +1,75 @@ +#pragma once + +#define WIN32_NO_STATUS +#include <stdarg.h> +#include <tchar.h> + +#define COBJMACROS +#define WIN32_NO_STATUS +#define _INC_WINDOWS +#define COM_NO_WINDOWS_H +#define NTOS_MODE_USER + +#include <windef.h> +#include <winbase.h> +#include <winreg.h> +#include <winuser.h> +#include <wincon.h> +#include <ddeml.h> +#include <shlguid_undoc.h> +#include <shlwapi.h> +#include <shlobj.h> +#include <shlobj_undoc.h> +#include <shlwapi_undoc.h> +#include <tchar.h> +#include <strsafe.h> +#include <atlbase.h> +#include <atlcom.h> + +#include <wine/debug.h> +#include <wine/unicode.h> + +extern const GUID CLSID_SysTray; + +#include "resource.h" + +static __inline ULONG +Win32DbgPrint(const char *filename, int line, const char *lpFormat, ...) +{ + char szMsg[512]; + char *szMsgStart; + const char *fname; + va_list vl; + ULONG uRet; + + fname = strrchr(filename, '\\'); + if (fname == NULL) + { + fname = strrchr(filename, '/'); + } + + if (fname == NULL) + fname = filename; + else + fname++; + + szMsgStart = szMsg + sprintf(szMsg, "[%10lu] %s:%d: ", GetTickCount(), fname, line); + + va_start(vl, lpFormat); + uRet = (ULONG) vsprintf(szMsgStart, lpFormat, vl); + va_end(vl); + + OutputDebugStringA(szMsg); + + return uRet; +} + +#define DbgPrint(fmt, ...) \ + Win32DbgPrint(__FILE__, __LINE__, fmt, ##__VA_ARGS__) + +#if 1 +#define FAILED_UNEXPECTEDLY(hr) (FAILED(hr) && (DbgPrint("Unexpected failure %08x.\n", hr), TRUE)) +#else +#define FAILED_UNEXPECTEDLY(hr) FAILED(hr) +#endif + Propchange: branches/shell-experiments/dll/win32/stobject/precomp.h ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/shell-experiments/dll/win32/stobject/resource.h URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/sto… ============================================================================== --- branches/shell-experiments/dll/win32/stobject/resource.h (added) +++ branches/shell-experiments/dll/win32/stobject/resource.h [iso-8859-1] Wed Jul 23 18:00:36 2014 @@ -0,0 +1,8 @@ +#pragma once + +#define IDI_ICON1 200 +#define IDI_ICON2 210 +#define IDI_ICON3 230 +#define IDI_ICON4 231 + +#define IDR_SYSTRAY 11001 Propchange: branches/shell-experiments/dll/win32/stobject/resource.h ------------------------------------------------------------------------------ svn:eol-style = native Propchange: branches/shell-experiments/dll/win32/stobject/resources/ ------------------------------------------------------------------------------ --- bugtraq:logregex (added) +++ bugtraq:logregex Wed Jul 23 18:00:36 2014 @@ -0,0 +1,2 @@ +([Ii]ssue|[Bb]ug)s? #?(\d+)(,? ?#?(\d+))*(,? ?(and |or )?#?(\d+))? +(\d+) Propchange: branches/shell-experiments/dll/win32/stobject/resources/ ------------------------------------------------------------------------------ bugtraq:message = See issue #%BUGID% for more details. Propchange: branches/shell-experiments/dll/win32/stobject/resources/ ------------------------------------------------------------------------------ bugtraq:url = http://www.reactos.org/bugzilla/show_bug.cgi?id=%BUGID% Propchange: branches/shell-experiments/dll/win32/stobject/resources/ ------------------------------------------------------------------------------ tsvn:logminsize = 10 Added: branches/shell-experiments/dll/win32/stobject/resources/1.ico URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/sto… ============================================================================== Binary file - no diff available. Propchange: branches/shell-experiments/dll/win32/stobject/resources/1.ico ------------------------------------------------------------------------------ svn:mime-type = application/octet-stream Added: branches/shell-experiments/dll/win32/stobject/resources/2.ico URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/sto… ============================================================================== Binary file - no diff available. Propchange: branches/shell-experiments/dll/win32/stobject/resources/2.ico ------------------------------------------------------------------------------ svn:mime-type = application/octet-stream Added: branches/shell-experiments/dll/win32/stobject/resources/3.ico URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/sto… ============================================================================== Binary file - no diff available. Propchange: branches/shell-experiments/dll/win32/stobject/resources/3.ico ------------------------------------------------------------------------------ svn:mime-type = application/octet-stream Added: branches/shell-experiments/dll/win32/stobject/resources/4.ico URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/sto… ============================================================================== Binary file - no diff available. Propchange: branches/shell-experiments/dll/win32/stobject/resources/4.ico ------------------------------------------------------------------------------ svn:mime-type = application/octet-stream Propchange: branches/shell-experiments/dll/win32/stobject/resources/rgs/ ------------------------------------------------------------------------------ --- bugtraq:logregex (added) +++ bugtraq:logregex Wed Jul 23 18:00:36 2014 @@ -0,0 +1,2 @@ +([Ii]ssue|[Bb]ug)s? #?(\d+)(,? ?#?(\d+))*(,? ?(and |or )?#?(\d+))? +(\d+) Propchange: branches/shell-experiments/dll/win32/stobject/resources/rgs/ ------------------------------------------------------------------------------ bugtraq:message = See issue #%BUGID% for more details. Propchange: branches/shell-experiments/dll/win32/stobject/resources/rgs/ ------------------------------------------------------------------------------ bugtraq:url = http://www.reactos.org/bugzilla/show_bug.cgi?id=%BUGID% Propchange: branches/shell-experiments/dll/win32/stobject/resources/rgs/ ------------------------------------------------------------------------------ tsvn:logminsize = 10 Added: branches/shell-experiments/dll/win32/stobject/resources/rgs/systray.rgs URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/sto… ============================================================================== --- branches/shell-experiments/dll/win32/stobject/resources/rgs/systray.rgs (added) +++ branches/shell-experiments/dll/win32/stobject/resources/rgs/systray.rgs [iso-8859-1] Wed Jul 23 18:00:36 2014 @@ -0,0 +1,13 @@ +HKCR +{ + NoRemove CLSID + { + ForceRemove {35CEC8A3-2BE6-11D2-8773-92E220524153} = s 'SysTray' + { + InprocServer32 = s '%MODULE%' + { + val ThreadingModel = s 'Both' + } + } + } +} Propchange: branches/shell-experiments/dll/win32/stobject/resources/rgs/systray.rgs ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/shell-experiments/dll/win32/stobject/stobject.cpp URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/sto… ============================================================================== --- branches/shell-experiments/dll/win32/stobject/stobject.cpp (added) +++ branches/shell-experiments/dll/win32/stobject/stobject.cpp [iso-8859-1] Wed Jul 23 18:00:36 2014 @@ -0,0 +1,165 @@ +/* + * PROJECT: ReactOS system libraries + * LICENSE: GPL - See COPYING in the top level directory + * FILE: dll\win32\stobject\stobject.c + * PURPOSE: Systray shell service object + * PROGRAMMERS: Copyright 2014 Robert Naumann + */ + +#include "precomp.h" + +#include <olectl.h> + +WINE_DEFAULT_DEBUG_CHANNEL(stobject); + +const GUID CLSID_SysTray = { 0x35CEC8A3, 0x2BE6, 0x11D2, { 0x87, 0x73, 0x92, 0xE2, 0x20, 0x52, 0x41, 0x53 } }; + +class CShellTrayModule : public CComModule +{ +public: +}; + +class CSysTray : + public CComCoClass<CSysTray, &CLSID_SysTray>, + public CComObjectRootEx<CComMultiThreadModelNoCS>, + public IOleCommandTarget +{ + // TODO: keep icon handlers here + +public: + CSysTray() {} + virtual ~CSysTray() {} + + // *** IOleCommandTarget methods *** + virtual HRESULT STDMETHODCALLTYPE QueryStatus(const GUID *pguidCmdGroup, ULONG cCmds, OLECMD prgCmds [], OLECMDTEXT *pCmdText) + { + UNIMPLEMENTED; + return S_OK; + } + + virtual HRESULT STDMETHODCALLTYPE Exec(const GUID *pguidCmdGroup, DWORD nCmdID, DWORD nCmdexecopt, VARIANT *pvaIn, VARIANT *pvaOut) + { + if (!IsEqualGUID(*pguidCmdGroup, CGID_ShellServiceObject)) + return E_FAIL; + + switch (nCmdID) + { + case OLECMDID_NEW: // init + DbgPrint("CSysTray Init TODO: Initialize tray icon handlers.\n"); + break; + case OLECMDID_SAVE: // shutdown + DbgPrint("CSysTray Shutdown TODO: Shutdown.\n"); + break; + } + return S_OK; + } + + DECLARE_REGISTRY_RESOURCEID(IDR_SYSTRAY) + DECLARE_NOT_AGGREGATABLE(CSysTray) + DECLARE_PROTECT_FINAL_CONSTRUCT() + + BEGIN_COM_MAP(CSysTray) + COM_INTERFACE_ENTRY_IID(IID_IOleCommandTarget, IOleCommandTarget) + END_COM_MAP() +}; + + +BEGIN_OBJECT_MAP(ObjectMap) + OBJECT_ENTRY(CLSID_SysTray, CSysTray) +END_OBJECT_MAP() + +CShellTrayModule gModule; + +HINSTANCE g_hInstance; + +HRESULT RegisterShellServiceObject(REFGUID guidClass, LPCWSTR lpName, BOOL bRegister) +{ + const LPCWSTR strRegistryLocation = L"Software\\Microsoft\\Windows\\CurrentVersion\\ShellServiceObjectDelayLoad"; + + OLECHAR strGuid[128]; // shouldn't need so much! + LSTATUS ret = 0; + HKEY hKey = 0; + if (StringFromGUID2(guidClass, strGuid, 128)) + { + if (!RegOpenKeyExW(HKEY_LOCAL_MACHINE, strRegistryLocation, 0, KEY_WRITE, &hKey)) + { + if (bRegister) + { + LONG cbGuid = (lstrlenW(strGuid) + 1) * 2; + ret = RegSetValueExW(hKey, lpName, 0, REG_SZ, (const BYTE *) strGuid, cbGuid); + } + else + { + ret = RegDeleteValueW(hKey, lpName); + } + } + } + if (hKey) + RegCloseKey(hKey); + return /*HRESULT_FROM_NT*/(ret); // regsvr32 considers anything != S_OK to be an error + +} + +void *operator new (size_t, void *buf) +{ + return buf; +} + +BOOL +WINAPI +DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID fImpLoad) +{ + if (fdwReason == DLL_PROCESS_ATTACH) + { + /* HACK - the global constructors don't run, so I placement new them here */ + new (&gModule) CShellTrayModule; + new (&_AtlWinModule) CAtlWinModule; + new (&_AtlBaseModule) CAtlBaseModule; + new (&_AtlComModule) CAtlComModule; + + g_hInstance = hinstDLL; + DisableThreadLibraryCalls(g_hInstance); + + gModule.Init(ObjectMap, g_hInstance, NULL); + } + else if (fdwReason == DLL_PROCESS_DETACH) + { + g_hInstance = NULL; + gModule.Term(); + } + return TRUE; +} + +HRESULT +WINAPI +DllCanUnloadNow(void) +{ + return gModule.DllCanUnloadNow(); +} + +STDAPI +DllRegisterServer(void) +{ + HRESULT hr = gModule.DllRegisterServer(FALSE); + if (FAILED(hr)) + return hr; + + return RegisterShellServiceObject(CLSID_SysTray, L"SysTray", TRUE); +} + +STDAPI +DllUnregisterServer(void) +{ + RegisterShellServiceObject(CLSID_SysTray, L"SysTray", FALSE); + + return gModule.DllUnregisterServer(FALSE); +} + +STDAPI +DllGetClassObject( + REFCLSID rclsid, + REFIID riid, + LPVOID *ppv) +{ + return gModule.DllGetClassObject(rclsid, riid, ppv); +} Propchange: branches/shell-experiments/dll/win32/stobject/stobject.cpp ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/shell-experiments/dll/win32/stobject/stobject.rc URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/sto… ============================================================================== --- branches/shell-experiments/dll/win32/stobject/stobject.rc (added) +++ branches/shell-experiments/dll/win32/stobject/stobject.rc [iso-8859-1] Wed Jul 23 18:00:36 2014 @@ -0,0 +1,26 @@ +#include <windef.h> +#include <winuser.h> +#include <commctrl.h> + +#include "resource.h" + +LANGUAGE LANG_NEUTRAL, SUBLANG_NEUTRAL + +IDI_ICON1 ICON "resources/1.ico" +IDI_ICON2 ICON "resources/2.ico" +IDI_ICON3 ICON "resources/3.ico" +IDI_ICON4 ICON "resources/4.ico" + +IDR_SYSTRAY REGISTRY "resources/rgs/systray.rgs" + +/* UTF-8 */ +#pragma code_page(65001) + +#if 0 +#ifdef LANGUAGE_DE_DE + #include "lang/de-DE.rc" +#endif +#ifdef LANGUAGE_EN_US + #include "lang/en-US.rc" +#endif +#endif Propchange: branches/shell-experiments/dll/win32/stobject/stobject.rc ------------------------------------------------------------------------------ svn:eol-style = native Added: branches/shell-experiments/dll/win32/stobject/stobject.spec URL: http://svn.reactos.org/svn/reactos/branches/shell-experiments/dll/win32/sto… ============================================================================== --- branches/shell-experiments/dll/win32/stobject/stobject.spec (added) +++ branches/shell-experiments/dll/win32/stobject/stobject.spec [iso-8859-1] Wed Jul 23 18:00:36 2014 @@ -0,0 +1,4 @@ +@ stdcall -private DllCanUnloadNow() +@ stdcall -private DllGetClassObject(ptr ptr ptr) +@ stdcall -private DllRegisterServer() +@ stdcall -private DllUnregisterServer() Propchange: branches/shell-experiments/dll/win32/stobject/stobject.spec ------------------------------------------------------------------------------ svn:eol-style = native

1 0

[jgardou] 63731: [WIN32K] - Correctly use the Clip, Meta, Rao, API and Vis Regions in DCs - Update the DC clipping object on demand each time a blt is performed - Reduce the number of region alloca...
by jgardou＠svn.reactos.org 23 Jul '14

23 Jul '14

Author: jgardou Date: Wed Jul 23 16:05:47 2014 New Revision: 63731 URL: http://svn.reactos.org/svn/reactos?rev=63731&view=rev Log: [WIN32K] - Correctly use the Clip, Meta, Rao, API and Vis Regions in DCs - Update the DC clipping object on demand each time a blt is performed - Reduce the number of region allocated with handles when it's not needed This commit fixes numerous bugs regarding clipping, most notably SetDIBitsToDevice overlapping foreground windows CORE-8353 #resolve #comment There are still a few quirks in winetests which will get fixed in a next commit CORE-7628 #comment should be fixed in rev 63731, please confirm. Modified: trunk/reactos/win32ss/gdi/ntgdi/arc.c trunk/reactos/win32ss/gdi/ntgdi/bitblt.c trunk/reactos/win32ss/gdi/ntgdi/cliprgn.c trunk/reactos/win32ss/gdi/ntgdi/cliprgn.h trunk/reactos/win32ss/gdi/ntgdi/dc.h trunk/reactos/win32ss/gdi/ntgdi/dclife.c trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c trunk/reactos/win32ss/gdi/ntgdi/dcstate.c trunk/reactos/win32ss/gdi/ntgdi/dcutil.c trunk/reactos/win32ss/gdi/ntgdi/dibobj.c trunk/reactos/win32ss/gdi/ntgdi/fillshap.c trunk/reactos/win32ss/gdi/ntgdi/freetype.c trunk/reactos/win32ss/gdi/ntgdi/line.c trunk/reactos/win32ss/gdi/ntgdi/path.c trunk/reactos/win32ss/gdi/ntgdi/region.c trunk/reactos/win32ss/gdi/ntgdi/region.h trunk/reactos/win32ss/user/ntuser/cursoricon.c trunk/reactos/win32ss/user/ntuser/cursoricon_new.c trunk/reactos/win32ss/user/ntuser/dce.h trunk/reactos/win32ss/user/ntuser/windc.c Modified: trunk/reactos/win32ss/gdi/ntgdi/arc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/arc.c?re… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/arc.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/arc.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -335,8 +335,7 @@ worker.l = dwStartAngle; worker1.l = dwSweepAngle; - DC_vPrepareDCsForBlit(pDC, pDC->rosdc.CombinedClip->rclBounds, - NULL, pDC->rosdc.CombinedClip->rclBounds); + DC_vPrepareDCsForBlit(pDC, NULL, NULL, NULL); if (pDC->pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) DC_vUpdateFillBrush(pDC); if (pDC->pdcattr->ulDirty_ & (DIRTY_LINE | DC_PEN_DIRTY)) @@ -381,8 +380,7 @@ return TRUE; } - DC_vPrepareDCsForBlit(dc, dc->rosdc.CombinedClip->rclBounds, - NULL, dc->rosdc.CombinedClip->rclBounds); + DC_vPrepareDCsForBlit(dc, NULL, NULL, NULL); if (dc->pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) DC_vUpdateFillBrush(dc); Modified: trunk/reactos/win32ss/gdi/ntgdi/bitblt.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/bitblt.c… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/bitblt.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/bitblt.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -107,7 +107,7 @@ /* Prepare DCs for blit */ TRACE("Preparing DCs for blit\n"); - DC_vPrepareDCsForBlit(DCDest, DestRect, DCSrc, SourceRect); + DC_vPrepareDCsForBlit(DCDest, &DestRect, DCSrc, &SourceRect); /* Determine surfaces to be used in the bitblt */ BitmapDest = DCDest->dclevel.pSurface; @@ -268,7 +268,7 @@ rcSrc.bottom += DCSrc->ptlDCOrig.y; /* Prepare for blit */ - DC_vPrepareDCsForBlit(DCDest, rcDest, DCSrc, rcSrc); + DC_vPrepareDCsForBlit(DCDest, &rcDest, DCSrc, &rcSrc); BitmapDest = DCDest->dclevel.pSurface; if (!BitmapDest) @@ -456,7 +456,7 @@ } /* Prepare blit */ - DC_vPrepareDCsForBlit(DCDest, DestRect, DCSrc, SourceRect); + DC_vPrepareDCsForBlit(DCDest, &DestRect, DCSrc, &SourceRect); if (pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) DC_vUpdateFillBrush(DCDest); @@ -647,7 +647,7 @@ BrushOrigin.y = 0; /* Only prepare Source and Dest, hdcMask represents a DIB */ - DC_vPrepareDCsForBlit(DCDest, DestRect, DCSrc, SourceRect); + DC_vPrepareDCsForBlit(DCDest, &DestRect, DCSrc, &SourceRect); if (pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) DC_vUpdateFillBrush(DCDest); @@ -826,7 +826,7 @@ BrushOrigin.y = pbrush->ptOrigin.y + pdc->ptlDCOrig.y; #endif - DC_vPrepareDCsForBlit(pdc, DestRect, NULL, DestRect); + DC_vPrepareDCsForBlit(pdc, &DestRect, NULL, NULL); psurf = pdc->dclevel.pSurface; Modified: trunk/reactos/win32ss/gdi/ntgdi/cliprgn.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/cliprgn.… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/cliprgn.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/cliprgn.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -11,164 +11,102 @@ #define NDEBUG #include <debug.h> -int FASTCALL -CLIPPING_UpdateGCRegion(DC* Dc) -{ - PROSRGNDATA CombinedRegion; - //HRGN hRgnVis; - PREGION prgnClip, prgnGCClip; - - /* Would prefer this, but the rest of the code sucks... */ - //ASSERT(Dc->rosdc.hGCClipRgn); - //ASSERT(Dc->rosdc.hClipRgn); - ASSERT(Dc->prgnVis); - //hRgnVis = Dc->prgnVis->BaseObject.hHmgr; - - if (Dc->rosdc.hGCClipRgn == NULL) - Dc->rosdc.hGCClipRgn = IntSysCreateRectRgn(0, 0, 0, 0); - - prgnGCClip = REGION_LockRgn(Dc->rosdc.hGCClipRgn); - ASSERT(prgnGCClip); - - if (Dc->rosdc.hClipRgn == NULL) - IntGdiCombineRgn(prgnGCClip, Dc->prgnVis, NULL, RGN_COPY); - else - { - prgnClip = REGION_LockRgn(Dc->rosdc.hClipRgn); // FIXME: Locking order, ugh! - IntGdiCombineRgn(prgnGCClip, Dc->prgnVis, prgnClip, RGN_AND); - REGION_UnlockRgn(prgnClip); - } - REGION_UnlockRgn(prgnGCClip); - - NtGdiOffsetRgn(Dc->rosdc.hGCClipRgn, Dc->ptlDCOrig.x, Dc->ptlDCOrig.y); - - if((CombinedRegion = RGNOBJAPI_Lock(Dc->rosdc.hGCClipRgn, NULL))) - { - CLIPOBJ *CombinedClip; - - CombinedClip = IntEngCreateClipRegion(CombinedRegion->rdh.nCount, - CombinedRegion->Buffer, - &CombinedRegion->rdh.rcBound); - - RGNOBJAPI_Unlock(CombinedRegion); - - if ( !CombinedClip ) - { - DPRINT1("IntEngCreateClipRegion() failed\n"); - return ERROR; - } - - if(Dc->rosdc.CombinedClip != NULL) - IntEngDeleteClipRegion(Dc->rosdc.CombinedClip); - - Dc->rosdc.CombinedClip = CombinedClip ; - } - - return NtGdiOffsetRgn(Dc->rosdc.hGCClipRgn, -Dc->ptlDCOrig.x, -Dc->ptlDCOrig.y); -} - -INT FASTCALL -GdiSelectVisRgn(HDC hdc, HRGN hrgn) -{ - int retval; - DC *dc; - PREGION prgn; - - if (!hrgn) - { - EngSetLastError(ERROR_INVALID_PARAMETER); - return ERROR; - } - if (!(dc = DC_LockDc(hdc))) - { - EngSetLastError(ERROR_INVALID_HANDLE); - return ERROR; - } - - dc->fs &= ~DC_FLAG_DIRTY_RAO; - - ASSERT (dc->prgnVis != NULL); - - prgn = RGNOBJAPI_Lock(hrgn, NULL); - retval = prgn ? IntGdiCombineRgn(dc->prgnVis, prgn, NULL, RGN_COPY) : ERROR; - RGNOBJAPI_Unlock(prgn); - if ( retval != ERROR ) - { +VOID +FASTCALL +GdiSelectVisRgn( + HDC hdc, + PREGION prgn) +{ + DC *dc; + + if (!(dc = DC_LockDc(hdc))) + { + EngSetLastError(ERROR_INVALID_HANDLE); + return; + } + + dc->fs |= DC_FLAG_DIRTY_RAO; + + ASSERT(dc->prgnVis != NULL); + ASSERT(prgn != NULL); + + IntGdiCombineRgn(dc->prgnVis, prgn, NULL, RGN_COPY); IntGdiOffsetRgn(dc->prgnVis, -dc->ptlDCOrig.x, -dc->ptlDCOrig.y); - CLIPPING_UpdateGCRegion(dc); - } - DC_UnlockDc(dc); - - return retval; -} - - -int FASTCALL GdiExtSelectClipRgn(PDC dc, - HRGN hrgn, - int fnMode) -{ - // dc->fs &= ~DC_FLAG_DIRTY_RAO; - - if (!hrgn) - { - if (fnMode == RGN_COPY) - { - if (dc->rosdc.hClipRgn != NULL) - { - GreDeleteObject(dc->rosdc.hClipRgn); - dc->rosdc.hClipRgn = NULL; - } - } - else - { - EngSetLastError(ERROR_INVALID_PARAMETER); - return ERROR; - } - } - else - { - if (!dc->rosdc.hClipRgn) - { - RECTL rect; - if(dc->prgnVis) - { + + DC_UnlockDc(dc); +} + + +int +FASTCALL +IntGdiExtSelectClipRgn( + PDC dc, + PREGION prgn, + int fnMode) +{ + if (!prgn) + { + if (fnMode == RGN_COPY) + { + if (dc->dclevel.prgnClip != NULL) + { + REGION_Delete(dc->dclevel.prgnClip); + dc->dclevel.prgnClip = NULL; + dc->fs |= DC_FLAG_DIRTY_RAO; + } + return SIMPLEREGION; + } + else + { + EngSetLastError(ERROR_INVALID_PARAMETER); + return ERROR; + } + } + + if (!dc->dclevel.prgnClip) + { + RECTL rect; + REGION_GetRgnBox(dc->prgnVis, &rect); - dc->rosdc.hClipRgn = IntSysCreateRectRgnIndirect(&rect); - } - else - { - dc->rosdc.hClipRgn = IntSysCreateRectRgn(0, 0, 0, 0); - } - } + dc->dclevel.prgnClip = IntSysCreateRectpRgnIndirect(&rect); + } + + dc->fs |= DC_FLAG_DIRTY_RAO; + if(fnMode == RGN_COPY) - { - NtGdiCombineRgn(dc->rosdc.hClipRgn, hrgn, 0, fnMode); - } - else - NtGdiCombineRgn(dc->rosdc.hClipRgn, dc->rosdc.hClipRgn, hrgn, fnMode); - } - - return CLIPPING_UpdateGCRegion(dc); -} - - -int APIENTRY NtGdiExtSelectClipRgn(HDC hDC, - HRGN hrgn, - int fnMode) -{ - int retval; - DC *dc; - - if (!(dc = DC_LockDc(hDC))) - { - EngSetLastError(ERROR_INVALID_HANDLE); - return ERROR; - } - - retval = GdiExtSelectClipRgn ( dc, hrgn, fnMode ); - - DC_UnlockDc(dc); - return retval; + return IntGdiCombineRgn(dc->dclevel.prgnClip, prgn, 0, fnMode); + + return IntGdiCombineRgn(dc->dclevel.prgnClip, dc->dclevel.prgnClip, prgn, fnMode); +} + + +int +APIENTRY +NtGdiExtSelectClipRgn( + HDC hDC, + HRGN hrgn, + int fnMode) +{ + int retval; + DC *dc; + PREGION prgn; + + if (!(dc = DC_LockDc(hDC))) + { + EngSetLastError(ERROR_INVALID_HANDLE); + return ERROR; + } + + prgn = REGION_LockRgn(hrgn); + + /* IntGdiExtSelectClipRgn takes care of checking for NULL region */ + retval = IntGdiExtSelectClipRgn(dc, prgn, fnMode); + + if (prgn) + REGION_UnlockRgn(prgn); + + DC_UnlockDc(dc); + return retval; } INT FASTCALL @@ -177,7 +115,6 @@ INT retval; PDC dc; PROSRGNDATA pRgnNew, pRgn = NULL; - BOOL Unlock = FALSE; // Small HACK if (!(dc = DC_LockDc(hDC))) { @@ -193,10 +130,9 @@ { pRgn = dc->dclevel.prgnMeta; } - else if (dc->rosdc.hClipRgn) + else if (dc->dclevel.prgnClip) // CLIPRGN { - Unlock = TRUE ; - pRgn = REGION_LockRgn(dc->rosdc.hClipRgn); // CLIPRGN + pRgn = dc->dclevel.prgnClip; } if (pRgn) @@ -206,7 +142,6 @@ if (!pRgnNew) { DC_UnlockDc(dc); - if(Unlock) REGION_UnlockRgn(pRgn); return ERROR; } @@ -217,12 +152,11 @@ REGION_Delete(pRgnNew); DC_UnlockDc(dc); - if(Unlock) REGION_UnlockRgn(pRgn); return retval; } retval = REGION_GetRgnBox(dc->prgnVis, rc); - IntDPtoLP(dc, (LPPOINT)rc, 2); + DC_UnlockDc(dc); return retval; @@ -265,53 +199,49 @@ int RightRect, int BottomRect) { - INT Result; - RECTL Rect; - PREGION prgnNew, prgnClip; - PDC dc = DC_LockDc(hDC); - - if (!dc) - { - EngSetLastError(ERROR_INVALID_HANDLE); - return ERROR; - } - - Rect.left = LeftRect; - Rect.top = TopRect; - Rect.right = RightRect; - Rect.bottom = BottomRect; - - IntLPtoDP(dc, (LPPOINT)&Rect, 2); - - prgnNew = IntSysCreateRectpRgnIndirect(&Rect); - if (!prgnNew) - { - Result = ERROR; - } - else - { - if (!dc->rosdc.hClipRgn) - { - dc->rosdc.hClipRgn = IntSysCreateRectRgn(0, 0, 0, 0); - prgnClip = REGION_LockRgn(dc->rosdc.hClipRgn); - IntGdiCombineRgn(prgnClip, dc->prgnVis, prgnNew, RGN_DIFF); - REGION_UnlockRgn(prgnClip); - Result = SIMPLEREGION; - } - else - { - prgnClip = REGION_LockRgn(dc->rosdc.hClipRgn); - Result = IntGdiCombineRgn(prgnClip, prgnClip, prgnNew, RGN_DIFF); - REGION_UnlockRgn(prgnClip); - } - REGION_Delete(prgnNew); - } - if (Result != ERROR) - CLIPPING_UpdateGCRegion(dc); - - DC_UnlockDc(dc); - - return Result; + INT Result; + RECTL Rect; + PREGION prgnNew; + PDC dc = DC_LockDc(hDC); + + if (!dc) + { + EngSetLastError(ERROR_INVALID_HANDLE); + return ERROR; + } + + Rect.left = LeftRect; + Rect.top = TopRect; + Rect.right = RightRect; + Rect.bottom = BottomRect; + + IntLPtoDP(dc, (LPPOINT)&Rect, 2); + + prgnNew = IntSysCreateRectpRgnIndirect(&Rect); + if (!prgnNew) + { + Result = ERROR; + } + else + { + if (!dc->dclevel.prgnClip) + { + dc->dclevel.prgnClip = IntSysCreateRectpRgn(0, 0, 0, 0); + IntGdiCombineRgn(dc->dclevel.prgnClip, dc->prgnVis, prgnNew, RGN_DIFF); + Result = SIMPLEREGION; + } + else + { + Result = IntGdiCombineRgn(dc->dclevel.prgnClip, dc->dclevel.prgnClip, prgnNew, RGN_DIFF); + } + REGION_Delete(prgnNew); + } + if (Result != ERROR) + dc->fs |= DC_FLAG_DIRTY_RAO; + + DC_UnlockDc(dc); + + return Result; } int APIENTRY NtGdiIntersectClipRect(HDC hDC, @@ -320,200 +250,191 @@ int RightRect, int BottomRect) { - INT Result; - RECTL Rect; - HRGN NewRgn; - PDC dc = DC_LockDc(hDC); - - DPRINT("NtGdiIntersectClipRect(%p, %d,%d-%d,%d)\n", - hDC, LeftRect, TopRect, RightRect, BottomRect); - - if (!dc) - { - EngSetLastError(ERROR_INVALID_HANDLE); - return ERROR; - } - - Rect.left = LeftRect; - Rect.top = TopRect; - Rect.right = RightRect; - Rect.bottom = BottomRect; - - IntLPtoDP(dc, (LPPOINT)&Rect, 2); - - NewRgn = IntSysCreateRectRgnIndirect(&Rect); - if (!NewRgn) - { - Result = ERROR; - } - else if (!dc->rosdc.hClipRgn) - { - dc->rosdc.hClipRgn = NewRgn; - Result = SIMPLEREGION; - } - else - { - Result = NtGdiCombineRgn(dc->rosdc.hClipRgn, dc->rosdc.hClipRgn, NewRgn, RGN_AND); - GreDeleteObject(NewRgn); - } - if (Result != ERROR) - CLIPPING_UpdateGCRegion(dc); - - DC_UnlockDc(dc); - - return Result; + INT Result; + RECTL Rect; + PREGION pNewRgn; + PDC dc = DC_LockDc(hDC); + + DPRINT("NtGdiIntersectClipRect(%p, %d,%d-%d,%d)\n", + hDC, LeftRect, TopRect, RightRect, BottomRect); + + if (!dc) + { + EngSetLastError(ERROR_INVALID_HANDLE); + return ERROR; + } + + Rect.left = LeftRect; + Rect.top = TopRect; + Rect.right = RightRect; + Rect.bottom = BottomRect; + + IntLPtoDP(dc, (LPPOINT)&Rect, 2); + + pNewRgn = IntSysCreateRectpRgnIndirect(&Rect); + if (!pNewRgn) + { + Result = ERROR; + } + else if (!dc->dclevel.prgnClip) + { + dc->dclevel.prgnClip = pNewRgn; + Result = SIMPLEREGION; + } + else + { + Result = IntGdiCombineRgn(dc->dclevel.prgnClip, dc->dclevel.prgnClip, pNewRgn, RGN_AND); + REGION_Delete(pNewRgn); + } + if (Result != ERROR) + dc->fs |= DC_FLAG_DIRTY_RAO; + + DC_UnlockDc(dc); + + return Result; } int APIENTRY NtGdiOffsetClipRgn(HDC hDC, int XOffset, int YOffset) { - INT Result; - DC *dc; - - if(!(dc = DC_LockDc(hDC))) - { - EngSetLastError(ERROR_INVALID_HANDLE); - return ERROR; - } - - if(dc->rosdc.hClipRgn != NULL) - { - Result = NtGdiOffsetRgn(dc->rosdc.hClipRgn, - XOffset, - YOffset); - CLIPPING_UpdateGCRegion(dc); - } - else - { - Result = NULLREGION; - } - - DC_UnlockDc(dc); - return Result; + INT Result; + DC *dc; + + if(!(dc = DC_LockDc(hDC))) + { + EngSetLastError(ERROR_INVALID_HANDLE); + return ERROR; + } + + if(dc->dclevel.prgnClip != NULL) + { + Result = IntGdiOffsetRgn(dc->dclevel.prgnClip, + XOffset, + YOffset); + dc->fs |= DC_FLAG_DIRTY_RAO; + } + else + { + Result = NULLREGION; + } + + DC_UnlockDc(dc); + return Result; } BOOL APIENTRY NtGdiPtVisible(HDC hDC, int X, int Y) { - HRGN rgn; - DC *dc; - - if(!(dc = DC_LockDc(hDC))) - { - EngSetLastError(ERROR_INVALID_HANDLE); - return FALSE; - } - - rgn = dc->rosdc.hGCClipRgn; - DC_UnlockDc(dc); - - return (rgn ? NtGdiPtInRegion(rgn, X, Y) : FALSE); -} - -BOOL APIENTRY NtGdiRectVisible(HDC hDC, - LPRECT UnsafeRect) -{ - NTSTATUS Status = STATUS_SUCCESS; - PROSRGNDATA Rgn; - PDC dc = DC_LockDc(hDC); - BOOL Result = FALSE; - RECTL Rect; - - if (!dc) - { - EngSetLastError(ERROR_INVALID_HANDLE); - return FALSE; - } - - _SEH2_TRY - { - ProbeForRead(UnsafeRect, + BOOL ret = FALSE; + PDC dc; + + if(!(dc = DC_LockDc(hDC))) + { + EngSetLastError(ERROR_INVALID_HANDLE); + return FALSE; + } + + if (dc->prgnRao) + { + POINT pt = {X, Y}; + IntLPtoDP(dc, &pt, 1); + ret = REGION_PtInRegion(dc->prgnRao, pt.x, pt.y); + } + + DC_UnlockDc(dc); + + return ret; +} + +BOOL +APIENTRY +NtGdiRectVisible( + HDC hDC, + LPRECT UnsafeRect) +{ + NTSTATUS Status = STATUS_SUCCESS; + PDC dc = DC_LockDc(hDC); + BOOL Result = FALSE; + RECTL Rect; + + if (!dc) + { + EngSetLastError(ERROR_INVALID_HANDLE); + return FALSE; + } + + _SEH2_TRY + { + ProbeForRead(UnsafeRect, sizeof(RECT), 1); - Rect = *UnsafeRect; - } - _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) - { - Status = _SEH2_GetExceptionCode(); - } - _SEH2_END; - - if(!NT_SUCCESS(Status)) - { - DC_UnlockDc(dc); - SetLastNtError(Status); - return FALSE; - } - - if (dc->rosdc.hGCClipRgn) - { - if((Rgn = (PROSRGNDATA)RGNOBJAPI_Lock(dc->rosdc.hGCClipRgn, NULL))) - { + Rect = *UnsafeRect; + } + _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) + { + Status = _SEH2_GetExceptionCode(); + } + _SEH2_END; + + if(!NT_SUCCESS(Status)) + { + DC_UnlockDc(dc); + SetLastNtError(Status); + return FALSE; + } + + if (dc->fs & DC_FLAG_DIRTY_RAO) + CLIPPING_UpdateGCRegion(dc); + + if (dc->prgnRao) + { IntLPtoDP(dc, (LPPOINT)&Rect, 2); - Result = REGION_RectInRegion(Rgn, &Rect); - RGNOBJAPI_Unlock(Rgn); - } - } - DC_UnlockDc(dc); - - return Result; + Result = REGION_RectInRegion(dc->prgnRao, &Rect); + } + DC_UnlockDc(dc); + + return Result; } int FASTCALL IntGdiSetMetaRgn(PDC pDC) { - INT Ret = ERROR; - PROSRGNDATA TempRgn; - - if ( pDC->dclevel.prgnMeta ) - { - if ( pDC->dclevel.prgnClip ) - { - TempRgn = IntSysCreateRectpRgn(0,0,0,0); - if (TempRgn) - { - Ret = IntGdiCombineRgn( TempRgn, - pDC->dclevel.prgnMeta, - pDC->dclevel.prgnClip, - RGN_AND); - if ( Ret ) - { - GDIOBJ_vDereferenceObject(&pDC->dclevel.prgnMeta->BaseObject); - if (!((PROSRGNDATA)pDC->dclevel.prgnMeta)->BaseObject.ulShareCount) - REGION_Delete(pDC->dclevel.prgnMeta); - - pDC->dclevel.prgnMeta = TempRgn; - - GDIOBJ_vDereferenceObject(&pDC->dclevel.prgnClip->BaseObject); - if (!((PROSRGNDATA)pDC->dclevel.prgnClip)->BaseObject.ulShareCount) - REGION_Delete(pDC->dclevel.prgnClip); - - pDC->dclevel.prgnClip = NULL; - - IntGdiReleaseRaoRgn(pDC); - } - else - REGION_Delete(TempRgn); - } - } - else - Ret = REGION_Complexity(pDC->dclevel.prgnMeta); - } - else - { - if ( pDC->dclevel.prgnClip ) - { - Ret = REGION_Complexity(pDC->dclevel.prgnClip); - pDC->dclevel.prgnMeta = pDC->dclevel.prgnClip; - pDC->dclevel.prgnClip = NULL; - } - else - Ret = SIMPLEREGION; - } - return Ret; + INT Ret = ERROR; + + if ( pDC->dclevel.prgnMeta ) + { + if ( pDC->dclevel.prgnClip ) + { + Ret = IntGdiCombineRgn(pDC->dclevel.prgnMeta, pDC->dclevel.prgnMeta, pDC->dclevel.prgnClip, RGN_AND); + if (Ret != ERROR) + { + REGION_Delete(pDC->dclevel.prgnClip); + pDC->dclevel.prgnClip = NULL; + IntGdiReleaseRaoRgn(pDC); + } + } + else + Ret = REGION_Complexity(pDC->dclevel.prgnMeta); + } + else + { + if ( pDC->dclevel.prgnClip ) + { + Ret = REGION_Complexity(pDC->dclevel.prgnClip); + pDC->dclevel.prgnMeta = pDC->dclevel.prgnClip; + pDC->dclevel.prgnClip = NULL; + } + else + Ret = SIMPLEREGION; + } + + if (Ret != ERROR) + pDC->fs |= DC_FLAG_DIRTY_RAO; + + return Ret; } @@ -533,88 +454,95 @@ return Ret; } -INT FASTCALL -NEW_CLIPPING_UpdateGCRegion(PDC pDC) -{ - CLIPOBJ * co; - - /* Must have VisRgn set to a valid state! */ - ASSERT (pDC->prgnVis); - -// FIXME: this seems to be broken! - - if (pDC->prgnAPI) - { - REGION_Delete(pDC->prgnAPI); - pDC->prgnAPI = IntSysCreateRectpRgn(0,0,0,0); - } - - if (pDC->prgnRao) - { - REGION_Delete(pDC->prgnRao); - pDC->prgnRao = IntSysCreateRectpRgn(0,0,0,0); - } - - if (!pDC->prgnRao) - { - return ERROR; - } - - if (pDC->dclevel.prgnMeta && pDC->dclevel.prgnClip) - { - IntGdiCombineRgn( pDC->prgnAPI, - pDC->dclevel.prgnClip, - pDC->dclevel.prgnMeta, - RGN_AND); - } - else - { - if (pDC->dclevel.prgnClip) - { - IntGdiCombineRgn( pDC->prgnAPI, - pDC->dclevel.prgnClip, - NULL, - RGN_COPY); - } - else if (pDC->dclevel.prgnMeta) - { - IntGdiCombineRgn( pDC->prgnAPI, - pDC->dclevel.prgnMeta, - NULL, - RGN_COPY); - } - } - - IntGdiCombineRgn( pDC->prgnRao, - pDC->prgnVis, - pDC->prgnAPI, - RGN_AND); - - RtlCopyMemory(&pDC->erclClip, +VOID +FASTCALL +CLIPPING_UpdateGCRegion(PDC pDC) +{ + CLIPOBJ * co; + + /* Must have VisRgn set to a valid state! */ + ASSERT (pDC->prgnVis); + + if (pDC->prgnAPI) + { + REGION_Delete(pDC->prgnAPI); + pDC->prgnAPI = NULL; + } + + if (pDC->prgnRao) + REGION_Delete(pDC->prgnRao); + + pDC->prgnRao = IntSysCreateRectpRgn(0,0,0,0); + + ASSERT(pDC->prgnRao); + + if (pDC->dclevel.prgnMeta || pDC->dclevel.prgnClip) + { + pDC->prgnAPI = IntSysCreateRectpRgn(0,0,0,0); + if (!pDC->dclevel.prgnMeta) + { + IntGdiCombineRgn(pDC->prgnAPI, + pDC->dclevel.prgnClip, + NULL, + RGN_COPY); + } + else if (!pDC->dclevel.prgnClip) + { + IntGdiCombineRgn(pDC->prgnAPI, + pDC->dclevel.prgnMeta, + NULL, + RGN_COPY); + } + else + { + IntGdiCombineRgn(pDC->prgnAPI, + pDC->dclevel.prgnClip, + pDC->dclevel.prgnMeta, + RGN_AND); + } + } + + if (pDC->prgnAPI) + { + IntGdiCombineRgn(pDC->prgnRao, + pDC->prgnVis, + pDC->prgnAPI, + RGN_AND); + } + else + { + IntGdiCombineRgn(pDC->prgnRao, + pDC->prgnVis, + NULL, + RGN_COPY); + } + + + IntGdiOffsetRgn(pDC->prgnRao, pDC->ptlDCOrig.x, pDC->ptlDCOrig.y); + + RtlCopyMemory(&pDC->erclClip, &pDC->prgnRao->rdh.rcBound, sizeof(RECTL)); - pDC->fs &= ~DC_FLAG_DIRTY_RAO; - - IntGdiOffsetRgn(pDC->prgnRao, pDC->ptlDCOrig.x, pDC->ptlDCOrig.y); - - // pDC->co should be used. Example, CLIPOBJ_cEnumStart uses XCLIPOBJ to build - // the rects from region objects rects in pClipRgn->Buffer. - // With pDC->co.pClipRgn->Buffer, - // pDC->co.pClipRgn = pDC->prgnRao ? pDC->prgnRao : pDC->prgnVis; - - co = IntEngCreateClipRegion(pDC->prgnRao->rdh.nCount, - pDC->prgnRao->Buffer, - &pDC->erclClip); - if (co) - { - if (pDC->rosdc.CombinedClip != NULL) - IntEngDeleteClipRegion(pDC->rosdc.CombinedClip); - - pDC->rosdc.CombinedClip = co; - } - - return IntGdiOffsetRgn(pDC->prgnRao, -pDC->ptlDCOrig.x, -pDC->ptlDCOrig.y); + pDC->fs &= ~DC_FLAG_DIRTY_RAO; + + // pDC->co should be used. Example, CLIPOBJ_cEnumStart uses XCLIPOBJ to build + // the rects from region objects rects in pClipRgn->Buffer. + // With pDC->co.pClipRgn->Buffer, + // pDC->co.pClipRgn = pDC->prgnRao ? pDC->prgnRao : pDC->prgnVis; + + co = IntEngCreateClipRegion(pDC->prgnRao->rdh.nCount, + pDC->prgnRao->Buffer, + &pDC->erclClip); + if (co) + { + if (pDC->rosdc.CombinedClip != NULL) + IntEngDeleteClipRegion(pDC->rosdc.CombinedClip); + + pDC->rosdc.CombinedClip = co; + } + + IntGdiOffsetRgn(pDC->prgnRao, -pDC->ptlDCOrig.x, -pDC->ptlDCOrig.y); } /* EOF */ Modified: trunk/reactos/win32ss/gdi/ntgdi/cliprgn.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/cliprgn.… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/cliprgn.h [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/cliprgn.h [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -1,6 +1,6 @@ #pragma once INT FASTCALL GdiGetClipBox(HDC hDC, RECTL *rc); -INT FASTCALL GdiSelectVisRgn(HDC hdc, HRGN hrgn); -INT FASTCALL GdiExtSelectClipRgn (PDC dc, HRGN hrgn, int fnMode); -int FASTCALL CLIPPING_UpdateGCRegion(DC* Dc); +VOID FASTCALL GdiSelectVisRgn(HDC hdc, PREGION prgn); +INT FASTCALL IntGdiExtSelectClipRgn (PDC dc, PREGION prgn, int fnMode); +VOID FASTCALL CLIPPING_UpdateGCRegion(DC* Dc); Modified: trunk/reactos/win32ss/gdi/ntgdi/dc.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dc.h?rev… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/dc.h [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/dc.h [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -48,9 +48,6 @@ typedef struct _ROS_DC_INFO { - HRGN hClipRgn; /* Clip region (may be 0) */ - HRGN hGCClipRgn; /* GC clip region (ClipRgn AND VisRgn) */ - CLIPOBJ *CombinedClip; } ROS_DC_INFO; @@ -199,9 +196,9 @@ BOOL FASTCALL DC_InvertXform(const XFORM *xformSrc, XFORM *xformDest); VOID FASTCALL DC_vUpdateViewportExt(PDC pdc); -VOID FASTCALL DC_vCopyState(PDC pdcSrc, PDC pdcDst, BOOL to); +VOID FASTCALL DC_vCopyState(PDC pdcSrc, PDC pdcDst, BOOL To); VOID FASTCALL DC_vFinishBlit(PDC pdc1, PDC pdc2); -VOID FASTCALL DC_vPrepareDCsForBlit(PDC pdc1, RECT rc1, PDC pdc2, RECT rc2); +VOID FASTCALL DC_vPrepareDCsForBlit(PDC pdcDest, const RECT* rcDest, PDC pdcSrc, const RECT* rcSrc); VOID NTAPI DC_vRestoreDC(IN PDC pdc, INT iSaveLevel); Modified: trunk/reactos/win32ss/gdi/ntgdi/dclife.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dclife.c… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/dclife.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/dclife.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -95,22 +95,11 @@ void DC_InitHack(PDC pdc) { - HRGN hVisRgn; - TextIntRealizeFont(pdc->pdcattr->hlfntNew,NULL); pdc->pdcattr->iCS_CP = ftGdiGetTextCharsetInfo(pdc,NULL,0); /* This should never fail */ ASSERT(pdc->dclevel.ppal); - - /* Select regions */ - pdc->rosdc.hClipRgn = NULL; - pdc->rosdc.hGCClipRgn = NULL; - - hVisRgn = IntSysCreateRectRgn(0, 0, 1, 1); - ASSERT(hVisRgn); - GdiSelectVisRgn(pdc->BaseObject.hHmgr, hVisRgn); - GreDeleteObject(hVisRgn); } VOID @@ -170,7 +159,7 @@ pdc->erclBoundsApp.right = 0x00007ffc; // FIXME pdc->erclBoundsApp.bottom = 0x00000333; // FIXME pdc->erclClip = pdc->erclBounds; -// pdc->co + pdc->co = gxcoTrivial; pdc->fs |= DC_SYNCHRONIZEACCESS | DC_ACCUM_APP | DC_PERMANANT | DC_DISPLAY; } @@ -185,7 +174,7 @@ pdc->erclBounds.bottom = 0; pdc->erclBoundsApp = pdc->erclBounds; pdc->erclClip = pdc->erclWindow; - //pdc->co = NULL + pdc->co = gxcoTrivial; } //pdc->dcattr.VisRectRegion: @@ -237,6 +226,8 @@ /* Setup regions */ pdc->prgnAPI = NULL; pdc->prgnRao = NULL; + pdc->dclevel.prgnClip = NULL; + pdc->dclevel.prgnMeta = NULL; /* Allocate a Vis region */ pdc->prgnVis = IntSysCreateRectpRgn(0, 0, pdc->dclevel.sizl.cx, pdc->dclevel.sizl.cy); ASSERT(pdc->prgnVis); @@ -372,17 +363,17 @@ LFONT_ShareUnlockFont(pdc->dclevel.plfnt); /* Free regions */ - if (pdc->rosdc.hClipRgn && GreIsHandleValid(pdc->rosdc.hClipRgn)) - GreDeleteObject(pdc->rosdc.hClipRgn); + if (pdc->dclevel.prgnClip) + REGION_Delete(pdc->dclevel.prgnClip); + if (pdc->dclevel.prgnMeta) + REGION_Delete(pdc->dclevel.prgnMeta); if (pdc->prgnVis) - { REGION_Delete(pdc->prgnVis); - } - if (pdc->rosdc.hGCClipRgn && GreIsHandleValid(pdc->rosdc.hGCClipRgn)) - { - GreDeleteObject(pdc->rosdc.hGCClipRgn); - } - if (NULL != pdc->rosdc.CombinedClip) + if (pdc->prgnRao) + REGION_Delete(pdc->prgnRao); + if (pdc->prgnAPI) + REGION_Delete(pdc->prgnAPI); + if (pdc->rosdc.CombinedClip) IntEngDeleteClipRegion(pdc->rosdc.CombinedClip); PATH_Delete(pdc->dclevel.hPath); @@ -397,17 +388,6 @@ NTAPI DC_vSetOwner(PDC pdc, ULONG ulOwner) { - - if (pdc->rosdc.hClipRgn) - { - IntGdiSetRegionOwner(pdc->rosdc.hClipRgn, ulOwner); - } - - if (pdc->rosdc.hGCClipRgn) - { - IntGdiSetRegionOwner(pdc->rosdc.hGCClipRgn, ulOwner); - } - if (pdc->dclevel.hPath) { GreSetObjectOwner(pdc->dclevel.hPath, ulOwner); @@ -466,7 +446,7 @@ void DC_vUpdateDC(PDC pdc) { - HRGN hVisRgn ; + // PREGION VisRgn ; PPDEVOBJ ppdev = pdc->ppdev ; pdc->dhpdev = ppdev->dhpdev; @@ -475,10 +455,12 @@ pdc->dclevel.pSurface = PDEVOBJ_pSurface(ppdev); PDEVOBJ_sizl(pdc->ppdev, &pdc->dclevel.sizl); - hVisRgn = NtGdiCreateRectRgn(0, 0, pdc->dclevel.sizl.cx, pdc->dclevel.sizl.cy); - ASSERT(hVisRgn); - GdiSelectVisRgn(pdc->BaseObject.hHmgr, hVisRgn); - GreDeleteObject(hVisRgn); +#if 0 + VisRgn = IntSysCreateRectpRgn(0, 0, pdc->dclevel.sizl.cx, pdc->dclevel.sizl.cy); + ASSERT(VisRgn); + GdiSelectVisRgn(pdc->BaseObject.hHmgr, VisRgn); + REGION_Delete(VisRgn); +#endif pdc->flGraphicsCaps = ppdev->devinfo.flGraphicsCaps; pdc->flGraphicsCaps2 = ppdev->devinfo.flGraphicsCaps2; @@ -492,75 +474,106 @@ * from where we take pixels. */ VOID FASTCALL -DC_vPrepareDCsForBlit(PDC pdc1, - RECT rc1, - PDC pdc2, - RECT rc2) +DC_vPrepareDCsForBlit( + PDC pdcDest, + const RECT* rcDest, + PDC pdcSrc, + const RECT* rcSrc) { PDC pdcFirst, pdcSecond; - PRECT prcFirst, prcSecond; + const RECT *prcFirst, *prcSecond; /* Update brushes */ - if (pdc1->pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) - DC_vUpdateFillBrush(pdc1); - if (pdc1->pdcattr->ulDirty_ & (DIRTY_LINE | DC_PEN_DIRTY)) - DC_vUpdateLineBrush(pdc1); - if(pdc1->pdcattr->ulDirty_ & DIRTY_TEXT) - DC_vUpdateTextBrush(pdc1); + if (pdcDest->pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) + DC_vUpdateFillBrush(pdcDest); + if (pdcDest->pdcattr->ulDirty_ & (DIRTY_LINE | DC_PEN_DIRTY)) + DC_vUpdateLineBrush(pdcDest); + if(pdcDest->pdcattr->ulDirty_ & DIRTY_TEXT) + DC_vUpdateTextBrush(pdcDest); /* Lock them in good order */ - if(pdc2) - { - if((ULONG_PTR)pdc1->ppdev->hsemDevLock >= (ULONG_PTR)pdc2->ppdev->hsemDevLock) - { - pdcFirst = pdc1; - prcFirst = &rc1; - pdcSecond = pdc2; - prcSecond = &rc2; + if(pdcSrc) + { + if((ULONG_PTR)pdcDest->ppdev->hsemDevLock >= (ULONG_PTR)pdcSrc->ppdev->hsemDevLock) + { + pdcFirst = pdcDest; + prcFirst = rcDest; + pdcSecond = pdcSrc; + prcSecond = rcSrc; } else { - pdcFirst = pdc2; - prcFirst = &rc2; - pdcSecond = pdc1; - prcSecond = &rc1; + pdcFirst = pdcSrc; + prcFirst = rcSrc; + pdcSecond = pdcDest; + prcSecond = rcDest; } } else { - pdcFirst = pdc1 ; - prcFirst = &rc1; + pdcFirst = pdcDest ; + prcFirst = rcDest; pdcSecond = NULL; prcSecond = NULL; } - if(pdcFirst && pdcFirst->dctype == DCTYPE_DIRECT) + /* Update clipping of dest DC if needed */ + if (pdcDest->dctype == DCTYPE_DIRECT) + { + DCE* dce = DceGetDceFromDC(pdcDest->BaseObject.hHmgr); + if (dce) + DceUpdateVisRgn(dce, dce->pwndOrg, dce->DCXFlags); + } + + if (pdcDest->fs & DC_FLAG_DIRTY_RAO) + CLIPPING_UpdateGCRegion(pdcDest); + + /* Lock and update first DC */ + if(pdcFirst->dctype == DCTYPE_DIRECT) { EngAcquireSemaphore(pdcFirst->ppdev->hsemDevLock); - MouseSafetyOnDrawStart(pdcFirst->ppdev, - prcFirst->left, - prcFirst->top, - prcFirst->right, - prcFirst->bottom) ; /* Update surface if needed */ if(pdcFirst->ppdev->pSurface != pdcFirst->dclevel.pSurface) { DC_vUpdateDC(pdcFirst); } } - if(pdcSecond && pdcSecond->dctype == DCTYPE_DIRECT) + + if(pdcFirst->dctype == DCTYPE_DIRECT) + { + if (!prcFirst) + prcFirst = &pdcFirst->erclClip; + + MouseSafetyOnDrawStart(pdcFirst->ppdev, + prcFirst->left, + prcFirst->top, + prcFirst->right, + prcFirst->bottom) ; + } + + if (!pdcSecond) + return; + + /* Lock and update second DC */ + if(pdcSecond->dctype == DCTYPE_DIRECT) { EngAcquireSemaphore(pdcSecond->ppdev->hsemDevLock); - MouseSafetyOnDrawStart(pdcSecond->ppdev, - prcSecond->left, - prcSecond->top, - prcSecond->right, - prcSecond->bottom) ; /* Update surface if needed */ if(pdcSecond->ppdev->pSurface != pdcSecond->dclevel.pSurface) { DC_vUpdateDC(pdcSecond); } + } + + if(pdcSecond->dctype == DCTYPE_DIRECT) + { + if (!prcSecond) + prcSecond = &pdcSecond->erclClip; + MouseSafetyOnDrawStart(pdcSecond->ppdev, + prcSecond->left, + prcSecond->top, + prcSecond->right, + prcSecond->bottom) ; } } Modified: trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/dcobjs.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -330,7 +330,7 @@ PDC pdc; HBITMAP hbmpOld; PSURFACE psurfNew, psurfOld; - HRGN hVisRgn; + PREGION VisRgn; HDC hdcOld; ULONG cBitsPixel; ASSERT_NOGDILOCKS(); @@ -452,15 +452,15 @@ pdc->pdcattr->ulDirty_ |= DIRTY_FILL | DIRTY_LINE; /* FIXME: Improve by using a region without a handle and selecting it */ - hVisRgn = IntSysCreateRectRgn( 0, + VisRgn = IntSysCreateRectpRgn( 0, 0, pdc->dclevel.sizl.cx, pdc->dclevel.sizl.cy); - if (hVisRgn) - { - GdiSelectVisRgn(hdc, hVisRgn); - GreDeleteObject(hVisRgn); + if (VisRgn) + { + GdiSelectVisRgn(hdc, VisRgn); + REGION_Delete(VisRgn); } /* Unlock the DC */ @@ -509,7 +509,10 @@ /* Construct a region from the path */ else if (PATH_PathToRegion(pPath, pdcattr->jFillMode, &hrgnPath)) { - success = GdiExtSelectClipRgn(pdc, hrgnPath, Mode) != ERROR; + PREGION prgnPath = REGION_LockRgn(hrgnPath); + ASSERT(prgnPath); + success = IntGdiExtSelectClipRgn(pdc, prgnPath, Mode) != ERROR; + REGION_UnlockRgn(prgnPath); GreDeleteObject( hrgnPath ); /* Empty the path */ @@ -693,9 +696,7 @@ { INT ret = 0; PDC pdc; - HRGN hrgnSrc = NULL; PREGION prgnSrc = NULL; - POINTL ptlOrg; pdc = DC_LockDc(hdc); if (!pdc) @@ -707,8 +708,7 @@ switch (iCode) { case CLIPRGN: - hrgnSrc = pdc->rosdc.hClipRgn; -// if (pdc->dclevel.prgnClip) prgnSrc = pdc->dclevel.prgnClip; + prgnSrc = pdc->dclevel.prgnClip; break; case METARGN: @@ -716,14 +716,15 @@ break; case APIRGN: + if (pdc->fs & DC_FLAG_DIRTY_RAO) + CLIPPING_UpdateGCRegion(pdc); if (pdc->prgnAPI) { prgnSrc = pdc->prgnAPI; } -// else if (pdc->dclevel.prgnClip) prgnSrc = pdc->dclevel.prgnClip; - else if (pdc->rosdc.hClipRgn) + else if (pdc->dclevel.prgnClip) { - hrgnSrc = pdc->rosdc.hClipRgn; + prgnSrc = pdc->dclevel.prgnClip; } else if (pdc->dclevel.prgnMeta) { @@ -739,26 +740,18 @@ break; } - if (hrgnSrc) - { - ret = NtGdiCombineRgn(hrgnDest, hrgnSrc, 0, RGN_COPY) == ERROR ? -1 : 1; - } - else if (prgnSrc) + if (prgnSrc) { PREGION prgnDest = REGION_LockRgn(hrgnDest); if (prgnDest) { ret = IntGdiCombineRgn(prgnDest, prgnSrc, 0, RGN_COPY) == ERROR ? -1 : 1; + if ((ret == 1) && (iCode == SYSRGN)) + IntGdiOffsetRgn(prgnDest, pdc->ptlDCOrig.x, pdc->ptlDCOrig.y); REGION_UnlockRgn(prgnDest); } else ret = -1; - } - - if (iCode == SYSRGN) - { - ptlOrg = pdc->ptlDCOrig; - NtGdiOffsetRgn(hrgnDest, ptlOrg.x, ptlOrg.y ); } DC_UnlockDc(pdc); Modified: trunk/reactos/win32ss/gdi/ntgdi/dcstate.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dcstate.… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/dcstate.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/dcstate.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -51,23 +51,26 @@ pdcDst->dclevel.plfnt = pdcSrc->dclevel.plfnt; /* Get/SetDCState() don't change hVisRgn field ("Undoc. Windows" p.559). */ - if (To) // Copy "To" SaveDC state. - { - if (pdcSrc->rosdc.hClipRgn) - { - pdcDst->rosdc.hClipRgn = IntSysCreateRectRgn(0, 0, 0, 0); - NtGdiCombineRgn(pdcDst->rosdc.hClipRgn, pdcSrc->rosdc.hClipRgn, 0, RGN_COPY); - } - // FIXME: Handle prgnMeta! - } - else // Copy "!To" RestoreDC state. - { /* The VisRectRegion field needs to be set to a valid state */ - GdiExtSelectClipRgn(pdcDst, pdcSrc->rosdc.hClipRgn, RGN_COPY); - } -} - - -BOOL FASTCALL + if (!To) + { + IntGdiExtSelectClipRgn(pdcDst, pdcSrc->dclevel.prgnClip, RGN_COPY); + if (pdcDst->dclevel.prgnMeta) + { + REGION_Delete(pdcDst->dclevel.prgnMeta); + pdcDst->dclevel.prgnMeta = NULL; + } + if (pdcSrc->dclevel.prgnMeta) + { + pdcDst->dclevel.prgnMeta = IntSysCreateRectpRgn(0, 0, 0, 0); + IntGdiCombineRgn(pdcDst->dclevel.prgnMeta, pdcSrc->dclevel.prgnMeta, NULL, RGN_COPY); + } + pdcDst->fs |= DC_FLAG_DIRTY_RAO; + } +} + + +BOOL +FASTCALL IntGdiCleanDC(HDC hDC) { PDC dc; @@ -87,6 +90,16 @@ DC_vUpdateTextBrush(dc); } + /* DC_vCopyState frees the Clip rgn and the Meta rgn. Take care of the other ones + * There is no need to clear prgnVis, as UserGetDC updates it immediately. */ + if (dc->prgnRao) + REGION_Delete(dc->prgnRao); + if (dc->prgnAPI) + REGION_Delete(dc->prgnAPI); + dc->prgnRao = dc->prgnAPI = NULL; + + dc->fs |= DC_FLAG_DIRTY_RAO; + DC_UnlockDc(dc); return TRUE; @@ -272,7 +285,7 @@ GDIOBJ_vSetObjectOwner(&pdcSave->BaseObject, GDI_OBJ_HMGR_PUBLIC); /* Copy the current state */ - DC_vCopyState(pdc, pdcSave, TRUE); + DC_vCopyState(pdc, pdcSave, FALSE); /* Only memory DC's change their surface */ if (pdc->dctype == DCTYPE_MEMORY) Modified: trunk/reactos/win32ss/gdi/ntgdi/dcutil.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dcutil.c… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/dcutil.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/dcutil.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -367,7 +367,7 @@ } else if (Flags & DCHF_VALIDATEVISRGN || 0 == Flags) { - dc->fs &= ~DC_FLAG_DIRTY_RAO; + //dc->fs &= ~DC_FLAG_DIRTY_RAO; } DC_UnlockDc(dc); Modified: trunk/reactos/win32ss/gdi/ntgdi/dibobj.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/dibobj.c… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/dibobj.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/dibobj.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -502,7 +502,7 @@ } /* This is actually a blit */ - DC_vPrepareDCsForBlit(pDC, rcDest, NULL, rcDest); + DC_vPrepareDCsForBlit(pDC, &rcDest, NULL, NULL); pSurf = pDC->dclevel.pSurface; if (!pSurf) { @@ -1184,7 +1184,7 @@ } /* Prepare DC for blit */ - DC_vPrepareDCsForBlit(pdc, rcDst, NULL, rcSrc); + DC_vPrepareDCsForBlit(pdc, &rcDst, NULL, NULL); psurfDst = pdc->dclevel.pSurface; Modified: trunk/reactos/win32ss/gdi/ntgdi/fillshap.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/fillshap… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/fillshap.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/fillshap.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -307,7 +307,7 @@ tmpFillBrushObj.ptOrigin.x += dc->ptlDCOrig.x; tmpFillBrushObj.ptOrigin.y += dc->ptlDCOrig.y; - DC_vPrepareDCsForBlit(dc, RectBounds, NULL, RectBounds); + DC_vPrepareDCsForBlit(dc, &RectBounds, NULL, NULL); ret = IntFillEllipse( dc, CenterX - RadiusX, @@ -480,8 +480,7 @@ return TRUE; } - DC_vPrepareDCsForBlit(dc, dc->rosdc.CombinedClip->rclBounds, - NULL, dc->rosdc.CombinedClip->rclBounds); + DC_vPrepareDCsForBlit(dc, NULL, NULL, NULL); if (dc->pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) DC_vUpdateFillBrush(dc); @@ -567,7 +566,7 @@ DestRect.bottom--; } - DC_vPrepareDCsForBlit(dc, DestRect, NULL, DestRect); + DC_vPrepareDCsForBlit(dc, &DestRect, NULL, NULL); if (pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) DC_vUpdateFillBrush(dc); @@ -796,7 +795,7 @@ else { - DC_vPrepareDCsForBlit(dc, RectBounds, NULL, RectBounds); + DC_vPrepareDCsForBlit(dc, &RectBounds, NULL, NULL); RtlCopyMemory(&brushTemp, pbrFill, sizeof(brushTemp)); brushTemp.ptOrigin.x += RectBounds.left - Left; @@ -962,9 +961,9 @@ EXLATEOBJ_vInitialize(&exlo, &gpalRGB, psurf->ppal, 0, 0, 0); + DC_vPrepareDCsForBlit(pdc, &rclExtent, NULL, NULL); + ASSERT(pdc->rosdc.CombinedClip); - - DC_vPrepareDCsForBlit(pdc, rclExtent, NULL, rclExtent); bRet = IntEngGradientFill(&psurf->SurfObj, pdc->rosdc.CombinedClip, @@ -1091,6 +1090,13 @@ return TRUE; } + psurf = dc->dclevel.pSurface; + if (!psurf) + { + Ret = FALSE; + goto cleanup; + } + pdcattr = dc->pdcattr; if (pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) @@ -1103,20 +1109,18 @@ Pt.y = YStart; IntLPtoDP(dc, (LPPOINT)&Pt, 1); - Ret = NtGdiPtInRegion(dc->rosdc.hGCClipRgn, Pt.x, Pt.y); - if (Ret) - IntGdiGetRgnBox(dc->rosdc.hGCClipRgn,(LPRECT)&DestRect); + if (dc->prgnRao) + { + Ret = REGION_PtInRegion(dc->prgnRao, Pt.x, Pt.y); + if (Ret) + REGION_GetRgnBox(dc->prgnRao ,(LPRECT)&DestRect); + else + goto cleanup; + } else - goto cleanup; - - DC_vPrepareDCsForBlit(dc, DestRect, NULL, DestRect); - - psurf = dc->dclevel.pSurface; - if (!psurf) - { - Ret = FALSE; - goto cleanup; - } + RECTL_vSetRect(&DestRect, 0, psurf->SurfObj.sizlBitmap.cx, 0, psurf->SurfObj.sizlBitmap.cy); + + DC_vPrepareDCsForBlit(dc, &DestRect, NULL, NULL); EXLATEOBJ_vInitialize(&exlo, &gpalRGB, psurf->ppal, 0, 0xffffff, 0); @@ -1127,10 +1131,11 @@ ConvColor = XLATEOBJ_iXlate(&exlo.xlo, Color); Ret = DIB_XXBPP_FloodFillSolid(&psurf->SurfObj, &dc->eboFill.BrushObject, &DestRect, &Pt, ConvColor, FillType); + DC_vFinishBlit(dc, NULL); + EXLATEOBJ_vCleanup(&exlo); cleanup: - DC_vFinishBlit(dc, NULL); DC_UnlockDc(dc); return Ret; } Modified: trunk/reactos/win32ss/gdi/ntgdi/freetype.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/freetype… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/freetype.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/freetype.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -3212,7 +3212,7 @@ LONGLONG TextLeft, RealXStart; ULONG TextTop, previous, BackgroundLeft; FT_Bool use_kerning; - RECTL DestRect, MaskRect, DummyRect = {0, 0, 0, 0}; + RECTL DestRect, MaskRect; POINTL SourcePoint, BrushOrigin; HBITMAP HSourceGlyph; SURFOBJ *SourceGlyphSurf; @@ -3308,7 +3308,7 @@ DestRect.right += dc->ptlDCOrig.x; DestRect.bottom += dc->ptlDCOrig.y; - DC_vPrepareDCsForBlit(dc, DestRect, NULL, DestRect); + DC_vPrepareDCsForBlit(dc, &DestRect, NULL, NULL); if (pdcattr->ulDirty_ & DIRTY_BACKGROUND) DC_vUpdateBackgroundBrush(dc); @@ -3502,7 +3502,7 @@ BackgroundLeft = (RealXStart + 32) >> 6; /* Lock blit with a dummy rect */ - DC_vPrepareDCsForBlit(dc, DummyRect, NULL, DummyRect); + DC_vPrepareDCsForBlit(dc, NULL, NULL, NULL); psurf = dc->dclevel.pSurface ; if(!psurf) psurf = psurfDefaultBitmap; Modified: trunk/reactos/win32ss/gdi/ntgdi/line.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/line.c?r… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/line.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/line.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -257,8 +257,7 @@ if (PATH_IsPathOpen(dc->dclevel)) return PATH_Polyline(dc, pt, Count); - DC_vPrepareDCsForBlit(dc, dc->rosdc.CombinedClip->rclBounds, - NULL, dc->rosdc.CombinedClip->rclBounds); + DC_vPrepareDCsForBlit(dc, NULL, NULL, NULL); if (pdcattr->ulDirty_ & (DIRTY_FILL | DC_BRUSH_DIRTY)) DC_vUpdateFillBrush(dc); @@ -411,7 +410,7 @@ rcLockRect.right += dc->ptlDCOrig.x; rcLockRect.bottom += dc->ptlDCOrig.y; - DC_vPrepareDCsForBlit(dc, rcLockRect, NULL, rcLockRect); + DC_vPrepareDCsForBlit(dc, &rcLockRect, NULL, NULL); if (dc->pdcattr->ulDirty_ & (DIRTY_LINE | DC_PEN_DIRTY)) DC_vUpdateLineBrush(dc); Modified: trunk/reactos/win32ss/gdi/ntgdi/path.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/path.c?r… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/path.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/path.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -2511,8 +2511,7 @@ return FALSE; } - DC_vPrepareDCsForBlit(dc, dc->rosdc.CombinedClip->rclBounds, - NULL, dc->rosdc.CombinedClip->rclBounds); + DC_vPrepareDCsForBlit(dc, NULL, NULL, NULL); pdcattr = dc->pdcattr; @@ -2777,8 +2776,7 @@ return FALSE; } - DC_vPrepareDCsForBlit(pDc, pDc->rosdc.CombinedClip->rclBounds, - NULL, pDc->rosdc.CombinedClip->rclBounds); + DC_vPrepareDCsForBlit(pDc, NULL, NULL, NULL); pdcattr = pDc->pdcattr; @@ -2822,8 +2820,7 @@ return FALSE; } - DC_vPrepareDCsForBlit(pDc, pDc->rosdc.CombinedClip->rclBounds, - NULL, pDc->rosdc.CombinedClip->rclBounds); + DC_vPrepareDCsForBlit(pDc, NULL, NULL, NULL); pdcattr = pDc->pdcattr; Modified: trunk/reactos/win32ss/gdi/ntgdi/region.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/region.c… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/region.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/region.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -2300,11 +2300,13 @@ VOID FASTCALL IntGdiReleaseRaoRgn(PDC pDC) { - INT Index = GDI_HANDLE_GET_INDEX(pDC->BaseObject.hHmgr); - PGDI_TABLE_ENTRY Entry = &GdiHandleTable->Entries[Index]; - pDC->fs |= DC_FLAG_DIRTY_RAO; - Entry->Flags |= GDI_ENTRY_VALIDATE_VIS; - RECTL_vSetEmptyRect(&pDC->erclClip); + INT Index = GDI_HANDLE_GET_INDEX(pDC->BaseObject.hHmgr); + PGDI_TABLE_ENTRY Entry = &GdiHandleTable->Entries[Index]; + pDC->fs |= DC_FLAG_DIRTY_RAO; + Entry->Flags |= GDI_ENTRY_VALIDATE_VIS; + RECTL_vSetEmptyRect(&pDC->erclClip); + REGION_Delete(pDC->prgnRao); + pDC->prgnRao = NULL; } VOID FASTCALL @@ -2503,14 +2505,15 @@ return FALSE; } - NtGdiCombineRgn(tmpVisRgn, tmpVisRgn, dc->rosdc.hGCClipRgn, RGN_AND); - visrgn = RGNOBJAPI_Lock(tmpVisRgn, NULL); if (visrgn == NULL) { GreDeleteObject(tmpVisRgn); return FALSE; } + + if (dc->prgnRao) + IntGdiCombineRgn(visrgn, visrgn, dc->prgnRao, RGN_AND); ClipRegion = IntEngCreateClipRegion(visrgn->rdh.nCount, visrgn->Buffer, @@ -2533,6 +2536,29 @@ // Fill the region return bRet; +} + +BOOL +FASTCALL +REGION_PtInRegion( + PREGION prgn, + INT X, + INT Y) +{ + ULONG i; + PRECT r; + + if (prgn->rdh.nCount > 0 && INRECT(prgn->rdh.rcBound, X, Y)) + { + r = prgn->Buffer; + for (i = 0; i < prgn->rdh.nCount; i++) + { + if (INRECT(r[i], X, Y)) + return TRUE; + } + } + + return FALSE; } BOOL @@ -3861,27 +3887,19 @@ INT Y ) { - PROSRGNDATA rgn; - ULONG i; - PRECTL r; - - if (!(rgn = RGNOBJAPI_Lock(hRgn, NULL) ) ) + PREGION prgn; + BOOL ret; + + if (!(prgn = RGNOBJAPI_Lock(hRgn, NULL) ) ) return FALSE; - if (rgn->rdh.nCount > 0 && INRECT(rgn->rdh.rcBound, X, Y)) - { - r = rgn->Buffer; - for (i = 0; i < rgn->rdh.nCount; i++) - { - if (INRECT(*r, X, Y)) - { - RGNOBJAPI_Unlock(rgn); - return TRUE; - } - r++; - } - } - RGNOBJAPI_Unlock(rgn); + ret = REGION_PtInRegion(prgn, X, Y); + + RGNOBJAPI_Unlock(prgn); + return ret; + + + RGNOBJAPI_Unlock(prgn); return FALSE; } Modified: trunk/reactos/win32ss/gdi/ntgdi/region.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/region.h… ============================================================================== --- trunk/reactos/win32ss/gdi/ntgdi/region.h [iso-8859-1] (original) +++ trunk/reactos/win32ss/gdi/ntgdi/region.h [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -27,6 +27,7 @@ VOID FASTCALL REGION_UnionRectWithRgn(ROSRGNDATA *rgn, const RECTL *rect); INT FASTCALL REGION_GetRgnBox(PROSRGNDATA Rgn, RECTL *pRect); BOOL FASTCALL REGION_RectInRegion(PROSRGNDATA Rgn, const RECTL *rc); +BOOL FASTCALL REGION_PtInRegion(PREGION, INT, INT); BOOL FASTCALL REGION_CropAndOffsetRegion(PROSRGNDATA rgnDst, PROSRGNDATA rgnSrc, const RECTL *rect, const POINT *off); VOID FASTCALL REGION_SetRectRgn(PROSRGNDATA pRgn, INT LeftRect, INT TopRect, INT RightRect, INT BottomRect); VOID NTAPI REGION_vCleanup(PVOID ObjectBody); Modified: trunk/reactos/win32ss/user/ntuser/cursoricon.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/cursor… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/cursoricon.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/cursoricon.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -1287,7 +1287,7 @@ RECTL_vOffsetRect(&rcDest, pdc->ptlDCOrig.x, pdc->ptlDCOrig.y); /* Prepare the underlying surface */ - DC_vPrepareDCsForBlit(pdc, rcDest, NULL, rcDest); + DC_vPrepareDCsForBlit(pdc, &rcDest, NULL, NULL); /* Get the clip object */ pdcClipObj = pdc->rosdc.CombinedClip; @@ -1472,7 +1472,7 @@ RECTL_vOffsetRect(&rcDest, pdc->ptlDCOrig.x, pdc->ptlDCOrig.y); /* Prepare the underlying surface */ - DC_vPrepareDCsForBlit(pdc, rcDest, NULL, rcDest); + DC_vPrepareDCsForBlit(pdc, &rcDest, NULL, NULL); /* Get the clip object */ pdcClipObj = pdc->rosdc.CombinedClip; Modified: trunk/reactos/win32ss/user/ntuser/cursoricon_new.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/cursor… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/cursoricon_new.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/cursoricon_new.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -1195,7 +1195,7 @@ RECTL_vOffsetRect(&rcDest, pdc->ptlDCOrig.x, pdc->ptlDCOrig.y); /* Prepare the underlying surface */ - DC_vPrepareDCsForBlit(pdc, rcDest, NULL, rcDest); + DC_vPrepareDCsForBlit(pdc, &rcDest, NULL, NULL); /* We now have our destination surface and rectangle */ psurfDest = pdc->dclevel.pSurface; Modified: trunk/reactos/win32ss/user/ntuser/dce.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/dce.h?… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/dce.h [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/dce.h [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -51,3 +51,5 @@ HWND FASTCALL UserGethWnd(HDC,PWNDOBJ*); void FASTCALL DceFreeWindowDCE(PWND); void FASTCALL DceFreeThreadDCE(PTHREADINFO); +VOID FASTCALL DceUpdateVisRgn(DCE *Dce, PWND Window, ULONG Flags); +DCE* FASTCALL DceGetDceFromDC(HDC hdc); Modified: trunk/reactos/win32ss/user/ntuser/windc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/windc.… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/windc.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/windc.c [iso-8859-1] Wed Jul 23 16:05:47 2014 @@ -46,21 +46,47 @@ return IntGdiCreateDC(&DriverName, NULL, NULL, NULL, FALSE); } +/* Returns the DCE pointer from the HDC handle */ +DCE* +FASTCALL +DceGetDceFromDC(HDC hdc) +{ + LIST_ENTRY* Entry = LEDce.Flink; + DCE* dce; + + while (Entry != &LEDce) + { + dce = CONTAINING_RECORD(Entry, DCE, List); + if (dce->hDC == hdc) + return dce; + Entry = Entry->Flink; + } + + return NULL; +} + static -HRGN FASTCALL +PREGION FASTCALL DceGetVisRgn(PWND Window, ULONG Flags, HWND hWndChild, ULONG CFlags) { - HRGN VisRgn; - - VisRgn = VIS_ComputeVisibleRegion( Window, - 0 == (Flags & DCX_WINDOW), - 0 != (Flags & DCX_CLIPCHILDREN), - 0 != (Flags & DCX_CLIPSIBLINGS)); - - if (VisRgn == NULL) - VisRgn = IntSysCreateRectRgn(0, 0, 0, 0); - - return VisRgn; + PREGION RetRgn; + HRGN hVisRgn; + hVisRgn = VIS_ComputeVisibleRegion( Window, + 0 == (Flags & DCX_WINDOW), + 0 != (Flags & DCX_CLIPCHILDREN), + 0 != (Flags & DCX_CLIPSIBLINGS)); + + RetRgn = IntSysCreateRectpRgn(0, 0, 0, 0); + + if (hVisRgn != NULL) + { + PREGION VisRgn = REGION_LockRgn(hVisRgn); + IntGdiCombineRgn(RetRgn, VisRgn, NULL, RGN_COPY); + REGION_UnlockRgn(VisRgn); + GreDeleteObject(hVisRgn); + } + + return RetRgn; } PDCE FASTCALL @@ -150,6 +176,7 @@ dc->ptlDCOrig.y = Window->rcClient.top; } } + dc->fs |= DC_FLAG_DIRTY_RAO; DC_UnlockDc(dc); } @@ -175,10 +202,11 @@ IntGdiSetHookFlags(Dce->hDC, DCHF_INVALIDATEVISRGN); } -static VOID FASTCALL +VOID +FASTCALL DceUpdateVisRgn(DCE *Dce, PWND Window, ULONG Flags) { - HANDLE hRgnVisible = NULL; + PREGION RgnVisible = NULL; ULONG DcxFlags; PWND DesktopWindow; @@ -189,7 +217,7 @@ Parent = Window->spwndParent; if(!Parent) { - hRgnVisible = NULL; + RgnVisible = NULL; goto noparent; } @@ -202,23 +230,23 @@ { DcxFlags = Flags & ~(DCX_CLIPSIBLINGS | DCX_CLIPCHILDREN | DCX_WINDOW); } - hRgnVisible = DceGetVisRgn(Parent, DcxFlags, Window->head.h, Flags); + RgnVisible = DceGetVisRgn(Parent, DcxFlags, Window->head.h, Flags); } else if (Window == NULL) { DesktopWindow = UserGetWindowObject(IntGetDesktopWindow()); if (NULL != DesktopWindow) { - hRgnVisible = IntSysCreateRectRgnIndirect(&DesktopWindow->rcWindow); + RgnVisible = IntSysCreateRectpRgnIndirect(&DesktopWindow->rcWindow); } else { - hRgnVisible = NULL; + RgnVisible = NULL; } } else { - hRgnVisible = DceGetVisRgn(Window, Flags, 0, 0); + RgnVisible = DceGetVisRgn(Window, Flags, 0, 0); } noparent: @@ -226,33 +254,37 @@ { if(Dce->hrgnClip != NULL) { - NtGdiCombineRgn(hRgnVisible, hRgnVisible, Dce->hrgnClip, RGN_AND); + PREGION RgnClip = REGION_LockRgn(Dce->hrgnClip); + IntGdiCombineRgn(RgnVisible, RgnVisible, RgnClip, RGN_AND); + REGION_UnlockRgn(RgnClip); } else { - if(hRgnVisible != NULL) - { - GreDeleteObject(hRgnVisible); - } - hRgnVisible = IntSysCreateRectRgn(0, 0, 0, 0); - } - } - else if (Flags & DCX_EXCLUDERGN && Dce->hrgnClip != NULL) - { - NtGdiCombineRgn(hRgnVisible, hRgnVisible, Dce->hrgnClip, RGN_DIFF); + if(RgnVisible != NULL) + { + REGION_Delete(RgnVisible); + } + RgnVisible = IntSysCreateRectpRgn(0, 0, 0, 0); + } + } + else if ((Flags & DCX_EXCLUDERGN) && Dce->hrgnClip != NULL) + { + PREGION RgnClip = REGION_LockRgn(Dce->hrgnClip); + IntGdiCombineRgn(RgnVisible, RgnVisible, RgnClip, RGN_DIFF); + REGION_UnlockRgn(RgnClip); } Dce->DCXFlags &= ~DCX_DCEDIRTY; - GdiSelectVisRgn(Dce->hDC, hRgnVisible); + GdiSelectVisRgn(Dce->hDC, RgnVisible); if (VerifyWnd(Window)) // Window maybe dead by this time before finishing the DCE release. { IntEngWindowChanged(Window, WOC_RGN_CLIENT); } - if (hRgnVisible != NULL) - { - GreDeleteObject(hRgnVisible); + if (RgnVisible != NULL) + { + REGION_Delete(RgnVisible); } } @@ -889,10 +921,10 @@ dc->ptlDCOrig.y = CurrentWindow->rcClient.top; } - if (NULL != dc->rosdc.hClipRgn) + if (NULL != dc->dclevel.prgnClip) { - NtGdiOffsetRgn(dc->rosdc.hClipRgn, DeltaX, DeltaY); - CLIPPING_UpdateGCRegion(dc); + IntGdiOffsetRgn(dc->dclevel.prgnClip, DeltaX, DeltaY); + dc->fs |= DC_FLAG_DIRTY_RAO; } if (NULL != pDCE->hrgnClip) {

1 0

[tfaber] 63730: [FORREALTHISTIME] - Apparently we got that typo from Wine. Fix build. Will send upstream.
by tfaber＠svn.reactos.org 23 Jul '14

23 Jul '14

Author: tfaber Date: Wed Jul 23 12:46:47 2014 New Revision: 63730 URL: http://svn.reactos.org/svn/reactos?rev=63730&view=rev Log: [FORREALTHISTIME] - Apparently we got that typo from Wine. Fix build. Will send upstream. Modified: trunk/reactos/lib/rtl/error.c trunk/rostests/winetests/ntdll/error.c Modified: trunk/reactos/lib/rtl/error.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/error.c?rev=63730&… ============================================================================== --- trunk/reactos/lib/rtl/error.c [iso-8859-1] (original) +++ trunk/reactos/lib/rtl/error.c [iso-8859-1] Wed Jul 23 12:46:47 2014 @@ -1523,7 +1523,7 @@ ERROR_SXS_MANIFEST_TOO_BIG, /* c0150022 (STATUS_SXS_MANIFEST_TOO_BIG) */ ERROR_SXS_SETTING_NOT_REGISTERED, /* c0150023 (STATUS_SXS_SETTING_NOT_REGISTERED) */ ERROR_SXS_TRANSACTION_CLOSURE_INCOMPLETE, /* c0150024 (STATUS_SXS_TRANSACTION_CLOSURE_INCOMPLETE) */ - ERROR_SMI_PRIMITIVE_INSTALLER_FAILED, /* c0150025 (STATUS_SXS_PRIMITIVE_INSTALLER_FAILED) */ + ERROR_SMI_PRIMITIVE_INSTALLER_FAILED, /* c0150025 (STATUS_SMI_PRIMITIVE_INSTALLER_FAILED) */ ERROR_GENERIC_COMMAND_FAILED, /* c0150026 (STATUS_GENERIC_COMMAND_FAILED) */ ERROR_SXS_FILE_HASH_MISSING /* c0150027 (STATUS_SXS_FILE_HASH_MISSING) */ }; Modified: trunk/rostests/winetests/ntdll/error.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/ntdll/error.c?r… ============================================================================== --- trunk/rostests/winetests/ntdll/error.c [iso-8859-1] (original) +++ trunk/rostests/winetests/ntdll/error.c [iso-8859-1] Wed Jul 23 12:46:47 2014 @@ -963,7 +963,7 @@ cmp2(STATUS_SXS_MANIFEST_TOO_BIG, ERROR_SXS_MANIFEST_TOO_BIG); cmp2(STATUS_SXS_SETTING_NOT_REGISTERED, ERROR_SXS_SETTING_NOT_REGISTERED); cmp2(STATUS_SXS_TRANSACTION_CLOSURE_INCOMPLETE, ERROR_SXS_TRANSACTION_CLOSURE_INCOMPLETE); - cmp2(STATUS_SXS_PRIMITIVE_INSTALLER_FAILED, ERROR_SMI_PRIMITIVE_INSTALLER_FAILED); + cmp2(STATUS_SMI_PRIMITIVE_INSTALLER_FAILED, ERROR_SMI_PRIMITIVE_INSTALLER_FAILED); cmp2(STATUS_GENERIC_COMMAND_FAILED, ERROR_GENERIC_COMMAND_FAILED); cmp2(STATUS_SXS_FILE_HASH_MISSING, ERROR_SXS_FILE_HASH_MISSING); cmp2(STATUS_REDIRECTOR_STARTED, ERROR_SERVICE_ALREADY_RUNNING);

1 0

[tfaber] 63729: [CLT2014] - ... and remove the branch
by tfaber＠svn.reactos.org 23 Jul '14

23 Jul '14

Author: tfaber Date: Wed Jul 23 12:39:02 2014 New Revision: 63729 URL: http://svn.reactos.org/svn/reactos?rev=63729&view=rev Log: [CLT2014] - ... and remove the branch Removed: branches/ros-branch-0_3_16-clt2014/

1 0

[tfaber] 63728: [CLT2014] - Erm, that branch shouldn't even exist anymore. Tag the release...
by tfaber＠svn.reactos.org 23 Jul '14

23 Jul '14

Author: tfaber Date: Wed Jul 23 12:38:37 2014 New Revision: 63728 URL: http://svn.reactos.org/svn/reactos?rev=63728&view=rev Log: [CLT2014] - Erm, that branch shouldn't even exist anymore. Tag the release... Added: tags/ReactOS-0.3.16-CLT2014/ - copied from r62441, branches/ros-branch-0_3_16-clt2014/

1 0

[tfaber] 63727: [OOPS] - Apparently we got that typo from Wine. Fix build. Will send upstream.
by tfaber＠svn.reactos.org 23 Jul '14

23 Jul '14

Author: tfaber Date: Wed Jul 23 12:33:25 2014 New Revision: 63727 URL: http://svn.reactos.org/svn/reactos?rev=63727&view=rev Log: [OOPS] - Apparently we got that typo from Wine. Fix build. Will send upstream. Modified: branches/ros-branch-0_3_16-clt2014/reactos/lib/rtl/error.c branches/ros-branch-0_3_16-clt2014/rostests/winetests/ntdll/error.c Modified: branches/ros-branch-0_3_16-clt2014/reactos/lib/rtl/error.c URL: http://svn.reactos.org/svn/reactos/branches/ros-branch-0_3_16-clt2014/react… ============================================================================== --- branches/ros-branch-0_3_16-clt2014/reactos/lib/rtl/error.c [iso-8859-1] (original) +++ branches/ros-branch-0_3_16-clt2014/reactos/lib/rtl/error.c [iso-8859-1] Wed Jul 23 12:33:25 2014 @@ -1523,7 +1523,7 @@ ERROR_SXS_MANIFEST_TOO_BIG, /* c0150022 (STATUS_SXS_MANIFEST_TOO_BIG) */ ERROR_SXS_SETTING_NOT_REGISTERED, /* c0150023 (STATUS_SXS_SETTING_NOT_REGISTERED) */ ERROR_SXS_TRANSACTION_CLOSURE_INCOMPLETE, /* c0150024 (STATUS_SXS_TRANSACTION_CLOSURE_INCOMPLETE) */ - ERROR_SMI_PRIMITIVE_INSTALLER_FAILED, /* c0150025 (STATUS_SXS_PRIMITIVE_INSTALLER_FAILED) */ + ERROR_SMI_PRIMITIVE_INSTALLER_FAILED, /* c0150025 (STATUS_SMI_PRIMITIVE_INSTALLER_FAILED) */ ERROR_GENERIC_COMMAND_FAILED, /* c0150026 (STATUS_GENERIC_COMMAND_FAILED) */ ERROR_SXS_FILE_HASH_MISSING /* c0150027 (STATUS_SXS_FILE_HASH_MISSING) */ }; Modified: branches/ros-branch-0_3_16-clt2014/rostests/winetests/ntdll/error.c URL: http://svn.reactos.org/svn/reactos/branches/ros-branch-0_3_16-clt2014/roste… ============================================================================== --- branches/ros-branch-0_3_16-clt2014/rostests/winetests/ntdll/error.c [iso-8859-1] (original) +++ branches/ros-branch-0_3_16-clt2014/rostests/winetests/ntdll/error.c [iso-8859-1] Wed Jul 23 12:33:25 2014 @@ -963,7 +963,7 @@ cmp2(STATUS_SXS_MANIFEST_TOO_BIG, ERROR_SXS_MANIFEST_TOO_BIG); cmp2(STATUS_SXS_SETTING_NOT_REGISTERED, ERROR_SXS_SETTING_NOT_REGISTERED); cmp2(STATUS_SXS_TRANSACTION_CLOSURE_INCOMPLETE, ERROR_SXS_TRANSACTION_CLOSURE_INCOMPLETE); - cmp2(STATUS_SXS_PRIMITIVE_INSTALLER_FAILED, ERROR_SMI_PRIMITIVE_INSTALLER_FAILED); + cmp2(STATUS_SMI_PRIMITIVE_INSTALLER_FAILED, ERROR_SMI_PRIMITIVE_INSTALLER_FAILED); cmp2(STATUS_GENERIC_COMMAND_FAILED, ERROR_GENERIC_COMMAND_FAILED); cmp2(STATUS_SXS_FILE_HASH_MISSING, ERROR_SXS_FILE_HASH_MISSING); cmp2(STATUS_REDIRECTOR_STARTED, ERROR_SERVICE_ALREADY_RUNNING);

1 0

[tfaber] 63726: [PSDK] - Fix a typo in ntstatus.h
by tfaber＠svn.reactos.org 23 Jul '14

23 Jul '14

Author: tfaber Date: Wed Jul 23 12:23:31 2014 New Revision: 63726 URL: http://svn.reactos.org/svn/reactos?rev=63726&view=rev Log: [PSDK] - Fix a typo in ntstatus.h Modified: trunk/reactos/include/psdk/ntstatus.h Modified: trunk/reactos/include/psdk/ntstatus.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/ntstatus.h?re… ============================================================================== --- trunk/reactos/include/psdk/ntstatus.h [iso-8859-1] (original) +++ trunk/reactos/include/psdk/ntstatus.h [iso-8859-1] Wed Jul 23 12:23:31 2014 @@ -1181,7 +1181,7 @@ #define STATUS_SXS_MANIFEST_TOO_BIG ((NTSTATUS) 0xC0150022) #define STATUS_SXS_SETTING_NOT_REGISTERED ((NTSTATUS) 0xC0150023) #define STATUS_SXS_TRANSACTION_CLOSURE_INCOMPLETE ((NTSTATUS) 0xC0150024) -#define STATUS_SXS_PRIMITIVE_INSTALLER_FAILED ((NTSTATUS) 0xC0150025) +#define STATUS_SMI_PRIMITIVE_INSTALLER_FAILED ((NTSTATUS) 0xC0150025) #define STATUS_GENERIC_COMMAND_FAILED ((NTSTATUS) 0xC0150026) #define STATUS_SXS_FILE_HASH_MISSING ((NTSTATUS) 0xC0150027)

1 0

[tfaber] 63725: [WININET_WINETEST] - Add back work-around for testbot hang lost in winesync. ROSTESTS-73
by tfaber＠svn.reactos.org 20 Jul '14

20 Jul '14

Author: tfaber Date: Sun Jul 20 11:19:44 2014 New Revision: 63725 URL: http://svn.reactos.org/svn/reactos?rev=63725&view=rev Log: [WININET_WINETEST] - Add back work-around for testbot hang lost in winesync. ROSTESTS-73 Modified: trunk/rostests/winetests/wininet/http.c Modified: trunk/rostests/winetests/wininet/http.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/wininet/http.c?… ============================================================================== --- trunk/rostests/winetests/wininet/http.c [iso-8859-1] (original) +++ trunk/rostests/winetests/wininet/http.c [iso-8859-1] Sun Jul 20 11:19:44 2014 @@ -3143,7 +3143,11 @@ SET_EXPECT(INTERNET_STATUS_CONNECTION_CLOSED); SET_EXPECT(INTERNET_STATUS_REQUEST_COMPLETE); SetEvent(conn_close_event); +#ifdef ROSTESTS_73_FIXED WaitForSingleObject(hCompleteEvent, INFINITE); +#else /* ROSTESTS_73_FIXED */ + ok(WaitForSingleObject(hCompleteEvent, 5000) == WAIT_OBJECT_0, "Wait timed out\n"); +#endif /* ROSTESTS_73_FIXED */ ok(req_error == ERROR_SUCCESS, "req_error = %u\n", req_error); CHECK_NOTIFIED(INTERNET_STATUS_CLOSING_CONNECTION); CHECK_NOTIFIED(INTERNET_STATUS_CONNECTION_CLOSED);

1 0

[tfaber] 63724: [WININET] - Apply Wine commit 8fd44a3d (wininet: Fix invalid memory access in HTTP_QUERY_RAW_HEADERS (Valgrind).) by Sebastian Lackner. Fixes crash in wininet:http InternetReadFile_...
by tfaber＠svn.reactos.org 20 Jul '14

20 Jul '14

Author: tfaber Date: Sun Jul 20 08:30:21 2014 New Revision: 63724 URL: http://svn.reactos.org/svn/reactos?rev=63724&view=rev Log: [WININET] - Apply Wine commit 8fd44a3d (wininet: Fix invalid memory access in HTTP_QUERY_RAW_HEADERS (Valgrind).) by Sebastian Lackner. Fixes crash in wininet:http InternetReadFile_test Modified: trunk/reactos/dll/win32/wininet/http.c Modified: trunk/reactos/dll/win32/wininet/http.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/wininet/http.c?r… ============================================================================== --- trunk/reactos/dll/win32/wininet/http.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/wininet/http.c [iso-8859-1] Sun Jul 20 08:30:21 2014 @@ -3568,12 +3568,12 @@ TRACE("returning data: %s\n", debugstr_wn(headers, len / sizeof(WCHAR))); - for (i=0; i<len; i++) + for (i = 0; i < len / sizeof(WCHAR); i++) { if (headers[i] == '\n') headers[i] = 0; } - memcpy(lpBuffer, headers, len + sizeof(WCHAR)); + memcpy(lpBuffer, headers, len); } *lpdwBufferLength = len - sizeof(WCHAR);

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs