Ros-diffs

ros-diffs@reactos.org

81390 discussions

[jimtabor] 72652: [USER32] - Sync with Wine Staging 1.9.18.
by jimtabor＠svn.reactos.org 11 Sep '16

11 Sep '16

Author: jimtabor Date: Sun Sep 11 00:58:12 2016 New Revision: 72652 URL: http://svn.reactos.org/svn/reactos?rev=72652&view=rev Log: [USER32] - Sync with Wine Staging 1.9.18. Modified: trunk/reactos/win32ss/user/user32/controls/listbox.c Modified: trunk/reactos/win32ss/user/user32/controls/listbox.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/user32/contro… ============================================================================== --- trunk/reactos/win32ss/user/user32/controls/listbox.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/user32/controls/listbox.c [iso-8859-1] Sun Sep 11 00:58:12 2016 @@ -1069,14 +1069,10 @@ else rect.bottom = rect.top + descr->items[i].height; + /* keep the focus rect, to paint the focus item after */ if (i == descr->focus_item) - { - /* keep the focus rect, to paint the focus item after */ - focusRect.left = rect.left; - focusRect.right = rect.right; - focusRect.top = rect.top; - focusRect.bottom = rect.bottom; - } + focusRect = rect; + LISTBOX_PaintItem( descr, hdc, &rect, i, ODA_DRAWENTIRE, TRUE ); rect.top = rect.bottom;

1 0

[jimtabor] 72651: [USER32] - Sync with Wine Staging 1.9.18.
by jimtabor＠svn.reactos.org 11 Sep '16

11 Sep '16

Author: jimtabor Date: Sun Sep 11 00:58:00 2016 New Revision: 72651 URL: http://svn.reactos.org/svn/reactos?rev=72651&view=rev Log: [USER32] - Sync with Wine Staging 1.9.18. Modified: trunk/reactos/win32ss/user/user32/controls/edit.c Modified: trunk/reactos/win32ss/user/user32/controls/edit.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/user32/contro… ============================================================================== --- trunk/reactos/win32ss/user/user32/controls/edit.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/user32/controls/edit.c [iso-8859-1] Sun Sep 11 00:58:00 2016 @@ -1,9 +1,9 @@ /* - * Edit control - * - * Copyright David W. Metcalfe, 1994 - * Copyright William Magro, 1995, 1996 - * Copyright Frans van Dorsselaer, 1996, 1997 + * Edit control + * + * Copyright David W. Metcalfe, 1994 + * Copyright William Magro, 1995, 1996 + * Copyright Frans van Dorsselaer, 1996, 1997 * * * This library is free software; you can redistribute it and/or @@ -2904,30 +2904,30 @@ /********************************************************************* * - * EM_SETMARGINS + * EM_SETMARGINS * * EC_USEFONTINFO is used as a left or right value i.e. lParam and not as an * action wParam despite what the docs say. EC_USEFONTINFO calculates the * margin according to the textmetrics of the current font. * - * FIXME - With TrueType or vector fonts EC_USEFONTINFO currently sets one third - * of the char's width as the margin, but this is not how Windows handles this. - * For all other fonts Windows sets the margins to zero. - * - * FIXME - When EC_USEFONTINFO is used the margins only change if the - * edit control is equal to or larger than a certain size. - * Interestingly if one subtracts both the left and right margins from - * this size one always seems to get an even number. The extents of - * the (four character) string "'**'" match this quite closely, so - * we'll use this until we come up with a better idea. - */ -static int calc_min_set_margin_size(HDC dc, INT left, INT right) -{ - static const WCHAR magic_string[] = {'\'','*','*','\'', 0}; - SIZE sz; - - GetTextExtentPointW(dc, magic_string, sizeof(magic_string)/sizeof(WCHAR) - 1, &sz); - return sz.cx + left + right; + * When EC_USEFONTINFO is used in the non_cjk case the margins only + * change if the edit control is equal to or larger than a certain + * size. Though there is an exception for the empty client rect case + * with small font sizes. + */ +static BOOL is_cjk(UINT charset) +{ + switch(charset) + { + case SHIFTJIS_CHARSET: + case HANGUL_CHARSET: + case GB2312_CHARSET: + case CHINESEBIG5_CHARSET: + return TRUE; + } + /* HANGUL_CHARSET is strange, though treated as CJK by Win 8, it is + * not by other versions including Win 10. */ + return FALSE; } static void EDIT_EM_SetMargins(EDITSTATE *es, INT action, @@ -2941,19 +2941,25 @@ if (es->font && (left == EC_USEFONTINFO || right == EC_USEFONTINFO)) { HDC dc = GetDC(es->hwndSelf); HFONT old_font = SelectObject(dc, es->font); - GetTextMetricsW(dc, &tm); + LONG width = GdiGetCharDimensions(dc, &tm, NULL); + RECT rc; + /* The default margins are only non zero for TrueType or Vector fonts */ if (tm.tmPitchAndFamily & ( TMPF_VECTOR | TMPF_TRUETYPE )) { - int min_size; - RECT rc; - /* This must be calculated more exactly! But how? */ - default_left_margin = tm.tmAveCharWidth / 2; - default_right_margin = tm.tmAveCharWidth / 2; - min_size = calc_min_set_margin_size(dc, default_left_margin, default_right_margin); - GetClientRect(es->hwndSelf, &rc); - if (!IsRectEmpty(&rc) && (rc.right - rc.left < min_size)) { - default_left_margin = es->left_margin; - default_right_margin = es->right_margin; + if (!is_cjk(tm.tmCharSet)) { + default_left_margin = width / 2; + default_right_margin = width / 2; + + GetClientRect(es->hwndSelf, &rc); + if (rc.right - rc.left < (width / 2 + width) * 2 && + (width >= 28 || !IsRectEmpty(&rc)) ) { + default_left_margin = es->left_margin; + default_right_margin = es->right_margin; + } + } else { + /* FIXME: figure out the CJK values. They are not affected by the client rect. */ + default_left_margin = width / 2; + default_right_margin = width / 2; } } SelectObject(dc, old_font);

