Ros-diffs

ros-diffs@reactos.org

81390 discussions

[ekohl] 57314: [INF] Add german translations of device classes.
by ekohl＠svn.reactos.org 16 Sep '12

16 Sep '12

Author: ekohl Date: Sun Sep 16 21:01:06 2012 New Revision: 57314 URL: http://svn.reactos.org/svn/reactos?rev=57314&view=rev Log: [INF] Add german translations of device classes. Modified: trunk/reactos/media/inf/cpu.inf trunk/reactos/media/inf/legcydrv.inf trunk/reactos/media/inf/monitor.inf trunk/reactos/media/inf/unknown.inf Modified: trunk/reactos/media/inf/cpu.inf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/inf/cpu.inf?rev=5731… ============================================================================== Binary files - no diff available. Modified: trunk/reactos/media/inf/legcydrv.inf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/inf/legcydrv.inf?rev… ============================================================================== Binary files - no diff available. Modified: trunk/reactos/media/inf/monitor.inf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/inf/monitor.inf?rev=… ============================================================================== Binary files - no diff available. Modified: trunk/reactos/media/inf/unknown.inf URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/inf/unknown.inf?rev=… ============================================================================== Binary files - no diff available.

1 0

[akhaldi] 57313: * Don't use -momit-leaf-frame-pointer in debug gcc builds, it messes up stack traces. * Remove it from the amd64 gcc builds too.
by akhaldi＠svn.reactos.org 16 Sep '12

16 Sep '12

Author: akhaldi Date: Sun Sep 16 12:57:22 2012 New Revision: 57313 URL: http://svn.reactos.org/svn/reactos?rev=57313&view=rev Log: * Don't use -momit-leaf-frame-pointer in debug gcc builds, it messes up stack traces. * Remove it from the amd64 gcc builds too. Modified: trunk/reactos/cmake/gcc.cmake Modified: trunk/reactos/cmake/gcc.cmake URL: http://svn.reactos.org/svn/reactos/trunk/reactos/cmake/gcc.cmake?rev=57313&… ============================================================================== --- trunk/reactos/cmake/gcc.cmake [iso-8859-1] (original) +++ trunk/reactos/cmake/gcc.cmake [iso-8859-1] Sun Sep 16 12:57:22 2012 @@ -83,12 +83,15 @@ if(ARCH STREQUAL "i386") add_compile_flags("-mpreferred-stack-boundary=3 -fno-set-stack-executable -fno-optimize-sibling-calls -fno-omit-frame-pointer") if(OPTIMIZE STREQUAL "1") - add_compile_flags("-ftracer -momit-leaf-frame-pointer") + add_compile_flags("-ftracer") + if(NOT CMAKE_BUILD_TYPE STREQUAL "Debug") + add_compile_flags("-momit-leaf-frame-pointer") + endif() endif() elseif(ARCH STREQUAL "amd64") add_compile_flags("-mpreferred-stack-boundary=4") if(OPTIMIZE STREQUAL "1") - add_compile_flags("-ftracer -momit-leaf-frame-pointer") + add_compile_flags("-ftracer") endif() elseif(ARCH STREQUAL "arm") if(OPTIMIZE STREQUAL "1")

1 0

[akhaldi] 57312: [LIBUSB]: Handle configuration descriptors partial reads. By Jérôme Gardou. CORE-6634 #resolve
by akhaldi＠svn.reactos.org 16 Sep '12

16 Sep '12