1 0

[jimtabor] 72650: [USER32] - Sync with Wine Staging 1.9.18.
by jimtabor＠svn.reactos.org 11 Sep '16

11 Sep '16

Author: jimtabor Date: Sun Sep 11 00:57:38 2016 New Revision: 72650 URL: http://svn.reactos.org/svn/reactos?rev=72650&view=rev Log: [USER32] - Sync with Wine Staging 1.9.18. Modified: trunk/reactos/win32ss/user/user32/controls/button.c Modified: trunk/reactos/win32ss/user/user32/controls/button.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/user32/contro… ============================================================================== --- trunk/reactos/win32ss/user/user32/controls/button.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/user32/controls/button.c [iso-8859-1] Sun Sep 11 00:57:38 2016 @@ -135,10 +135,6 @@ OB_Paint /* BS_OWNERDRAW */ }; -static HBITMAP hbitmapCheckBoxes = 0; -static WORD checkBoxWidth = 0, checkBoxHeight = 0; - - /********************************************************************* * button class descriptor */ @@ -272,7 +268,8 @@ } } } - + else + return 0; #else if (!IsWindow( hWnd )) return 0; #endif @@ -299,14 +296,6 @@ break; case WM_CREATE: - if (!hbitmapCheckBoxes) - { - BITMAP bmp; - hbitmapCheckBoxes = LoadBitmapW(0, MAKEINTRESOURCEW(OBM_CHECKBOXES)); - GetObjectW( hbitmapCheckBoxes, sizeof(bmp), &bmp ); - checkBoxWidth = bmp.bmWidth / 4; - checkBoxHeight = bmp.bmHeight / 3; - } if (btn_type >= MAX_BTN_TYPE) return -1; /* abort */ @@ -400,8 +389,9 @@ break; /* fall through */ case WM_LBUTTONUP: - { +#ifdef _REACTOS_ BOOL TellParent = FALSE; //// ReactOS see note below. +#endif state = get_button_state( hWnd ); if (!(state & BUTTON_BTNPRESSED)) break; state &= BUTTON_NSTATES; @@ -429,15 +419,27 @@ (state & BST_INDETERMINATE) ? 0 : ((state & 3) + 1), 0 ); break; } +#ifdef _REACTOS_ TellParent = TRUE; // <---- Fix CORE-10194, Notify parent after capture is released. - } +#else + ReleaseCapture(); + BUTTON_NOTIFY_PARENT(hWnd, BN_CLICKED); +#endif + } +#ifdef _REACTOS_ ReleaseCapture(); if (TellParent) BUTTON_NOTIFY_PARENT(hWnd, BN_CLICKED); - } +#else + else + { + ReleaseCapture(); + } +#endif break; case WM_CAPTURECHANGED: TRACE("WM_CAPTURECHANGED %p\n", hWnd); + if (hWnd == (HWND)lParam) break; state = get_button_state( hWnd ); if (state & BUTTON_BTNPRESSED) { @@ -459,36 +461,37 @@ { /* Clear an old text here as Windows does */ // +// ReactOS Note : // wine Bug: http://bugs.winehq.org/show_bug.cgi?id=25790 // Patch: http://source.winehq.org/patches/data/70889 // By: Alexander LAW, Replicate Windows behavior of WM_SETTEXT handler regarding WM_CTLCOLOR* // #ifdef __REACTOS__ if (style & WS_VISIBLE) - { -#endif +#else + if (IsWindowVisible(hWnd)) +#endif + { HDC hdc = GetDC(hWnd); HBRUSH hbrush; RECT client, rc; HWND parent = GetParent(hWnd); -#ifdef __REACTOS__ - UINT ctlMessage = (btn_type == BS_PUSHBUTTON || - btn_type == BS_DEFPUSHBUTTON || - btn_type == BS_PUSHLIKE || - btn_type == BS_USERBUTTON || - btn_type == BS_OWNERDRAW) ? - WM_CTLCOLORBTN : WM_CTLCOLORSTATIC; -#endif + UINT message = (btn_type == BS_PUSHBUTTON || + btn_type == BS_DEFPUSHBUTTON || + btn_type == BS_PUSHLIKE || + btn_type == BS_USERBUTTON || + btn_type == BS_OWNERDRAW) ? + WM_CTLCOLORBTN : WM_CTLCOLORSTATIC; if (!parent) parent = hWnd; #ifdef __REACTOS__ - hbrush = GetControlColor(parent, hWnd, hdc, ctlMessage); -#else - hbrush = (HBRUSH)SendMessageW(parent, WM_CTLCOLORSTATIC, - (WPARAM)hdc, (LPARAM)hWnd); - if (!hbrush) /* did the app forget to call DefWindowProc ? */ - hbrush = (HBRUSH)DefWindowProcW(parent, WM_CTLCOLORSTATIC, - (WPARAM)hdc, (LPARAM)hWnd); + hbrush = GetControlColor(parent, hWnd, hdc, message); +#else + hbrush = (HBRUSH)SendMessageW(parent, message, + (WPARAM)hdc, (LPARAM)hWnd); + if (!hbrush) /* did the app forget to call DefWindowProc ? */ + hbrush = (HBRUSH)DefWindowProcW(parent, message, + (WPARAM)hdc, (LPARAM)hWnd); #endif GetClientRect(hWnd, &client); @@ -499,9 +502,7 @@ if (rc.bottom > client.bottom) rc.bottom = client.bottom; FillRect(hdc, &rc, hbrush); ReleaseDC(hWnd, hdc); -#ifdef __REACTOS__ - } -#endif + } if (unicode) DefWindowProcW( hWnd, WM_SETTEXT, wParam, lParam ); else DefWindowProcA( hWnd, WM_SETTEXT, wParam, lParam ); @@ -842,11 +843,8 @@ static BOOL CALLBACK BUTTON_DrawTextCallback(HDC hdc, LPARAM lp, WPARAM wp, int cx, int cy) { RECT rc; - rc.left = 0; - rc.top = 0; - rc.right = cx; - rc.bottom = cy; - + + SetRect(&rc, 0, 0, cx, cy); DrawTextW(hdc, (LPCWSTR)lp, -1, &rc, (UINT)wp); return TRUE; } @@ -1026,7 +1024,7 @@ { RECT rbox, rtext, client; HBRUSH hBrush; - int delta; + int delta, text_offset, checkBoxWidth, checkBoxHeight; UINT dtFlags; HFONT hFont; LONG state = get_button_state( hwnd ); @@ -1043,7 +1041,12 @@ GetClientRect(hwnd, &client); rbox = rtext = client; + checkBoxWidth = 12 * GetDeviceCaps( hDC, LOGPIXELSX ) / 96 + 1; + checkBoxHeight = 12 * GetDeviceCaps( hDC, LOGPIXELSY ) / 96 + 1; + if ((hFont = get_button_font( hwnd ))) SelectObject( hDC, hFont ); + GetCharWidthW( hDC, '0', '0', &text_offset ); + text_offset /= 2; parent = GetParent(hwnd); if (!parent) parent = hwnd; @@ -1062,12 +1065,12 @@ { /* magic +4 is what CTL3D expects */ - rtext.right -= checkBoxWidth + 4; + rtext.right -= checkBoxWidth + text_offset;; rbox.left = rbox.right - checkBoxWidth; } else { - rtext.left += checkBoxWidth + 4; + rtext.left += checkBoxWidth + text_offset;; rbox.right = checkBoxWidth; }

1 0

[dchapyshev] 72649: [NtUser] Unnecessary check is removed and correct value of an error code is returned * Fixes 2 tests for win32knt_apitest:NtUserEnumDisplaySettings
by dchapyshev＠svn.reactos.org 10 Sep '16

10 Sep '16

Author: dchapyshev Date: Sat Sep 10 22:30:53 2016 New Revision: 72649 URL: http://svn.reactos.org/svn/reactos?rev=72649&view=rev Log: [NtUser] Unnecessary check is removed and correct value of an error code is returned * Fixes 2 tests for win32knt_apitest:NtUserEnumDisplaySettings Modified: trunk/reactos/win32ss/user/ntuser/display.c Modified: trunk/reactos/win32ss/user/ntuser/display.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/displa… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/display.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/display.c [iso-8859-1] Sat Sep 10 22:30:53 2016 @@ -477,9 +477,6 @@ return STATUS_UNSUCCESSFUL; } - if (iModeNum >= pGraphicsDevice->cDevModes) - return STATUS_NO_MORE_ENTRIES; - iFoundMode = 0; for (i = 0; i < pGraphicsDevice->cDevModes; i++) { @@ -504,7 +501,7 @@ } /* Nothing was found */ - return STATUS_INVALID_PARAMETER; + return STATUS_INVALID_PARAMETER_2; } NTSTATUS