Author: akhaldi Date: Sun Sep 16 12:44:39 2012 New Revision: 57312 URL: http://svn.reactos.org/svn/reactos?rev=57312&view=rev Log: [LIBUSB]: Handle configuration descriptors partial reads. By Jérôme Gardou. CORE-6634 #resolve Modified: trunk/reactos/lib/drivers/libusb/hub_controller.cpp Modified: trunk/reactos/lib/drivers/libusb/hub_controller.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/drivers/libusb/hub_con… ============================================================================== --- trunk/reactos/lib/drivers/libusb/hub_controller.cpp [iso-8859-1] (original) +++ trunk/reactos/lib/drivers/libusb/hub_controller.cpp [iso-8859-1] Sun Sep 16 12:44:39 2012 @@ -1494,11 +1494,10 @@ IN OUT PURB Urb) { NTSTATUS Status = STATUS_NOT_IMPLEMENTED; - PUSB_CONFIGURATION_DESCRIPTOR ConfigurationDescriptor; USB_DEFAULT_PIPE_SETUP_PACKET CtrlSetup; PUCHAR Buffer; PUSBDEVICE UsbDevice; - ULONG Length; + ULONG Length, BufferLength; DPRINT("[USBLIB] HandleGetDescriptor\n"); @@ -1557,44 +1556,70 @@ // sanity checks // PC_ASSERT(Urb->UrbControlDescriptorRequest.TransferBuffer); - PC_ASSERT(Urb->UrbControlDescriptorRequest.TransferBufferLength >= sizeof(USB_CONFIGURATION_DESCRIPTOR)); + // + // From MSDN + // The caller must allocate a buffer large enough to hold all of this information or the data is truncated without error. + // + BufferLength = Urb->UrbControlDescriptorRequest.TransferBufferLength; + Buffer = (PUCHAR) Urb->UrbControlDescriptorRequest.TransferBuffer; if (Urb->UrbHeader.UsbdDeviceHandle == PVOID(this)) { // // request is for the root bus controller // - RtlCopyMemory(Urb->UrbControlDescriptorRequest.TransferBuffer, &ROOTHUB2_CONFIGURATION_DESCRIPTOR, sizeof(USB_CONFIGURATION_DESCRIPTOR)); - - // - // get configuration descriptor, very retarded! - // - ConfigurationDescriptor = (PUSB_CONFIGURATION_DESCRIPTOR)Urb->UrbControlDescriptorRequest.TransferBuffer; - - // - // check if buffer can hold interface and endpoint descriptor - // - if (ConfigurationDescriptor->wTotalLength > Urb->UrbControlDescriptorRequest.TransferBufferLength) + Length = BufferLength > sizeof(USB_CONFIGURATION_DESCRIPTOR) ? + sizeof(USB_CONFIGURATION_DESCRIPTOR) : BufferLength; + RtlCopyMemory(Buffer, &ROOTHUB2_CONFIGURATION_DESCRIPTOR, Length); + + // + // Check if we still have some space left + // + if(Length == BufferLength) { // - // buffer too small + // We copied all we could // Status = STATUS_SUCCESS; - ASSERT(FALSE); break; } + // + // Go further + // + Buffer += Length; + BufferLength -= Length; // // copy interface descriptor template // - Buffer = (PUCHAR)(ConfigurationDescriptor + 1); - RtlCopyMemory(Buffer, &ROOTHUB2_INTERFACE_DESCRIPTOR, sizeof(USB_INTERFACE_DESCRIPTOR)); - + Length = BufferLength > sizeof(USB_INTERFACE_DESCRIPTOR) ? + sizeof(USB_INTERFACE_DESCRIPTOR) : BufferLength; + RtlCopyMemory(Buffer, &ROOTHUB2_INTERFACE_DESCRIPTOR, Length); + + // + // Check if we still have some space left + // + if(Length == BufferLength) + { + // + // We copied all we could + // + Status = STATUS_SUCCESS; + break; + } + // + // Go further + // + Buffer += Length; + BufferLength -= Length; + + // // copy end point descriptor template // - Buffer += sizeof(USB_INTERFACE_DESCRIPTOR); - RtlCopyMemory(Buffer, &ROOTHUB2_ENDPOINT_DESCRIPTOR, sizeof(USB_ENDPOINT_DESCRIPTOR)); + Length = BufferLength > sizeof(USB_ENDPOINT_DESCRIPTOR) ? + sizeof(USB_ENDPOINT_DESCRIPTOR) : BufferLength; + RtlCopyMemory(Buffer, &ROOTHUB2_ENDPOINT_DESCRIPTOR, Length); // // done @@ -1621,25 +1646,34 @@ // get device // UsbDevice = PUSBDEVICE(Urb->UrbHeader.UsbdDeviceHandle); - - if (sizeof(USB_CONFIGURATION_DESCRIPTOR) > Urb->UrbControlDescriptorRequest.TransferBufferLength) + + // + // Allocate temporary buffer + // + BufferLength = UsbDevice->GetConfigurationDescriptorsLength(); + Buffer = (PUCHAR)ExAllocatePoolWithTag(NonPagedPool, BufferLength, TAG_USBLIB); + if(!Buffer) { - // - // buffer too small - // - Urb->UrbControlDescriptorRequest.TransferBufferLength = UsbDevice->GetConfigurationDescriptorsLength(); - - // - // bail out - // - Status = STATUS_SUCCESS; + Status = STATUS_NO_MEMORY; break; } // // perform work in IUSBDevice // - UsbDevice->GetConfigurationDescriptors((PUSB_CONFIGURATION_DESCRIPTOR)Urb->UrbControlDescriptorRequest.TransferBuffer, Urb->UrbControlDescriptorRequest.TransferBufferLength, &Length); + UsbDevice->GetConfigurationDescriptors((PUSB_CONFIGURATION_DESCRIPTOR)Buffer, BufferLength, &Length); + + // + // Copy what we can + // + Length = Urb->UrbControlDescriptorRequest.TransferBufferLength > Length ? + Length : Urb->UrbControlDescriptorRequest.TransferBufferLength; + RtlCopyMemory(Urb->UrbControlDescriptorRequest.TransferBuffer, Buffer, Length); + + // + // Free temporary buffer + // + ExFreePoolWithTag(Buffer, TAG_USBLIB); // // store result size