1 0

[dchapyshev] 72648: [NtUser] Fix 3 new tests for GET/SET NONCLIENTMETRICS, MINIMIZEDMETRICS, ICONMETRICS
by dchapyshev＠svn.reactos.org 10 Sep '16

10 Sep '16

Author: dchapyshev Date: Sat Sep 10 18:39:40 2016 New Revision: 72648 URL: http://svn.reactos.org/svn/reactos?rev=72648&view=rev Log: [NtUser] Fix 3 new tests for GET/SET NONCLIENTMETRICS, MINIMIZEDMETRICS, ICONMETRICS Modified: trunk/reactos/win32ss/user/ntuser/sysparams.c Modified: trunk/reactos/win32ss/user/ntuser/sysparams.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/syspar… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/sysparams.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/sysparams.c [iso-8859-1] Sat Sep 10 18:39:40 2016 @@ -912,7 +912,12 @@ case SPI_SETNONCLIENTMETRICS: { - if (!SpiSet(&gspv.ncm, pvParam, sizeof(NONCLIENTMETRICSW), fl)) + LPNONCLIENTMETRICSW metrics = (LPNONCLIENTMETRICSW)pvParam; + + /* Fixup user's structure size */ + metrics->cbSize = sizeof(NONCLIENTMETRICSW); + + if (!SpiSet(&gspv.ncm, metrics, sizeof(NONCLIENTMETRICSW), fl)) return 0; if (fl & SPIF_UPDATEINIFILE) @@ -949,7 +954,12 @@ case SPI_SETMINIMIZEDMETRICS: { - if (!SpiSet(&gspv.mm, pvParam, sizeof(MINIMIZEDMETRICS), fl)) + LPMINIMIZEDMETRICS metrics = (LPMINIMIZEDMETRICS)pvParam; + + /* Fixup user's structure size */ + metrics->cbSize = sizeof(MINIMIZEDMETRICS); + + if (!SpiSet(&gspv.mm, metrics, sizeof(MINIMIZEDMETRICS), fl)) return 0; gspv.mm.iWidth = max(0, gspv.mm.iWidth); @@ -975,7 +985,12 @@ case SPI_SETICONMETRICS: { - if (!SpiSet(&gspv.im, pvParam, sizeof(ICONMETRICSW), fl)) + LPICONMETRICSW metrics = (LPICONMETRICSW)pvParam; + + /* Fixup user's structure size */ + metrics->cbSize = sizeof(ICONMETRICSW); + + if (!SpiSet(&gspv.im, metrics, sizeof(ICONMETRICSW), fl)) return 0; if (fl & SPIF_UPDATEINIFILE)

1 0

[dchapyshev] 72647: [WIN32KNT_APITEST] Add some tests for GET/SET NONCLIENTMETRICS, MINIMIZEDMETRICS, ICONMETRICS. It is checked in win2k3
by dchapyshev＠svn.reactos.org 10 Sep '16

10 Sep '16

Author: dchapyshev Date: Sat Sep 10 18:36:56 2016 New Revision: 72647 URL: http://svn.reactos.org/svn/reactos?rev=72647&view=rev Log: [WIN32KNT_APITEST] Add some tests for GET/SET NONCLIENTMETRICS, MINIMIZEDMETRICS, ICONMETRICS. It is checked in win2k3 Modified: trunk/rostests/apitests/win32nt/ntuser/NtUserSystemParametersInfo.c Modified: trunk/rostests/apitests/win32nt/ntuser/NtUserSystemParametersInfo.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/win32nt/ntuser/N… ============================================================================== --- trunk/rostests/apitests/win32nt/ntuser/NtUserSystemParametersInfo.c [iso-8859-1] (original) +++ trunk/rostests/apitests/win32nt/ntuser/NtUserSystemParametersInfo.c [iso-8859-1] Sat Sep 10 18:36:56 2016 @@ -716,8 +716,6 @@ /* Test uiParam < 0 */ TEST(NtUserSystemParametersInfo(SPI_GETICONTITLELOGFONT, -1, &buf.lf, 0) == 1); - - } void @@ -740,30 +738,60 @@ Test_SPI_SETNONCLIENTMETRICS(void) { NONCLIENTMETRICSW metrics; + NONCLIENTMETRICSW origMetrics; metrics.cbSize = sizeof(NONCLIENTMETRICSW); TEST(NtUserSystemParametersInfo(SPI_GETNONCLIENTMETRICS, sizeof(NONCLIENTMETRICSW), &metrics, 0) == 1); TEST(NtUserSystemParametersInfo(SPI_GETNONCLIENTMETRICS, sizeof(NONCLIENTMETRICSW), (PVOID)0xdeadbeef, 0) == 0); + + origMetrics = metrics; + + metrics.cbSize = sizeof(NONCLIENTMETRICSW) + 10; + TEST(NtUserSystemParametersInfo(SPI_SETNONCLIENTMETRICS, 0, (PVOID)&metrics, 0) == 1); + TEST(NtUserSystemParametersInfo(SPI_GETNONCLIENTMETRICS, 0, (PVOID)&metrics, 0) == 1); + ok(metrics.cbSize == sizeof(NONCLIENTMETRICSW), "Expected size: %lu, got %lu\n", (ULONG)sizeof(NONCLIENTMETRICSW), (ULONG)metrics.cbSize); + + TEST(NtUserSystemParametersInfo(SPI_SETNONCLIENTMETRICS, 0, (PVOID)&origMetrics, 0) == 1); } void Test_SPI_SETMINIMIZEDMETRICS(void) { MINIMIZEDMETRICS metrics; + MINIMIZEDMETRICS origMetrics; metrics.cbSize = sizeof(MINIMIZEDMETRICS); TEST(NtUserSystemParametersInfo(SPI_GETMINIMIZEDMETRICS, sizeof(MINIMIZEDMETRICS), (PVOID)&metrics, 0) == 1); TEST(NtUserSystemParametersInfo(SPI_GETMINIMIZEDMETRICS, sizeof(MINIMIZEDMETRICS), (PVOID)0xdeadbeef, 0) == 0); + + origMetrics = metrics; + + metrics.cbSize = sizeof(MINIMIZEDMETRICS) + 10; + TEST(NtUserSystemParametersInfo(SPI_SETMINIMIZEDMETRICS, 0, (PVOID)&metrics, 0) == 1); + TEST(NtUserSystemParametersInfo(SPI_GETMINIMIZEDMETRICS, 0, (PVOID)&metrics, 0) == 1); + ok(metrics.cbSize == sizeof(MINIMIZEDMETRICS), "Expected size: %lu, got %lu\n", (ULONG)sizeof(MINIMIZEDMETRICS), (ULONG)metrics.cbSize); + + TEST(NtUserSystemParametersInfo(SPI_SETMINIMIZEDMETRICS, 0, (PVOID)&origMetrics, 0) == 1); } void Test_SPI_SETICONMETRICS(void) { ICONMETRICSW metrics; + ICONMETRICSW origMetrics; metrics.cbSize = sizeof(ICONMETRICSW); TEST(NtUserSystemParametersInfo(SPI_GETICONMETRICS, sizeof(ICONMETRICSW), (PVOID)&metrics, 0) == 1); TEST(NtUserSystemParametersInfo(SPI_GETICONMETRICS, sizeof(ICONMETRICSW), (PVOID)0xdeadbeef, 0) == 0); + + origMetrics = metrics; + + metrics.cbSize = sizeof(ICONMETRICSW) + 10; + TEST(NtUserSystemParametersInfo(SPI_SETICONMETRICS, 0, (PVOID)&metrics, 0) == 1); + TEST(NtUserSystemParametersInfo(SPI_GETICONMETRICS, 0, (PVOID)&metrics, 0) == 1); + ok(metrics.cbSize == sizeof(ICONMETRICSW), "Expected size: %lu, got %lu\n", (ULONG)sizeof(ICONMETRICSW), (ULONG)metrics.cbSize); + + TEST(NtUserSystemParametersInfo(SPI_SETICONMETRICS, 0, (PVOID)&origMetrics, 0) == 1); } void

1 0

[dchapyshev] 72646: [NtUser] - Initialize cbSize for animationinfo - Use widechar versions of structures
by dchapyshev＠svn.reactos.org 10 Sep '16

10 Sep '16