1 0

[jimtabor] 57311: [User32Test] - Update msg test to wine 1.5.13.
by jimtabor＠svn.reactos.org 16 Sep '12

16 Sep '12

Author: jimtabor Date: Sun Sep 16 01:09:00 2012 New Revision: 57311 URL: http://svn.reactos.org/svn/reactos?rev=57311&view=rev Log: [User32Test] - Update msg test to wine 1.5.13. Modified: trunk/rostests/winetests/user32/msg.c Modified: trunk/rostests/winetests/user32/msg.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/user32/msg.c?re… ============================================================================== --- trunk/rostests/winetests/user32/msg.c [iso-8859-1] (original) +++ trunk/rostests/winetests/user32/msg.c [iso-8859-1] Sun Sep 16 01:09:00 2012 @@ -1721,6 +1721,7 @@ static BOOL (WINAPI *pUnhookWinEvent)(HWINEVENTHOOK); static BOOL (WINAPI *pGetMonitorInfoA)(HMONITOR,LPMONITORINFO); static HMONITOR (WINAPI *pMonitorFromPoint)(POINT,DWORD); +static BOOL (WINAPI *pUpdateLayeredWindow)(HWND,HDC,POINT*,SIZE*,HDC,POINT*,COLORREF,BLENDFUNCTION*,DWORD); /* kernel32 functions */ static BOOL (WINAPI *pGetCPInfoExA)(UINT, DWORD, LPCPINFOEXA); @@ -1744,6 +1745,7 @@ GET_PROC(user32, UnhookWinEvent) GET_PROC(user32, GetMonitorInfoA) GET_PROC(user32, MonitorFromPoint) + GET_PROC(user32, UpdateLayeredWindow) GET_PROC(kernel32, GetCPInfoExA) @@ -13772,6 +13774,131 @@ CloseHandle(hthread); } +static const struct message WmSetLayeredStyle[] = { + { WM_STYLECHANGING, sent }, + { WM_STYLECHANGED, sent }, + { WM_GETTEXT, sent|defwinproc|optional }, + { 0 } +}; + +static const struct message WmSetLayeredStyle2[] = { + { WM_STYLECHANGING, sent }, + { WM_STYLECHANGED, sent }, + { WM_WINDOWPOSCHANGING, sent|optional|wparam|defwinproc, SWP_FRAMECHANGED|SWP_NOACTIVATE|SWP_NOSIZE|SWP_NOMOVE|SWP_NOCLIENTSIZE|SWP_NOCLIENTMOVE }, + { WM_NCCALCSIZE, sent|optional|wparam|defwinproc, 1 }, + { WM_WINDOWPOSCHANGED, sent|optional|wparam|defwinproc, SWP_FRAMECHANGED|SWP_NOACTIVATE|SWP_NOSIZE|SWP_NOMOVE }, + { WM_MOVE, sent|optional|defwinproc|wparam, 0 }, + { WM_SIZE, sent|optional|defwinproc|wparam, SIZE_RESTORED }, + { 0 } +}; + +static void test_layered_window(void) +{ + HWND hwnd; + HDC hdc; + HBITMAP bmp; + BOOL ret; + SIZE size; + POINT pos, src; + RECT rect, client; + + if (!pUpdateLayeredWindow) + { + win_skip( "UpdateLayeredWindow not supported\n" ); + return; + } + + hdc = CreateCompatibleDC( 0 ); + bmp = CreateCompatibleBitmap( hdc, 300, 300 ); + SelectObject( hdc, bmp ); + + hwnd = CreateWindowExA(0, "TestWindowClass", NULL, WS_CAPTION | WS_THICKFRAME | WS_SYSMENU, + 100, 100, 300, 300, 0, 0, 0, NULL); + ok( hwnd != 0, "failed to create window\n" ); + ShowWindow( hwnd, SW_SHOWNORMAL ); + UpdateWindow( hwnd ); + flush_events(); + flush_sequence(); + + GetWindowRect( hwnd, &rect ); + GetClientRect( hwnd, &client ); + ok( client.right < rect.right - rect.left, "wrong client area\n" ); + ok( client.bottom < rect.bottom - rect.top, "wrong client area\n" ); + + src.x = src.y = 0; + pos.x = pos.y = 300; + size.cx = size.cy = 250; + ret = pUpdateLayeredWindow( hwnd, 0, &pos, &size, hdc, &src, 0, NULL, ULW_OPAQUE ); + ok( !ret, "UpdateLayeredWindow should fail on non-layered window\n" ); + ok( GetLastError() == ERROR_INVALID_PARAMETER, "wrong error %u\n", GetLastError() ); + SetWindowLong( hwnd, GWL_EXSTYLE, GetWindowLong(hwnd, GWL_EXSTYLE) | WS_EX_LAYERED ); + ok_sequence( WmSetLayeredStyle, "WmSetLayeredStyle", FALSE ); + + ret = pUpdateLayeredWindow( hwnd, 0, &pos, &size, hdc, &src, 0, NULL, ULW_OPAQUE ); + ok( ret, "UpdateLayeredWindow failed err %u\n", GetLastError() ); + ok_sequence( WmEmptySeq, "UpdateLayeredWindow", FALSE ); + GetWindowRect( hwnd, &rect ); + ok( rect.left == 300 && rect.top == 300 && rect.right == 550 && rect.bottom == 550, + "wrong window rect %d,%d,%d,%d\n", rect.left, rect.top, rect.right, rect.bottom ); + GetClientRect( hwnd, &rect ); + ok( rect.right == client.right - 50 && rect.bottom == client.bottom - 50, + "wrong client rect %d,%d,%d,%d\n", rect.left, rect.top, rect.right, rect.bottom ); + + size.cx = 150; + pos.y = 200; + ret = pUpdateLayeredWindow( hwnd, 0, &pos, &size, hdc, &src, 0, NULL, ULW_OPAQUE ); + ok( ret, "UpdateLayeredWindow failed err %u\n", GetLastError() ); + ok_sequence( WmEmptySeq, "UpdateLayeredWindow", FALSE ); + GetWindowRect( hwnd, &rect ); + ok( rect.left == 300 && rect.top == 200 && rect.right == 450 && rect.bottom == 450, + "wrong window rect %d,%d,%d,%d\n", rect.left, rect.top, rect.right, rect.bottom ); + GetClientRect( hwnd, &rect ); + ok( rect.right == client.right - 150 && rect.bottom == client.bottom - 50, + "wrong client rect %d,%d,%d,%d\n", rect.left, rect.top, rect.right, rect.bottom ); + + SetWindowLong( hwnd, GWL_STYLE, + GetWindowLong(hwnd, GWL_STYLE) & ~(WS_CAPTION | WS_THICKFRAME | WS_SYSMENU) ); + ok_sequence( WmSetLayeredStyle2, "WmSetLayeredStyle2", FALSE ); + + size.cx = 200; + pos.x = 200; + ret = pUpdateLayeredWindow( hwnd, 0, &pos, &size, hdc, &src, 0, NULL, ULW_OPAQUE ); + ok( ret, "UpdateLayeredWindow failed err %u\n", GetLastError() ); + ok_sequence( WmEmptySeq, "UpdateLayeredWindow", FALSE ); + GetWindowRect( hwnd, &rect ); + ok( rect.left == 200 && rect.top == 200 && rect.right == 400 && rect.bottom == 450, + "wrong window rect %d,%d,%d,%d\n", rect.left, rect.top, rect.right, rect.bottom ); + GetClientRect( hwnd, &rect ); + ok( (rect.right == 200 && rect.bottom == 250) || + broken(rect.right == client.right - 100 && rect.bottom == client.bottom - 50), + "wrong client rect %d,%d,%d,%d\n", rect.left, rect.top, rect.right, rect.bottom ); + + size.cx = 0; + ret = pUpdateLayeredWindow( hwnd, 0, &pos, &size, hdc, &src, 0, NULL, ULW_OPAQUE ); + ok( !ret, "UpdateLayeredWindow should fail on non-layered window\n" ); + ok( GetLastError() == ERROR_INVALID_PARAMETER || broken(ERROR_MR_MID_NOT_FOUND) /* win7 */, + "wrong error %u\n", GetLastError() ); + size.cx = 1; + size.cy = -1; + ret = pUpdateLayeredWindow( hwnd, 0, &pos, &size, hdc, &src, 0, NULL, ULW_OPAQUE ); + ok( !ret, "UpdateLayeredWindow should fail on non-layered window\n" ); + ok( GetLastError() == ERROR_INVALID_PARAMETER, "wrong error %u\n", GetLastError() ); + + SetWindowLong( hwnd, GWL_EXSTYLE, GetWindowLong(hwnd, GWL_EXSTYLE) & ~WS_EX_LAYERED ); + ok_sequence( WmSetLayeredStyle, "WmSetLayeredStyle", FALSE ); + GetWindowRect( hwnd, &rect ); + ok( rect.left == 200 && rect.top == 200 && rect.right == 400 && rect.bottom == 450, + "wrong window rect %d,%d,%d,%d\n", rect.left, rect.top, rect.right, rect.bottom ); + GetClientRect( hwnd, &rect ); + ok( (rect.right == 200 && rect.bottom == 250) || + broken(rect.right == client.right - 100 && rect.bottom == client.bottom - 50), + "wrong client rect %d,%d,%d,%d\n", rect.left, rect.top, rect.right, rect.bottom ); + + DestroyWindow( hwnd ); + DeleteDC( hdc ); + DeleteObject( bmp ); +} + START_TEST(msg) { char **test_argv; @@ -13884,6 +14011,7 @@ test_clipboard_viewers(); test_keyflags(); test_hotkey(); + test_layered_window(); /* keep it the last test, under Windows it tends to break the tests * which rely on active/foreground windows being correct. */