Author: dchapyshev Date: Sat Sep 10 17:58:32 2016 New Revision: 72646 URL: http://svn.reactos.org/svn/reactos?rev=72646&view=rev Log: [NtUser] - Initialize cbSize for animationinfo - Use widechar versions of structures Modified: trunk/reactos/win32ss/user/ntuser/sysparams.c trunk/reactos/win32ss/user/ntuser/sysparams.h Modified: trunk/reactos/win32ss/user/ntuser/sysparams.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/syspar… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/sysparams.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/sysparams.c [iso-8859-1] Sat Sep 10 17:58:32 2016 @@ -312,6 +312,7 @@ gspv.serialkeys.cbSize = sizeof(SERIALKEYS); gspv.soundsentry.cbSize = sizeof(SOUNDSENTRYW); gspv.highcontrast.cbSize = sizeof(HIGHCONTRASTW); + gspv.animationinfo.cbSize = sizeof(ANIMATIONINFO); /* Make sure we don't use broken values */ SpiFixupValues(); Modified: trunk/reactos/win32ss/user/ntuser/sysparams.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/syspar… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/sysparams.h [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/sysparams.h [iso-8859-1] Sat Sep 10 17:58:32 2016 @@ -52,7 +52,7 @@ /* Accessability */ ACCESSTIMEOUT accesstimeout; - HIGHCONTRAST highcontrast; + HIGHCONTRASTW highcontrast; BOOL bScreenReader; #if(WINVER >= 0x0600) AUDIODESCRIPTION audiodescription; @@ -63,7 +63,7 @@ #endif /* Sound */ - SOUNDSENTRY soundsentry; + SOUNDSENTRYW soundsentry; BOOL bShowSounds; BOOL bBeep; @@ -175,11 +175,11 @@ STICKYKEYS stickykeys; ACCESSTIMEOUT accesstimeout; SERIALKEYS serialkeys; - SOUNDSENTRY soundsentry; + SOUNDSENTRYW soundsentry; NONCLIENTMETRICSW ncmetrics; MINIMIZEDMETRICS mmmetrics; - ICONMETRICS iconmetrics; - HIGHCONTRAST highcontrast; + ICONMETRICSW iconmetrics; + HIGHCONTRASTW highcontrast; ANIMATIONINFO animationinfo; #if(WINVER >= 0x0600) AUDIODESCRIPTION audiodescription;

1 0

[dchapyshev] 72645: [WIN32KNT_APITEST] - Remove wrong tests. Win2k3 ignores the size of structure for GET/SET NONCLIENTMETRICS, MINIMIZEDMETRICS, ICONMETRICS [NtUser] - Remove wrong checks
by dchapyshev＠svn.reactos.org 10 Sep '16

10 Sep '16