1 0

[mnordell] 57310: Fix crash in xmlnodelist_QueryInterface. Solves CORE-6665.
by mnordell＠svn.reactos.org 15 Sep '12

15 Sep '12

Author: mnordell Date: Sat Sep 15 21:41:46 2012 New Revision: 57310 URL: http://svn.reactos.org/svn/reactos?rev=57310&view=rev Log: Fix crash in xmlnodelist_QueryInterface. Solves CORE-6665. Modified: trunk/reactos/dll/win32/msxml3/msxml3_ros.diff trunk/reactos/dll/win32/msxml3/nodelist.c Modified: trunk/reactos/dll/win32/msxml3/msxml3_ros.diff URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/msxml3/msxml3_ro… ============================================================================== --- trunk/reactos/dll/win32/msxml3/msxml3_ros.diff [iso-8859-1] (original) +++ trunk/reactos/dll/win32/msxml3/msxml3_ros.diff [iso-8859-1] Sat Sep 15 21:41:46 2012 @@ -20,3 +20,21 @@ @ stdcall -private DllCanUnloadNow() @ stdcall -private DllGetClassObject(ptr ptr ptr) + + +Index: nodelist.c +=================================================================== +--- nodelist.c (revision 57307) ++++ nodelist.c (working copy) +@@ -87,6 +87,11 @@ + + TRACE("(%p)->(%s %p)\n", This, debugstr_guid(riid), ppvObject); + ++ if (!ppvObject) ++ { ++ return E_POINTER; ++ } ++ + if ( IsEqualGUID( riid, &IID_IUnknown ) || + IsEqualGUID( riid, &IID_IDispatch ) || + IsEqualGUID( riid, &IID_IXMLDOMNodeList ) ) Modified: trunk/reactos/dll/win32/msxml3/nodelist.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/msxml3/nodelist.… ============================================================================== --- trunk/reactos/dll/win32/msxml3/nodelist.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/msxml3/nodelist.c [iso-8859-1] Sat Sep 15 21:41:46 2012 @@ -87,6 +87,11 @@ TRACE("(%p)->(%s %p)\n", This, debugstr_guid(riid), ppvObject); + if (!ppvObject) + { + return E_POINTER; + } + if ( IsEqualGUID( riid, &IID_IUnknown ) || IsEqualGUID( riid, &IID_IDispatch ) || IsEqualGUID( riid, &IID_IXMLDOMNodeList ) )

1 0

[mnordell] 57309: Disable crashing usermode test. See CORE-6640.
by mnordell＠svn.reactos.org 15 Sep '12

15 Sep '12

Author: mnordell Date: Sat Sep 15 19:39:14 2012 New Revision: 57309 URL: http://svn.reactos.org/svn/reactos?rev=57309&view=rev Log: Disable crashing usermode test. See CORE-6640. Modified: trunk/rostests/kmtests/rtl/RtlException.c Modified: trunk/rostests/kmtests/rtl/RtlException.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/kmtests/rtl/RtlException.… ============================================================================== --- trunk/rostests/kmtests/rtl/RtlException.c [iso-8859-1] (original) +++ trunk/rostests/kmtests/rtl/RtlException.c [iso-8859-1] Sat Sep 15 19:39:14 2012 @@ -43,6 +43,7 @@ /* We cannot test this in kernel mode easily - the stack is just "somewhere" * in system space, and there's no guard page below it */ +#if CORE_6640_IS_FIXED #ifdef KMT_USER_MODE /* Overflow the stack - must cause a special exception */ KmtStartSeh() @@ -55,4 +56,5 @@ } KmtEndSeh(STATUS_STACK_OVERFLOW); #endif +#endif /* CORE_6640_IS_FIXED */ }