Author: dchapyshev Date: Sat Sep 10 16:17:35 2016 New Revision: 72645 URL: http://svn.reactos.org/svn/reactos?rev=72645&view=rev Log: [WIN32KNT_APITEST] - Remove wrong tests. Win2k3 ignores the size of structure for GET/SET NONCLIENTMETRICS, MINIMIZEDMETRICS, ICONMETRICS [NtUser] - Remove wrong checks Modified: trunk/reactos/win32ss/user/ntuser/sysparams.c trunk/rostests/apitests/win32nt/ntuser/NtUserSystemParametersInfo.c Modified: trunk/reactos/win32ss/user/ntuser/sysparams.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/ntuser/syspar… ============================================================================== --- trunk/reactos/win32ss/user/ntuser/sysparams.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/ntuser/sysparams.c [iso-8859-1] Sat Sep 10 16:17:35 2016 @@ -906,27 +906,11 @@ case SPI_GETNONCLIENTMETRICS: { - LPNONCLIENTMETRICSW metrics = (LPNONCLIENTMETRICSW)pvParam; - - if (uiParam != 0 && uiParam != sizeof(NONCLIENTMETRICSW)) - return 0; - - if (!metrics || metrics->cbSize != sizeof(NONCLIENTMETRICSW)) - return 0; - return SpiGet(pvParam, &gspv.ncm, sizeof(NONCLIENTMETRICSW), fl); } case SPI_SETNONCLIENTMETRICS: { - LPNONCLIENTMETRICSW metrics = (LPNONCLIENTMETRICSW)pvParam; - - if (uiParam != 0 && uiParam != sizeof(NONCLIENTMETRICSW)) - return 0; - - if (!metrics || metrics->cbSize != sizeof(NONCLIENTMETRICSW)) - return 0; - if (!SpiSet(&gspv.ncm, pvParam, sizeof(NONCLIENTMETRICSW), fl)) return 0; @@ -959,27 +943,11 @@ case SPI_GETMINIMIZEDMETRICS: { - LPMINIMIZEDMETRICS metrics = (LPMINIMIZEDMETRICS)pvParam; - - if (uiParam != 0 && uiParam != sizeof(MINIMIZEDMETRICS)) - return 0; - - if (!metrics || metrics->cbSize != sizeof(MINIMIZEDMETRICS)) - return 0; - return SpiGet(pvParam, &gspv.mm, sizeof(MINIMIZEDMETRICS), fl); } case SPI_SETMINIMIZEDMETRICS: { - LPMINIMIZEDMETRICS metrics = (LPMINIMIZEDMETRICS)pvParam; - - if (uiParam != 0 && uiParam != sizeof(MINIMIZEDMETRICS)) - return 0; - - if (!metrics || metrics->cbSize != sizeof(MINIMIZEDMETRICS)) - return 0; - if (!SpiSet(&gspv.mm, pvParam, sizeof(MINIMIZEDMETRICS), fl)) return 0; @@ -1001,27 +969,11 @@ case SPI_GETICONMETRICS: { - LPICONMETRICSW IconMetrics = (LPICONMETRICSW)pvParam; - - if (uiParam != 0 && uiParam != sizeof(ICONMETRICSW)) - return 0; - - if (!IconMetrics || IconMetrics->cbSize != sizeof(ICONMETRICSW)) - return 0; - return SpiGet(pvParam, &gspv.im, sizeof(ICONMETRICSW), fl); } case SPI_SETICONMETRICS: { - LPICONMETRICSW IconMetrics = (LPICONMETRICSW)pvParam; - - if (uiParam != 0 && uiParam != sizeof(ICONMETRICSW)) - return 0; - - if (!IconMetrics || IconMetrics->cbSize != sizeof(ICONMETRICSW)) - return 0; - if (!SpiSet(&gspv.im, pvParam, sizeof(ICONMETRICSW), fl)) return 0; Modified: trunk/rostests/apitests/win32nt/ntuser/NtUserSystemParametersInfo.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/win32nt/ntuser/N… ============================================================================== --- trunk/rostests/apitests/win32nt/ntuser/NtUserSystemParametersInfo.c [iso-8859-1] (original) +++ trunk/rostests/apitests/win32nt/ntuser/NtUserSystemParametersInfo.c [iso-8859-1] Sat Sep 10 16:17:35 2016 @@ -743,7 +743,6 @@ metrics.cbSize = sizeof(NONCLIENTMETRICSW); TEST(NtUserSystemParametersInfo(SPI_GETNONCLIENTMETRICS, sizeof(NONCLIENTMETRICSW), &metrics, 0) == 1); - TEST(NtUserSystemParametersInfo(SPI_GETNONCLIENTMETRICS, sizeof(NONCLIENTMETRICSW) + 1, &metrics, 0) == 0); TEST(NtUserSystemParametersInfo(SPI_GETNONCLIENTMETRICS, sizeof(NONCLIENTMETRICSW), (PVOID)0xdeadbeef, 0) == 0); } @@ -754,7 +753,6 @@ metrics.cbSize = sizeof(MINIMIZEDMETRICS); TEST(NtUserSystemParametersInfo(SPI_GETMINIMIZEDMETRICS, sizeof(MINIMIZEDMETRICS), (PVOID)&metrics, 0) == 1); - TEST(NtUserSystemParametersInfo(SPI_GETMINIMIZEDMETRICS, sizeof(MINIMIZEDMETRICS) + 1, (PVOID)&metrics, 0) == 0); TEST(NtUserSystemParametersInfo(SPI_GETMINIMIZEDMETRICS, sizeof(MINIMIZEDMETRICS), (PVOID)0xdeadbeef, 0) == 0); } @@ -765,7 +763,6 @@ metrics.cbSize = sizeof(ICONMETRICSW); TEST(NtUserSystemParametersInfo(SPI_GETICONMETRICS, sizeof(ICONMETRICSW), (PVOID)&metrics, 0) == 1); - TEST(NtUserSystemParametersInfo(SPI_GETICONMETRICS, sizeof(ICONMETRICSW) + 1, (PVOID)&metrics, 0) == 0); TEST(NtUserSystemParametersInfo(SPI_GETICONMETRICS, sizeof(ICONMETRICSW), (PVOID)0xdeadbeef, 0) == 0); }

1 0

[ekohl] 72644: [NTOSKRNL] Fix lock/unlock order in SeLockSubjectContext/SeUnlockSubjectContext. Patch by Volodymyr Shcherbyna. CORE-11966 #resolve #comment Thanks a lot!
by ekohl＠svn.reactos.org 10 Sep '16

10 Sep '16

Author: ekohl Date: Sat Sep 10 13:54:16 2016 New Revision: 72644 URL: http://svn.reactos.org/svn/reactos?rev=72644&view=rev Log: [NTOSKRNL] Fix lock/unlock order in SeLockSubjectContext/SeUnlockSubjectContext. Patch by Volodymyr Shcherbyna. CORE-11966 #resolve #comment Thanks a lot! Modified: trunk/reactos/ntoskrnl/se/access.c Modified: trunk/reactos/ntoskrnl/se/access.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/se/access.c?rev=7… ============================================================================== --- trunk/reactos/ntoskrnl/se/access.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/se/access.c [iso-8859-1] Sat Sep 10 13:54:16 2016 @@ -338,12 +338,14 @@ PrimaryToken = SubjectContext->PrimaryToken; ClientToken = SubjectContext->ClientToken; + /* Unlock the impersonation one if it's there */ + if (ClientToken) + { + SepReleaseTokenLock(ClientToken); + } + /* Always unlock the primary one */ SepReleaseTokenLock(PrimaryToken); - - /* Unlock the impersonation one if it's there */ - if (!ClientToken) return; - SepReleaseTokenLock(ClientToken); } /*

1 0

[cwittich] 72643: [CRT] sync ischira.c to wine 1.9.16
by cwittich＠svn.reactos.org 10 Sep '16