1 0

[mnordell] 57308: Disable more crashing font tests.
by mnordell＠svn.reactos.org 15 Sep '12

15 Sep '12

Author: mnordell Date: Sat Sep 15 19:31:48 2012 New Revision: 57308 URL: http://svn.reactos.org/svn/reactos?rev=57308&view=rev Log: Disable more crashing font tests. Modified: trunk/rostests/winetests/gdiplus/font.c trunk/rostests/winetests/gdiplus/gdiplus_winetest_ros.diff Modified: trunk/rostests/winetests/gdiplus/font.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/gdiplus/font.c?… ============================================================================== --- trunk/rostests/winetests/gdiplus/font.c [iso-8859-1] (original) +++ trunk/rostests/winetests/gdiplus/font.c [iso-8859-1] Sat Sep 15 19:31:48 2012 @@ -87,6 +87,7 @@ GdipDeleteFontFamily(fontfamily); } +#if CORE_6660_IS_FIXED static void test_logfont(void) { LOGFONTA lfa, lfa2; @@ -274,6 +275,7 @@ GdipDeleteFontFamily(clonedFontFamily); } +#endif // CORE_6660_IS_FIXED static void test_fontfamily_properties (void) { @@ -740,8 +742,10 @@ test_font_metrics(); #endif test_createfont(); +#if CORE_6660_IS_FIXED test_logfont(); test_fontfamily(); +#endif test_fontfamily_properties(); test_getgenerics(); test_installedfonts(); Modified: trunk/rostests/winetests/gdiplus/gdiplus_winetest_ros.diff URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/gdiplus/gdiplus… ============================================================================== --- trunk/rostests/winetests/gdiplus/gdiplus_winetest_ros.diff [iso-8859-1] (original) +++ trunk/rostests/winetests/gdiplus/gdiplus_winetest_ros.diff [iso-8859-1] Sat Sep 15 19:31:48 2012 @@ -31,31 +31,32 @@ Index: font.c =================================================================== ---- font.c (revision 57206) +--- font.c (revision 57301) +++ font.c (working copy) -@@ -485,6 +485,7 @@ +@@ -87,6 +87,7 @@ GdipDeleteFontFamily(fontfamily); } +#if CORE_6660_IS_FIXED - static int CALLBACK font_enum_proc(const LOGFONTW *lfe, const TEXTMETRICW *ntme, - DWORD type, LPARAM lparam) + static void test_logfont(void) { -@@ -721,6 +722,7 @@ - GdipDeleteGraphics(graphics); - DeleteDC(hdc); + LOGFONTA lfa, lfa2; +@@ -274,6 +275,7 @@ + + GdipDeleteFontFamily(clonedFontFamily); } +#endif // CORE_6660_IS_FIXED - START_TEST(font) + static void test_fontfamily_properties (void) { -@@ -734,7 +736,9 @@ - - GdiplusStartup(&gdiplusToken, &gdiplusStartupInput, NULL); - +@@ -740,8 +742,10 @@ + test_font_metrics(); + #endif + test_createfont(); +#if CORE_6660_IS_FIXED - test_font_metrics(); -+#endif - test_createfont(); test_logfont(); test_fontfamily(); ++#endif + test_fontfamily_properties(); + test_getgenerics(); + test_installedfonts();

1 0

[tkreuzer] 57307: {CSRSRV] Silence an uninformative DPRINT
by tkreuzer＠svn.reactos.org 15 Sep '12

15 Sep '12

Author: tkreuzer Date: Sat Sep 15 16:33:30 2012 New Revision: 57307 URL: http://svn.reactos.org/svn/reactos?rev=57307&view=rev Log: {CSRSRV] Silence an uninformative DPRINT Modified: trunk/reactos/subsystems/win32/csrss/csrsrv/api/wapi.c Modified: trunk/reactos/subsystems/win32/csrss/csrsrv/api/wapi.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/csrss/csr… ============================================================================== --- trunk/reactos/subsystems/win32/csrss/csrsrv/api/wapi.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/csrss/csrsrv/api/wapi.c [iso-8859-1] Sat Sep 15 16:33:30 2012 @@ -187,7 +187,7 @@ DPRINT1("CSRSS: %s Api Request received from server process\n", ServerDll->NameTable[ApiId]); } - + /* Validation complete, start SEH */ _SEH2_TRY { @@ -433,7 +433,7 @@ 0, 0, 0, 0, 0, 0, 0, &RestrictedSid); ASSERT(NT_SUCCESS(Status)); - + /* Allocate one ACL with 3 ACEs each for one SID */ AclLength = sizeof(ACL) + 3 * sizeof(ACCESS_ALLOWED_ACE) + RtlLengthSid(SystemSid) + @@ -471,7 +471,7 @@ ASSERT(NT_SUCCESS(Status)); Status = RtlAddAccessAllowedAce(*RestrictedDacl, ACL_REVISION2, DIRECTORY_TRAVERSE, RestrictedSid); ASSERT(NT_SUCCESS(Status)); - + /* The SIDs are captured, can free them now */ RtlFreeHeap(CsrHeap, 0, SystemSid); RtlFreeHeap(CsrHeap, 0, WorldSid); @@ -511,7 +511,7 @@ }, {0} }; - + /* Get the session ID */ SessionId = NtCurrentPeb()->SessionId; @@ -641,7 +641,7 @@ ASSERT(BnoSd); Status = RtlCreateSecurityDescriptor(BnoSd, SECURITY_DESCRIPTOR_REVISION); ASSERT(NT_SUCCESS(Status)); - + /* Create the BNO and \Restricted DACLs */ Status = CreateBaseAcls(&BnoDacl, &BnoRestrictedDacl); ASSERT(NT_SUCCESS(Status)); @@ -769,7 +769,7 @@ DPRINT1("CSRSRV: Unknown process: %lx. Will be rejecting connection\n", Request->ClientId.UniqueProcess); } - + if ((ProcessData) && (ProcessData != CsrRootProcess)) { /* Attach the Shared Section */ @@ -1068,13 +1068,13 @@ continue; } } - + /* Use whatever Client ID we got */ Teb->RealClientId = Request->Header.ClientId; /* Get the Message Type */ MessageType = Request->Header.u2.s2.Type; - + /* Handle connection requests */ if (MessageType == LPC_CONNECTION_REQUEST) { @@ -1085,7 +1085,7 @@ ReplyPort = CsrApiPort; continue; } - + /* It's some other kind of request. Get the lock for the lookup */ CsrAcquireProcessLock(); @@ -1098,7 +1098,7 @@ { /* This wasn't a CSR Thread, release lock */ CsrReleaseProcessLock(); - + /* If this was an exception, handle it */ if (MessageType == LPC_EXCEPTION) { @@ -1111,7 +1111,7 @@ MessageType == LPC_CLIENT_DIED) { /* The Client or Port are gone, loop again */ - DPRINT1("Death from unknown thread, just continue\n"); + DPRINT("Death from unknown thread, just continue\n"); Reply = NULL; ReplyPort = CsrApiPort; } @@ -1131,7 +1131,7 @@ { CsrHandleHardError(CsrThread, (PHARDERROR_MSG)Request); } - + /* If the response was 0xFFFFFFFF, we'll ignore it */ if (HardErrorMsg->Response == 0xFFFFFFFF) { @@ -1172,7 +1172,7 @@ /* Keep going */ continue; } - + /* We have a valid thread, was this an LPC Request? */ if (MessageType != LPC_REQUEST) { @@ -1207,7 +1207,7 @@ ReplyPort = CsrApiPort; continue; } - + /* Reference the thread and release the lock */ CsrLockedReferenceThread(CsrThread); CsrReleaseProcessLock(); @@ -1253,14 +1253,14 @@ /* We got an API Request */ CsrLockedReferenceThread(CsrThread); CsrReleaseProcessLock(); - + /* Assume success */ Reply = Request; Request->Status = STATUS_SUCCESS; /* Now we reply to a particular client */ ReplyPort = CsrThread->Process->ClientPort; - + DPRINT("CSR: Got CSR API: %x [Message Origin: %x]\n", Request->Type, Request->Header.ClientId.UniqueThread); @@ -1270,19 +1270,19 @@ { /* Make sure we have enough threads */ CsrpCheckRequestThreads(); - + /* Set the client thread pointer */ NtCurrentTeb()->CsrClientThread = CsrThread; /* Call the Handler */ CsrApiCallHandler(CsrThread->Process, Request); - + /* Increase the static thread count */ _InterlockedIncrement(&CsrpStaticThreadCount); /* Restore the server thread */ NtCurrentTeb()->CsrClientThread = ServerThread; - + /* Check if this is a dead client now */ if (Request->Type == 0xBABE) {