10 Sep '16

Author: cwittich Date: Sat Sep 10 11:40:54 2016 New Revision: 72643 URL: http://svn.reactos.org/svn/reactos?rev=72643&view=rev Log: [CRT] sync ischira.c to wine 1.9.16 Modified: trunk/reactos/media/doc/README.WINE trunk/reactos/sdk/lib/crt/mbstring/ischira.c Modified: trunk/reactos/media/doc/README.WINE URL: http://svn.reactos.org/svn/reactos/trunk/reactos/media/doc/README.WINE?rev=… ============================================================================== --- trunk/reactos/media/doc/README.WINE [iso-8859-1] (original) +++ trunk/reactos/media/doc/README.WINE [iso-8859-1] Sat Sep 10 11:40:54 2016 @@ -299,8 +299,9 @@ reactos/sdk/lib/crt/string/strtoul.c # Synced to WineStaging-1.9.9 reactos/sdk/lib/crt/string/strxfrm.c # Synced to Wine-1.9.16 reactos/sdk/lib/crt/string/wcs.c # Synced at 20080611 - reactos/sdk/lib/crt/string/wctype.c # Synced at WineStaging-1.9.16 - reactos/sdk/lib/crt/string/wtoi64.c # Synced at Wine-1.9.16 + reactos/sdk/lib/crt/string/wctype.c # Synced to WineStaging-1.9.16 + reactos/sdk/lib/crt/string/wtoi64.c # Synced to Wine-1.9.16 + reactos/sdk/lib/crt/mbstring/ischira.c # Synced to Wine-1.9.16 reactos/sdk/lib/crt/wine/heap.c # Synced at 20080529 reactos/sdk/lib/crt/wine/undname.c # Synced to WineStaging-1.9.16 reactos/sdk/lib/crt/process/thread.c # Synced to WineStaging-1.7.55 Modified: trunk/reactos/sdk/lib/crt/mbstring/ischira.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/sdk/lib/crt/mbstring/ischi… ============================================================================== --- trunk/reactos/sdk/lib/crt/mbstring/ischira.c [iso-8859-1] (original) +++ trunk/reactos/sdk/lib/crt/mbstring/ischira.c [iso-8859-1] Sat Sep 10 11:40:54 2016 @@ -1,58 +1,54 @@ -/* - * COPYRIGHT: See COPYING in the top level directory - * PROJECT: ReactOS system libraries - * FILE: lib/sdk/crt/mbstring/ischira.c - * PURPOSE: - * PROGRAMER: - * UPDATE HISTORY: - * 12/04/99: Ariadne Created - * 05/30/08: Samuel Serapion adapted from PROJECT C Library - * - */ - +#include <precomp.h> #include <mbctype.h> - -/* - * @implemented +/********************************************************************* + * _ismbchira(MSVCRT.@) */ -int _ismbchira( unsigned int c ) +int CDECL _ismbchira(unsigned int c) { - return ((c>=0x829F) && (c<=0x82F1)); + if(get_mbcinfo()->mbcodepage == 932) + { + /* Japanese/Hiragana, CP 932 */ + return (c >= 0x829f && c <= 0x82f1); + } + return 0; } -/* - * @implemented +/********************************************************************* + * _ismbckata(MSVCRT.@) */ -int _ismbckata( unsigned int c ) +int CDECL _ismbckata(unsigned int c) { - return ((c>=0x8340) && (c<=0x8396)); + if(get_mbcinfo()->mbcodepage == 932) + { + /* Japanese/Katakana, CP 932 */ + return (c >= 0x8340 && c <= 0x8396 && c != 0x837f); + } + return 0; } -/* - * @implemented +/********************************************************************* + * _mbctohira (MSVCRT.@) + * + * Converts a sjis katakana character to hiragana. */ -unsigned int _mbctohira( unsigned int c ) +unsigned int CDECL _mbctohira(unsigned int c) { - if (c >= 0x8340 && c <= 0x837e) - return c - 0xa1; - else if (c >= 0x8380 && c <= 0x8396) - return c - 0xa2; - else - return c; + if(_ismbckata(c) && c <= 0x8393) + return (c - 0x8340 - (c >= 0x837f ? 1 : 0)) + 0x829f; + return c; } -/* - * @implemented +/********************************************************************* + * _mbctokata (MSVCRT.@) + * + * Converts a sjis hiragana character to katakana. */ -unsigned int _mbctokata( unsigned int c ) +unsigned int CDECL _mbctokata(unsigned int c) { - if (c >= 0x829f && c <= 0x82dd) - return c + 0xa1; - else if (c >= 0x82de && c <= 0x82f1) - return c + 0xa2; - else - return c; + if(_ismbchira(c)) + return (c - 0x829f) + 0x8340 + (c >= 0x82de ? 1 : 0); + return c; }

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