1 0

[tkreuzer] 57306: [WPP] Fix WDK build.
by tkreuzer＠svn.reactos.org 15 Sep '12

15 Sep '12

Author: tkreuzer Date: Sat Sep 15 16:15:54 2012 New Revision: 57306 URL: http://svn.reactos.org/svn/reactos?rev=57306&view=rev Log: [WPP] Fix WDK build. Modified: trunk/reactos/tools/wpp/CMakeLists.txt Modified: trunk/reactos/tools/wpp/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/wpp/CMakeLists.txt?r… ============================================================================== --- trunk/reactos/tools/wpp/CMakeLists.txt [iso-8859-1] (original) +++ trunk/reactos/tools/wpp/CMakeLists.txt [iso-8859-1] Sat Sep 15 16:15:54 2012 @@ -5,8 +5,8 @@ -Dstrtoull=_strtoui64 -Dstrtoll=_strtoi64) - # VS 9 doesn't like this - if (NOT MSVC90) + # Add this definition for WDK only, VS 9 doesn't like that + if (DEFINED ENV{DDKBUILDENV}) add_definitions( -Dvsnprintf=_vsnprintf) endif()

1 0

[akhaldi] 57305: * Change newinflib to inflib. CORE-6662 #resolve
by akhaldi＠svn.reactos.org 15 Sep '12

15 Sep '12

Author: akhaldi Date: Sat Sep 15 16:04:32 2012 New Revision: 57305 URL: http://svn.reactos.org/svn/reactos?rev=57305&view=rev Log: * Change newinflib to inflib. CORE-6662 #resolve Added: trunk/reactos/lib/inflib/ - copied from r57303, trunk/reactos/lib/newinflib/ Removed: trunk/reactos/lib/newinflib/ Modified: trunk/reactos/base/setup/usetup/CMakeLists.txt trunk/reactos/lib/CMakeLists.txt trunk/reactos/tools/mkhive/CMakeLists.txt Modified: trunk/reactos/base/setup/usetup/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/setup/usetup/CMakeLis… ============================================================================== --- trunk/reactos/base/setup/usetup/CMakeLists.txt [iso-8859-1] (original) +++ trunk/reactos/base/setup/usetup/CMakeLists.txt [iso-8859-1] Sat Sep 15 16:04:32 2012 @@ -2,7 +2,7 @@ add_definitions(${I18N_DEFS}) include_directories( ${CMAKE_CURRENT_SOURCE_DIR} - ${REACTOS_SOURCE_DIR}/lib/newinflib + ${REACTOS_SOURCE_DIR}/lib/inflib ${REACTOS_SOURCE_DIR}/include/reactos/libs/zlib ${REACTOS_SOURCE_DIR}/include/reactos/drivers) Modified: trunk/reactos/lib/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/CMakeLists.txt?rev=573… ============================================================================== --- trunk/reactos/lib/CMakeLists.txt [iso-8859-1] (original) +++ trunk/reactos/lib/CMakeLists.txt [iso-8859-1] Sat Sep 15 16:04:32 2012 @@ -1,6 +1,6 @@ add_subdirectory(cmlib) -add_subdirectory(newinflib) +add_subdirectory(inflib) if(CMAKE_CROSSCOMPILING) Modified: trunk/reactos/tools/mkhive/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/mkhive/CMakeLists.tx… ============================================================================== --- trunk/reactos/tools/mkhive/CMakeLists.txt [iso-8859-1] (original) +++ trunk/reactos/tools/mkhive/CMakeLists.txt [iso-8859-1] Sat Sep 15 16:04:32 2012 @@ -2,7 +2,7 @@ add_definitions(-DMKHIVE_HOST) include_directories( - ${REACTOS_SOURCE_DIR}/lib/newinflib + ${REACTOS_SOURCE_DIR}/lib/inflib ${REACTOS_SOURCE_DIR}/lib/cmlib ${REACTOS_SOURCE_DIR}/lib/rtl)

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