Ros-diffs

ros-diffs@reactos.org

81390 discussions

[reactos] 01/01: [NTUSER] Refactoring ime.c
by Katayama Hirofumi MZ 09 Feb '22

09 Feb '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=e52ce89be339efc89fed3… commit e52ce89be339efc89fed3bc0ea09b4f73022762f Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Wed Feb 9 11:51:27 2022 +0900 Commit: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> CommitDate: Wed Feb 9 11:51:27 2022 +0900 [NTUSER] Refactoring ime.c s/WINAPI/NTAPI/ CORE-11700 --- win32ss/user/ntuser/ime.c | 50 +++++++++++++++++++++++------------------------ 1 file changed, 25 insertions(+), 25 deletions(-) diff --git a/win32ss/user/ntuser/ime.c b/win32ss/user/ntuser/ime.c index 25adc48265d..f6b0cb18bb2 100644 --- a/win32ss/user/ntuser/ime.c +++ b/win32ss/user/ntuser/ime.c @@ -313,7 +313,7 @@ HIMC FASTCALL IntAssociateInputContext(PWND pWnd, PIMC pImc) } DWORD -APIENTRY +NTAPI NtUserSetThreadLayoutHandles(HKL hNewKL, HKL hOldKL) { PTHREADINFO pti; @@ -399,7 +399,7 @@ IntImmProcessKey(PUSER_MESSAGE_QUEUE MessageQueue, PWND pWnd, UINT Msg, WPARAM w } NTSTATUS -APIENTRY +NTAPI NtUserBuildHimcList(DWORD dwThreadId, DWORD dwCount, HIMC *phList, LPDWORD pdwCount) { NTSTATUS ret = STATUS_UNSUCCESSFUL; @@ -509,7 +509,7 @@ static VOID FASTCALL UserSetImeConversionKeyState(PTHREADINFO pti, DWORD dwConve } DWORD -APIENTRY +NTAPI NtUserNotifyIMEStatus(HWND hwnd, BOOL fOpen, DWORD dwConversion) { PWND pwnd; @@ -552,7 +552,7 @@ Quit: } DWORD -APIENTRY +NTAPI NtUserCheckImeHotKey( DWORD VirtualKey, LPARAM lParam) @@ -562,7 +562,7 @@ NtUserCheckImeHotKey( } BOOL -APIENTRY +NTAPI NtUserDisableThreadIme( DWORD dwThreadID) { @@ -630,7 +630,7 @@ Quit: } DWORD -APIENTRY +NTAPI NtUserGetAppImeLevel(HWND hWnd) { DWORD ret = 0; @@ -708,7 +708,7 @@ BOOL FASTCALL UserGetImeInfoEx(LPVOID pUnknown, PIMEINFOEX pInfoEx, IMEINFOEXCLA } BOOL -APIENTRY +NTAPI NtUserGetImeInfoEx( PIMEINFOEX pImeInfoEx, IMEINFOEXCLASS SearchType) @@ -733,18 +733,18 @@ NtUserGetImeInfoEx( _SEH2_END; ret = UserGetImeInfoEx(NULL, &ImeInfoEx, SearchType); - if (ret) + if (!ret) + goto Quit; + + _SEH2_TRY { - _SEH2_TRY - { - *pImeInfoEx = ImeInfoEx; - } - _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) - { - ret = FALSE; - } - _SEH2_END; + *pImeInfoEx = ImeInfoEx; + } + _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) + { + ret = FALSE; } + _SEH2_END; Quit: UserLeave(); @@ -752,7 +752,7 @@ Quit: } BOOL -APIENTRY +NTAPI NtUserSetAppImeLevel(HWND hWnd, DWORD dwLevel) { BOOL ret = FALSE; @@ -807,7 +807,7 @@ BOOL FASTCALL UserSetImeInfoEx(LPVOID pUnknown, PIMEINFOEX pImeInfoEx) } BOOL -APIENTRY +NTAPI NtUserSetImeInfoEx(PIMEINFOEX pImeInfoEx) { BOOL ret = FALSE; @@ -836,7 +836,7 @@ Quit: return ret; } -BOOL APIENTRY +BOOL NTAPI NtUserSetImeOwnerWindow(HWND hImeWnd, HWND hwndFocus) { BOOL ret = FALSE; @@ -960,7 +960,7 @@ BOOLEAN UserDestroyInputContext(PVOID Object) return TRUE; } -BOOL APIENTRY NtUserDestroyInputContext(HIMC hIMC) +BOOL NTAPI NtUserDestroyInputContext(HIMC hIMC) { PIMC pIMC; BOOL ret = FALSE; @@ -1053,7 +1053,7 @@ PIMC FASTCALL UserCreateInputContext(ULONG_PTR dwClientImcData) } HIMC -APIENTRY +NTAPI NtUserCreateInputContext(ULONG_PTR dwClientImcData) { PIMC pIMC; @@ -1141,7 +1141,7 @@ DWORD FASTCALL IntAssociateInputContextEx(PWND pWnd, PIMC pIMC, DWORD dwFlags) } DWORD -APIENTRY +NTAPI NtUserAssociateInputContext(HWND hWnd, HIMC hIMC, DWORD dwFlags) { DWORD ret = 2; @@ -1194,7 +1194,7 @@ BOOL FASTCALL UserUpdateInputContext(PIMC pIMC, DWORD dwType, DWORD_PTR dwValue) } BOOL -APIENTRY +NTAPI NtUserUpdateInputContext( HIMC hIMC, DWORD dwType, @@ -1220,7 +1220,7 @@ Quit: } DWORD_PTR -APIENTRY +NTAPI NtUserQueryInputContext(HIMC hIMC, DWORD dwType) { PIMC pIMC;

1 0

[reactos] 01/01: [NTUSER] Implement NtUserSetImeHotKey (#4350)
by Katayama Hirofumi MZ 09 Feb '22

09 Feb '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=5d5cc578697966c5171fb… commit 5d5cc578697966c5171fb81ad2b59fef3744fb22 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Wed Feb 9 11:27:44 2022 +0900 Commit: GitHub <noreply(a)github.com> CommitDate: Wed Feb 9 11:27:44 2022 +0900 [NTUSER] Implement NtUserSetImeHotKey (#4350) - Modify NtUserGetImeHotKey and NtUserSetImeHotKey prototypes. - Define enum SETIMEHOTKEY_ACTION in <undocuser.h>. - Define IMEHOTKEY structure in ime.c. - Add IntGetImeHotKeyLangId, IntAddImeHotKey, IntGetImeHotKeyById, IntGetImeHotKeyByKeyAndLang, IntDeleteImeHotKey, IntFreeImeHotKeys, and IntSetImeHotKey helper functions. - Implement NtUserGetImeHotKey and NtUserSetImeHotKey functions. - Cleanup the IME hotkeys at process exit. CORE-11700 --- sdk/include/reactos/undocuser.h | 8 ++ win32ss/include/ntuser.h | 23 ++-- win32ss/user/ntuser/ime.c | 282 ++++++++++++++++++++++++++++++++++++---- win32ss/user/ntuser/input.h | 1 + win32ss/user/ntuser/main.c | 2 + 5 files changed, 280 insertions(+), 36 deletions(-) diff --git a/sdk/include/reactos/undocuser.h b/sdk/include/reactos/undocuser.h index 3c0ebb3c149..f396c58a7ba 100644 --- a/sdk/include/reactos/undocuser.h +++ b/sdk/include/reactos/undocuser.h @@ -394,6 +394,14 @@ typedef enum _QUERY_INPUT_CONTEXT QIC_DEFAULTIMC } QUERY_INPUT_CONTEXT; +/* NtUserSetImeHotKey actions */ +typedef enum tagSETIMEHOTKEY_ACTION +{ + SETIMEHOTKEY_DELETE = 1, + SETIMEHOTKEY_ADD, + SETIMEHOTKEY_DELETEALL +} SETIMEHOTKEY_ACTION; + #ifdef __cplusplus } /* extern "C" */ #endif /* defined(__cplusplus) */ diff --git a/win32ss/include/ntuser.h b/win32ss/include/ntuser.h index 6f897a24736..f792c08a2c0 100644 --- a/win32ss/include/ntuser.h +++ b/win32ss/include/ntuser.h @@ -2388,11 +2388,12 @@ NtUserGetIconSize( LONG *plcx, LONG *plcy); -BOOL NTAPI -NtUserGetImeHotKey(IN DWORD dwHotKey, - OUT LPUINT lpuModifiers, - OUT LPUINT lpuVKey, - OUT LPHKL lphKL); +BOOL +NTAPI +NtUserGetImeHotKey(DWORD dwHotKeyId, + LPUINT lpuModifiers, + LPUINT lpuVirtualKey, + LPHKL lphKL); BOOL NTAPI @@ -3184,14 +3185,14 @@ NTAPI NtUserSetFocus( HWND hWnd); -DWORD +BOOL NTAPI NtUserSetImeHotKey( - DWORD Unknown0, - DWORD Unknown1, - DWORD Unknown2, - DWORD Unknown3, - DWORD Unknown4); + DWORD dwHotKeyId, + UINT uModifiers, + UINT uVirtualKey, + HKL hKL, + DWORD dwAction); BOOL NTAPI diff --git a/win32ss/user/ntuser/ime.c b/win32ss/user/ntuser/ime.c index 5c67c673afe..25adc48265d 100644 --- a/win32ss/user/ntuser/ime.c +++ b/win32ss/user/ntuser/ime.c @@ -11,6 +11,14 @@ DBG_DEFAULT_CHANNEL(UserMisc); #define INVALID_THREAD_ID ((ULONG)-1) +#define MOD_KEYS (MOD_CONTROL | MOD_SHIFT | MOD_ALT | MOD_WIN) +#define MOD_LEFT_RIGHT (MOD_LEFT | MOD_RIGHT) + +#define LANGID_CHINESE_SIMPLIFIED MAKELANGID(LANG_CHINESE, SUBLANG_CHINESE_SIMPLIFIED) +#define LANGID_JAPANESE MAKELANGID(LANG_JAPANESE, SUBLANG_DEFAULT) +#define LANGID_KOREAN MAKELANGID(LANG_KOREAN, SUBLANG_KOREAN) +#define LANGID_CHINESE_TRADITIONAL MAKELANGID(LANG_CHINESE, SUBLANG_CHINESE_TRADITIONAL) +#define LANGID_NEUTRAL MAKELANGID(LANG_NEUTRAL, SUBLANG_NEUTRAL) #define IS_WND_IMELIKE(pwnd) \ (((pwnd)->pcls->style & CS_IME) || \ @@ -37,6 +45,255 @@ HIMC ghIMC = NULL; BOOL gfImeOpen = (BOOL)-1; DWORD gdwImeConversion = (DWORD)-1; +typedef struct tagIMEHOTKEY +{ + struct tagIMEHOTKEY *pNext; + DWORD dwHotKeyId; + UINT uVirtualKey; + UINT uModifiers; + HKL hKL; +} IMEHOTKEY, *PIMEHOTKEY; + +PIMEHOTKEY gpImeHotKeyList = NULL; + +static LANGID FASTCALL IntGetImeHotKeyLangId(DWORD dwHotKeyId) +{ +#define IME_CHOTKEY 0x10 +#define IME_JHOTKEY 0x30 +#define IME_KHOTKEY 0x50 +#define IME_THOTKEY 0x70 +#define IME_XHOTKEY 0x90 + static const LANGID s_array[] = + { + /* 0x00 */ (WORD)-1, + /* 0x10 */ LANGID_CHINESE_SIMPLIFIED, + /* 0x20 */ LANGID_CHINESE_SIMPLIFIED, + /* 0x30 */ LANGID_JAPANESE, + /* 0x40 */ LANGID_JAPANESE, + /* 0x50 */ LANGID_KOREAN, + /* 0x60 */ LANGID_KOREAN, + /* 0x70 */ LANGID_CHINESE_TRADITIONAL, + /* 0x80 */ LANGID_CHINESE_TRADITIONAL + }; + + if (IME_CHOTKEY <= dwHotKeyId && dwHotKeyId < IME_XHOTKEY) + return s_array[(dwHotKeyId & 0xF0) >> 4]; + return LANGID_NEUTRAL; +} + +static VOID FASTCALL IntAddImeHotKey(PIMEHOTKEY *ppList, PIMEHOTKEY pHotKey) +{ + PIMEHOTKEY pNode; + + if (!*ppList) + { + *ppList = pHotKey; + return; + } + + for (pNode = *ppList; pNode; pNode = pNode->pNext) + { + if (!pNode->pNext) + { + pNode->pNext = pHotKey; + return; + } + } +} + +static PIMEHOTKEY FASTCALL IntGetImeHotKeyById(PIMEHOTKEY pList, DWORD dwHotKeyId) +{ + PIMEHOTKEY pNode; + for (pNode = pList; pNode; pNode = pNode->pNext) + { + if (pNode->dwHotKeyId == dwHotKeyId) + return pNode; + } + return NULL; +} + +static PIMEHOTKEY APIENTRY +IntGetImeHotKeyByKeyAndLang(PIMEHOTKEY pList, UINT uModKeys, UINT uLeftRight, + UINT uVirtualKey, LANGID TargetLangId) +{ + PIMEHOTKEY pNode; + LANGID LangID; + UINT uModifiers; + + for (pNode = pList; pNode; pNode = pNode->pNext) + { + if (pNode->uVirtualKey != uVirtualKey) + continue; + + LangID = IntGetImeHotKeyLangId(pNode->dwHotKeyId); + if (LangID != TargetLangId) + continue; + + uModifiers = pNode->uModifiers; + if (uModifiers & MOD_IGNORE_ALL_MODIFIER) + return pNode; + + if ((uModifiers & MOD_KEYS) != uModKeys) + continue; + + if ((uModifiers & uLeftRight) || (uModifiers & MOD_LEFT_RIGHT) == uLeftRight) + return pNode; + } + + return NULL; +} + +static VOID FASTCALL IntDeleteImeHotKey(PIMEHOTKEY *ppList, PIMEHOTKEY pHotKey) +{ + PIMEHOTKEY pNode; + + if (*ppList == pHotKey) + { + *ppList = pHotKey->pNext; + ExFreePoolWithTag(pHotKey, USERTAG_IMEHOTKEY); + return; + } + + for (pNode = *ppList; pNode; pNode = pNode->pNext) + { + if (pNode->pNext == pHotKey) + { + pNode->pNext = pHotKey->pNext; + ExFreePoolWithTag(pHotKey, USERTAG_IMEHOTKEY); + return; + } + } +} + +VOID FASTCALL IntFreeImeHotKeys(VOID) +{ + PIMEHOTKEY pNode, pNext; + for (pNode = gpImeHotKeyList; pNode; pNode = pNext) + { + pNext = pNode->pNext; + ExFreePoolWithTag(pNode, USERTAG_IMEHOTKEY); + } + gpImeHotKeyList = NULL; +} + +static BOOL APIENTRY +IntSetImeHotKey(DWORD dwHotKeyId, UINT uModifiers, UINT uVirtualKey, HKL hKL, DWORD dwAction) +{ + PIMEHOTKEY pNode; + LANGID LangId; + + switch (dwAction) + { + case SETIMEHOTKEY_DELETE: + pNode = IntGetImeHotKeyById(gpImeHotKeyList, dwHotKeyId); + if (!pNode) + return FALSE; + + IntDeleteImeHotKey(&gpImeHotKeyList, pNode); + return TRUE; + + case SETIMEHOTKEY_ADD: + if (uVirtualKey == VK_PACKET) + return FALSE; + + LangId = IntGetImeHotKeyLangId(dwHotKeyId); + if (LangId == LANGID_KOREAN) + return FALSE; + + pNode = IntGetImeHotKeyByKeyAndLang(gpImeHotKeyList, + (uModifiers & MOD_KEYS), + (uModifiers & MOD_LEFT_RIGHT), + uVirtualKey, LangId); + if (!pNode) + pNode = IntGetImeHotKeyById(gpImeHotKeyList, dwHotKeyId); + + if (pNode) + { + pNode->uModifiers = uModifiers; + pNode->uVirtualKey = uVirtualKey; + pNode->hKL = hKL; + return TRUE; + } + + pNode = ExAllocatePoolWithTag(PagedPool, sizeof(IMEHOTKEY), USERTAG_IMEHOTKEY); + if (!pNode) + return FALSE; + + pNode->pNext = NULL; + pNode->dwHotKeyId = dwHotKeyId; + pNode->uModifiers = uModifiers; + pNode->uVirtualKey = uVirtualKey; + pNode->hKL = hKL; + IntAddImeHotKey(&gpImeHotKeyList, pNode); + return TRUE; + + case SETIMEHOTKEY_DELETEALL: + IntFreeImeHotKeys(); + return TRUE; + + default: + return FALSE; + } +} + +BOOL NTAPI +NtUserGetImeHotKey(DWORD dwHotKeyId, LPUINT lpuModifiers, LPUINT lpuVirtualKey, LPHKL lphKL) +{ + PIMEHOTKEY pNode = NULL; + + UserEnterExclusive(); + + _SEH2_TRY + { + ProbeForWrite(lpuModifiers, sizeof(UINT), 1); + ProbeForWrite(lpuVirtualKey, sizeof(UINT), 1); + if (lphKL) + ProbeForWrite(lphKL, sizeof(HKL), 1); + } + _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) + { + goto Quit; + } + _SEH2_END; + + pNode = IntGetImeHotKeyById(gpImeHotKeyList, dwHotKeyId); + if (!pNode) + goto Quit; + + _SEH2_TRY + { + *lpuModifiers = pNode->uModifiers; + *lpuVirtualKey = pNode->uVirtualKey; + if (lphKL) + *lphKL = pNode->hKL; + } + _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) + { + pNode = NULL; + } + _SEH2_END; + +Quit: + UserLeave(); + return !!pNode; +} + +BOOL +NTAPI +NtUserSetImeHotKey( + DWORD dwHotKeyId, + UINT uModifiers, + UINT uVirtualKey, + HKL hKL, + DWORD dwAction) +{ + BOOL ret; + UserEnterExclusive(); + ret = IntSetImeHotKey(dwHotKeyId, uModifiers, uVirtualKey, hKL, dwAction); + UserLeave(); + return ret; +} + PWND FASTCALL IntGetTopLevelWindow(PWND pwnd) { if (!pwnd) @@ -194,17 +451,6 @@ Quit: return ret; } -BOOL WINAPI -NtUserGetImeHotKey(IN DWORD dwHotKey, - OUT LPUINT lpuModifiers, - OUT LPUINT lpuVKey, - OUT LPHKL lphKL) -{ - STUB - - return FALSE; -} - static VOID FASTCALL UserSetImeConversionKeyState(PTHREADINFO pti, DWORD dwConversion) { HKL hKL; @@ -305,20 +551,6 @@ Quit: return 0; } -DWORD -APIENTRY -NtUserSetImeHotKey( - DWORD Unknown0, - DWORD Unknown1, - DWORD Unknown2, - DWORD Unknown3, - DWORD Unknown4) -{ - STUB - - return 0; -} - DWORD APIENTRY NtUserCheckImeHotKey( diff --git a/win32ss/user/ntuser/input.h b/win32ss/user/ntuser/input.h index bd185a12c04..3cae36048f1 100644 --- a/win32ss/user/ntuser/input.h +++ b/win32ss/user/ntuser/input.h @@ -85,6 +85,7 @@ BOOL NTAPI UserSendMouseInput(MOUSEINPUT *pMouseInput, BOOL bInjected); /* IMM */ UINT FASTCALL IntImmProcessKey(PUSER_MESSAGE_QUEUE, PWND, UINT, WPARAM, LPARAM); +VOID FASTCALL IntFreeImeHotKeys(VOID); extern DWORD gSystemFS; extern UINT gSystemCPCharSet; diff --git a/win32ss/user/ntuser/main.c b/win32ss/user/ntuser/main.c index 62e1f157271..dfdf2d64c0b 100644 --- a/win32ss/user/ntuser/main.c +++ b/win32ss/user/ntuser/main.c @@ -180,6 +180,8 @@ UserProcessDestroy(PEPROCESS Process) if (ppiScrnSaver == ppiCurrent) ppiScrnSaver = NULL; + IntFreeImeHotKeys(); + if (gpwlCache) { ExFreePoolWithTag(gpwlCache, USERTAG_WINDOWLIST);

1 0

[reactos] 01/01: [NOTEPAD] Add Croatian translation (hr-HR) (#4338)
by Andrej Bartulin 09 Feb '22

09 Feb '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=e0fc48d6e29c654889661… commit e0fc48d6e29c65488966116dd9b5eb10ed4e504a Author: Andrej Bartulin <andrej.bartulinvr(a)gmail.com> AuthorDate: Wed Feb 9 02:26:22 2022 +0100 Commit: GitHub <noreply(a)github.com> CommitDate: Wed Feb 9 10:26:22 2022 +0900 [NOTEPAD] Add Croatian translation (hr-HR) (#4338) --- base/applications/notepad/lang/hr-Hr.rc | 172 ++++++++++++++++++++++++++++++++ base/applications/notepad/rsrc.rc | 3 + 2 files changed, 175 insertions(+) diff --git a/base/applications/notepad/lang/hr-Hr.rc b/base/applications/notepad/lang/hr-Hr.rc new file mode 100644 index 00000000000..97bc5abda3a --- /dev/null +++ b/base/applications/notepad/lang/hr-Hr.rc @@ -0,0 +1,172 @@ +LANGUAGE LANG_CROATIAN, SUBLANG_DEFAULT + + +ID_ACCEL ACCELERATORS +BEGIN + "^A", CMD_SELECT_ALL + "^C", CMD_COPY + "^F", CMD_SEARCH + "^G", CMD_GOTO + "H", CMD_REPLACE, VIRTKEY, CONTROL + "^N", CMD_NEW + "N", CMD_NEW_WINDOW, VIRTKEY,CONTROL , SHIFT + "^O", CMD_OPEN + "^P", CMD_PRINT + "^S", CMD_SAVE + "^V", CMD_PASTE + "^X", CMD_CUT + "^Z", CMD_UNDO + VK_F3, CMD_SEARCH_NEXT, VIRTKEY + VK_F5, CMD_TIME_DATE, VIRTKEY +END + +MAIN_MENU MENU +BEGIN + POPUP "&Datoteka" + BEGIN + MENUITEM "&Novo\tCtrl+N", CMD_NEW + MENUITEM "Novi &prozor\tCtrl+Shift+N", CMD_NEW_WINDOW + MENUITEM "&Otvori...\tCtrl+O", CMD_OPEN + MENUITEM "&Spremi\tCtrl+S", CMD_SAVE + MENUITEM "Spremi &kao...", CMD_SAVE_AS + MENUITEM SEPARATOR + MENUITEM "Postavljanje st&ranice...", CMD_PAGE_SETUP + MENUITEM "&Ispiši...\tCtrl+P", CMD_PRINT + MENUITEM SEPARATOR + MENUITEM "I&zađi", CMD_EXIT + END + POPUP "&Uređivanje" + BEGIN + MENUITEM "&Poništi\tCtrl+Z", CMD_UNDO + MENUITEM SEPARATOR + MENUITEM "&Izreži\tCtrl+X", CMD_CUT + MENUITEM "&Kopiraj\tCtrl+C", CMD_COPY + MENUITEM "Z&alijepi\tCtrl+V", CMD_PASTE + MENUITEM "Iz&briši\tDel", CMD_DELETE + MENUITEM SEPARATOR + MENUITEM "&Traži...\tCtrl+F", CMD_SEARCH + MENUITEM "Pronađi s&ljedeće\tF3", CMD_SEARCH_NEXT + MENUITEM "&Zamijeni...\tCtrl+H", CMD_REPLACE + MENUITEM "Idi &na...\tCtrl+G", CMD_GOTO + MENUITEM SEPARATOR + MENUITEM "Odaberi &sve\tCtrl+A", CMD_SELECT_ALL + MENUITEM "Vrijeme/&Datum\tF5", CMD_TIME_DATE + END + POPUP "&Oblikovanje" + BEGIN + MENUITEM "Prelo&mi riječi", CMD_WRAP + MENUITEM "&Font...", CMD_FONT + END + POPUP "Pri&kaz" + BEGIN + MENUITEM "T&raka stanja", CMD_STATUSBAR + END + POPUP "&Pomoć" + BEGIN + MENUITEM "&Prikaži pomoć", CMD_HELP_CONTENTS + MENUITEM SEPARATOR + MENUITEM "&O programu Blok za pisanje", CMD_HELP_ABOUT_NOTEPAD + END +END + +/* Dialog 'Page setup' */ +DIALOG_PAGESETUP DIALOGEX 0, 0, 365, 193 +STYLE DS_SHELLFONT | DS_MODALFRAME | WS_CAPTION | WS_SYSMENU | DS_CONTEXTHELP +FONT 8, "MS Shell Dlg" +CAPTION "Postavljanje stranice" +BEGIN + GROUPBOX "Pretpregled", 0, 240, 6, 120, 153, BS_GROUPBOX + CONTROL "", rct1, "Static", SS_WHITERECT, 260, 42, 80, 80 + CONTROL "", rct2, "Static", SS_GRAYRECT, 340, 46, 4, 80 + CONTROL "", rct3, "Static", SS_GRAYRECT, 264, 122, 80, 4 + GROUPBOX "Papir", grp2, 8, 6, 224, 56, BS_GROUPBOX + LTEXT "Ve&ličina:", stc2, 16, 22, 36, 8 + COMBOBOX cmb2, 64, 20, 160, 160, CBS_SIMPLE | CBS_DROPDOWN | CBS_SORT | WS_GROUP | WS_TABSTOP | WS_VSCROLL + LTEXT "L&adica:", stc3, 16, 42, 36, 8 + COMBOBOX cmb3, 64, 40, 160, 160, CBS_SIMPLE | CBS_DROPDOWN | CBS_SORT | WS_GROUP | WS_TABSTOP | WS_VSCROLL + GROUPBOX "Orijentacija", grp1, 8, 66, 64, 56, BS_GROUPBOX + AUTORADIOBUTTON "P&ortret", rad1, 16, 80, 52, 12, BS_AUTORADIOBUTTON + AUTORADIOBUTTON "P&ejzaž", rad2, 16, 100, 52, 12, BS_AUTORADIOBUTTON + GROUPBOX "Granice", grp4, 80, 66, 152, 56, BS_GROUPBOX + LTEXT "&Lijevo:", stc15, 88, 82, 30, 8 + EDITTEXT edt4, 119, 80, 36, 12, WS_TABSTOP | WS_GROUP | WS_BORDER + LTEXT "&Desno:", stc16, 159, 82, 30, 8 + EDITTEXT edt6, 190, 80, 36, 12, WS_TABSTOP | WS_GROUP | WS_BORDER + LTEXT "&Vrh:", stc17, 88, 102, 30, 8 + EDITTEXT edt5, 119, 100, 36, 12, WS_TABSTOP | WS_GROUP | WS_BORDER + LTEXT "&D&no:", stc18, 159, 102, 30, 8 + EDITTEXT edt7, 190, 100, 36, 12, WS_TABSTOP | WS_GROUP | WS_BORDER + LTEXT "&Zaglavlje:", 0x140, 8, 132, 40, 15 + EDITTEXT 0x141, 58, 130, 173, 12, WS_BORDER | WS_TABSTOP | ES_AUTOHSCROLL + LTEXT "&Podnožje:", 0x142, 8, 149, 40, 15 + EDITTEXT 0x143, 58, 147, 173, 12, WS_BORDER | WS_TABSTOP | ES_AUTOHSCROLL + PUSHBUTTON "Pomoć", IDHELP, 8, 170, 50, 14 + DEFPUSHBUTTON "U redu", IDOK, 198, 170, 50, 14, BS_PUSHBUTTON + PUSHBUTTON "Odustani", IDCANCEL, 254, 170, 50, 14 + PUSHBUTTON "P&isač...", psh3, 310, 170, 50, 14 +END + +/* Dialog 'Encoding' */ +DIALOG_ENCODING DIALOGEX 0, 0, 256, 44 +STYLE DS_SHELLFONT | DS_CONTROL | WS_CHILD | WS_CLIPSIBLINGS | WS_CAPTION | WS_SYSMENU +FONT 8, "MS Shell Dlg" +CAPTION "Kodiranje" +BEGIN + COMBOBOX ID_ENCODING, 124, 0, 125, 80, CBS_DROPDOWNLIST | CBS_SORT | WS_VSCROLL | WS_TABSTOP + LTEXT "Kodiranje:", 0x155, 65, 2, 41, 12 + COMBOBOX ID_EOLN, 124, 18, 125, 80, CBS_DROPDOWNLIST | WS_VSCROLL | WS_TABSTOP + LTEXT "Krajnje linije:", 0x156, 65, 20, 41, 12 +END + +/* Dialog 'Go To' */ +DIALOG_GOTO DIALOGEX 0, 0, 165, 50 +STYLE DS_SHELLFONT | DS_MODALFRAME | WS_CAPTION | WS_SYSMENU +FONT 8, "MS Shell Dlg" +CAPTION "Idi na liniju" +BEGIN + EDITTEXT ID_LINENUMBER, 54, 10, 106, 12, WS_CHILD | WS_BORDER | WS_TABSTOP + LTEXT "Broj linije:", 0x155, 5, 12, 41, 12, WS_CHILD | WS_TABSTOP | ES_NUMBER + DEFPUSHBUTTON "U redu", IDOK, 75, 30, 40, 15, WS_CHILD | WS_TABSTOP + PUSHBUTTON "Odustani", IDCANCEL, 120, 30, 40, 15, WS_CHILD | WS_TABSTOP +END + +STRINGTABLE +BEGIN + STRING_PAGESETUP_HEADERVALUE "&n" /* FIXME */ + STRING_PAGESETUP_FOOTERVALUE "Stranica &s" /* FIXME */ + STRING_PAGESETUP_LEFTVALUE "20 mm" /* FIXME */ + STRING_PAGESETUP_RIGHTVALUE "20 mm" /* FIXME */ + STRING_PAGESETUP_TOPVALUE "25 mm" /* FIXME */ + STRING_PAGESETUP_BOTTOMVALUE "25 mm" /* FIXME */ + STRING_NOTEPAD "Blok za pisanje" + STRING_ERROR "Greška" + STRING_WARNING "Upozorenje" + STRING_INFO "Informacija" + STRING_UNTITLED "Bez naslova" + STRING_ALL_FILES "Sve datoteke (*.*)" + STRING_TEXT_FILES_TXT "Tekstne datoteke (*.txt)" + STRING_TOOLARGE "Datoteka '%s' je prevelik za program Blok za pisanje.\n\ +Molimo Vas da koristite drugu aplikaciju." + STRING_NOTEXT "Niste unijeli tekst. \ +\nMolimo Vas da nešto unesete i ponovo pokušate." + STRING_DOESNOTEXIST "Datoteka '%s'\nne postoji\n\n\ +Želite li stvoriti novu datoteku ?" + STRING_NOTSAVED "Datoteka '%s'\nje bila promijenjena\n\n\ +Želite li spremiti te promjene ?" + STRING_NOTFOUND "'%s' ne može biti pronađena." + STRING_OUT_OF_MEMORY "Nema dovoljno memorije da se dovrši \ +zadatak.\nZatvorite jednu ili više aplikacija da povećate\nslobodnu memoriju." + STRING_CANNOTFIND "Ne mogu pronaći '%s'" + STRING_ANSI "ANSI" + STRING_UNICODE "Unicode" + STRING_UNICODE_BE "Unicode (big endian)" + STRING_UTF8 "UTF-8" + STRING_CRLF "Windows (CR + LF)" + STRING_LF "Unix (LF)" + STRING_CR "Mac (CR)" + STRING_LINE_COLUMN "Linija %d, Stupac %d" + STRING_PRINTERROR "Ne mogu ispisati datoteku '%s'.\n\Provjerite je li pisač uključen i je li pravilno konfiguiran." + + STRING_TEXT_DOCUMENT "Tekstni dokument" + STRING_NOTEPAD_AUTHORS "Copyright 1997, 98 Marcel Baur, 2000 Mike McCormack, 2002 Sylvain Petreolle, 2002 Andriy Palamarchuk\r\n" +END diff --git a/base/applications/notepad/rsrc.rc b/base/applications/notepad/rsrc.rc index bcb9003c35d..7adcac3b944 100644 --- a/base/applications/notepad/rsrc.rc +++ b/base/applications/notepad/rsrc.rc @@ -74,6 +74,9 @@ IDI_NPICON ICON "res/notepad.ico" #ifdef LANGUAGE_HI_IN #include "lang/hi-IN.rc" #endif +#ifdef LANGUAGE_HR_HR + #include "lang/hr-Hr.rc" +#endif #ifdef LANGUAGE_HU_HU #include "lang/hu-HU.rc" #endif

1 0

[reactos] 03/03: [CONCFG:FONT] Implement support for the Windows 10 "__DefaultTTFont__" special FaceName value. (#4337)
by Hermès Bélusca-Maïto 08 Feb '22

08 Feb '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=2ae45e0985ba55a50f86a… commit 2ae45e0985ba55a50f86a88a25067d937fcd3062 Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Sun Jan 30 18:29:00 2022 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Tue Feb 8 15:59:10 2022 +0100 [CONCFG:FONT] Implement support for the Windows 10 "__DefaultTTFont__" special FaceName value. (#4337) This special value tells the console to default to whichever font is deemed most appropriate for the current code page. Add Doxygen documentation for FindCachedTTFont(). --- win32ss/user/winsrv/concfg/font.c | 124 ++++++++++++++++++++++++++++++++------ win32ss/user/winsrv/concfg/font.h | 15 ++++- 2 files changed, 117 insertions(+), 22 deletions(-) diff --git a/win32ss/user/winsrv/concfg/font.c b/win32ss/user/winsrv/concfg/font.c index 485126a4928..74ef6292fe0 100644 --- a/win32ss/user/winsrv/concfg/font.c +++ b/win32ss/user/winsrv/concfg/font.c @@ -24,7 +24,9 @@ /* GLOBALS ********************************************************************/ -#define TERMINAL_FACENAME L"Terminal" +#define TERMINAL_FACENAME L"Terminal" +#define DEFAULT_NON_DBCS_FONTFACE L"Lucida Console" // L"Consolas" +#define DEFAULT_TT_FONT_FACENAME L"__DefaultTTFont__" /* TrueType font list cache */ SINGLE_LIST_ENTRY TTFontCache = { NULL }; @@ -322,6 +324,35 @@ FindSuitableFont( Param.SearchFont.CharSet = CodePageToCharSet(CodePage); Param.CodePage = CodePage; + if (/* !FaceName || */ !*FaceName) + { + /* Find and use a default Raster font */ + + /* Use "Terminal" as the fallback */ + StringCchCopyW(FaceName, LF_FACESIZE, TERMINAL_FACENAME); +#if 0 + // FIXME: CJK font choose workaround: Don't choose Asian + // charset font if there is no preferred font for CJK. + if (IsCJKCodePage(CodePage)) + FontData->CharSet = ANSI_CHARSET; +#endif + FontData->Family &= ~TMPF_TRUETYPE; + } + else if (wcscmp(FaceName, DEFAULT_TT_FONT_FACENAME) == 0) + { + /* Find and use a default TrueType font */ + FontEntry = FindCachedTTFont(NULL, CodePage); + if (FontEntry) + { + StringCchCopyW(FaceName, LF_FACESIZE, FontEntry->FaceName); + } + else + { + StringCchCopyW(FaceName, LF_FACESIZE, DEFAULT_NON_DBCS_FONTFACE); + } + FontData->Family |= TMPF_TRUETYPE; + } + /* Search for a TrueType alternative face name */ FontEntry = FindCachedTTFont(FaceName, CodePage); if (FontEntry) @@ -492,12 +523,20 @@ CreateConsoleFontEx( FontData->Size.Y = Height; FontData->CharSet = 0; // CodePageToCharSet(CodePage); - hFont = CreateConsoleFontWorker(FontData, CodePage); - if (hFont) - return hFont; + if (/* !FaceName || */ !*FaceName || wcscmp(FaceName, DEFAULT_TT_FONT_FACENAME) == 0) + { + /* We do not have an actual font face name yet and should find one. + * Call FindSuitableFont() to determine the default font to use. */ + } + else + { + hFont = CreateConsoleFontWorker(FontData, CodePage); + if (hFont) + return hFont; - DBGFNT1("CreateConsoleFont('%S') failed - Try to find a suitable font...\n", - FaceName); + DBGFNT1("CreateConsoleFont('%S') failed - Try to find a suitable font...\n", + FaceName); + } /* * We could not create a font with the default settings. @@ -515,6 +554,9 @@ CreateConsoleFontEx( if (!UseDefaultFallback) return NULL; + // + // FIXME: See also !*FaceName case in FindSuitableFont(). + // /* Use "Terminal" as the fallback */ StringCchCopyW(FaceName, LF_FACESIZE, TERMINAL_FACENAME); #if 0 @@ -523,6 +565,7 @@ CreateConsoleFontEx( if (IsCJKCodePage(CodePage)) FontData->CharSet = ANSI_CHARSET; #endif + FontData->Family &= ~TMPF_TRUETYPE; } else { @@ -1113,33 +1156,74 @@ RefreshTTFontCache(VOID) InitTTFontCache(); } +/** + * @brief + * Searches for a font in the console TrueType font cache, + * with the specified code page. + * + * @param[in,opt] FaceName + * An optional pointer to a maximally @b LF_FACESIZE-sized buffer. + * The buffer contains the face name of the font to search for. + * + * - If FaceName != NULL, search for the named font that should + * match the provided code page (when CodePage != INVALID_CP). + * + * - If FaceName == NULL, search for a font with the provided + * code page. In this case, CodePage cannot be == INVALID_CP, + * otherwise the search fails. + * + * @param[in] CodePage + * The code page the font has to support, or @b INVALID_CP when + * searching a font by face name only. + * + * @return + * A pointer to the cache entry for the font, or @b NULL if not found. + **/ PTT_FONT_ENTRY FindCachedTTFont( - // _In_reads_or_z_(LF_FACESIZE) - _In_ PCWSTR FaceName, + _In_reads_or_z_opt_(LF_FACESIZE) + PCWSTR FaceName, _In_ UINT CodePage) { PSINGLE_LIST_ENTRY Entry; PTT_FONT_ENTRY FontEntry; - /* Search for the font in the cache */ - for (Entry = TTFontCache.Next; - Entry != NULL; - Entry = Entry->Next) + if (FaceName) { - FontEntry = CONTAINING_RECORD(Entry, TT_FONT_ENTRY, Entry); - - /* NOTE: The font face names are case-sensitive */ - if ((wcscmp(FontEntry->FaceName , FaceName) == 0) || - (wcscmp(FontEntry->FaceNameAlt, FaceName) == 0)) + /* Search for the named font */ + for (Entry = TTFontCache.Next; + Entry != NULL; + Entry = Entry->Next) { - /* Return a match if we don't look at the code pages, or when they match */ - if ((CodePage == INVALID_CP) || (CodePage == FontEntry->CodePage)) + FontEntry = CONTAINING_RECORD(Entry, TT_FONT_ENTRY, Entry); + + /* NOTE: The font face names are case-sensitive */ + if ((wcscmp(FontEntry->FaceName , FaceName) == 0) || + (wcscmp(FontEntry->FaceNameAlt, FaceName) == 0)) { - return FontEntry; + /* Return the font if we don't search by code page, or when they match */ + if ((CodePage == INVALID_CP) || (CodePage == FontEntry->CodePage)) + { + return FontEntry; + } } } } + else if (CodePage != INVALID_CP) + { + /* Search for a font with the specified code page */ + for (Entry = TTFontCache.Next; + Entry != NULL; + Entry = Entry->Next) + { + FontEntry = CONTAINING_RECORD(Entry, TT_FONT_ENTRY, Entry); + + /* Return the font if the code pages match */ + if (CodePage == FontEntry->CodePage) + return FontEntry; + } + } + return NULL; } diff --git a/win32ss/user/winsrv/concfg/font.h b/win32ss/user/winsrv/concfg/font.h index bff2107a7c8..382792817c5 100644 --- a/win32ss/user/winsrv/concfg/font.h +++ b/win32ss/user/winsrv/concfg/font.h @@ -150,10 +150,21 @@ RefreshTTFontCache(VOID); PTT_FONT_ENTRY FindCachedTTFont( - // _In_reads_or_z_(LF_FACESIZE) - _In_ PCWSTR FaceName, + _In_reads_or_z_opt_(LF_FACESIZE) + PCWSTR FaceName, _In_ UINT CodePage); +/** + * @brief + * Verifies whether the given font is an additional console TrueType font. + * Wrapper macros around FindCachedTTFont(). + * + * @remark + * These macros are equivalents of the functions + * IsAvailableTTFont() and IsAvailableTTFontCP() in + * https://github.com/microsoft/terminal/blob/main/src/propsheet/dbcs.cpp + **/ + #define IsAdditionalTTFont(FaceName) \ (FindCachedTTFont((FaceName), INVALID_CP) != NULL)

1 0

[reactos] 02/03: [CONCFG:FONT] Convert the TT_FONT_ENTRY structure to one similar to MS Terminal's _TT_FONT_LIST. (#4337)
by Hermès Bélusca-Maïto 08 Feb '22

08 Feb '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=0b6b0b0021d37f63e5eec… commit 0b6b0b0021d37f63e5eec7f69ce7bdc678951098 Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Mon Jul 5 20:07:21 2021 +0200 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Tue Feb 8 15:59:08 2022 +0100 [CONCFG:FONT] Convert the TT_FONT_ENTRY structure to one similar to MS Terminal's _TT_FONT_LIST. (#4337) See the definition of struct _TT_FONT_LIST in https://github.com/microsoft/terminal/blob/main/dep/Console/winconp.h Add Doxygen documentation. [CONSOLE.CPL][CONSRV] "fonts cache" -> "font cache". --- dll/cpl/console/console.c | 2 +- win32ss/user/winsrv/concfg/font.c | 54 ++++++++++++++-------- win32ss/user/winsrv/concfg/font.h | 22 +++++---- win32ss/user/winsrv/consrv/frontends/gui/guiterm.c | 2 +- 4 files changed, 50 insertions(+), 30 deletions(-) diff --git a/dll/cpl/console/console.c b/dll/cpl/console/console.c index 65640bc9282..077ff3a9278 100644 --- a/dll/cpl/console/console.c +++ b/dll/cpl/console/console.c @@ -227,7 +227,7 @@ InitApplet(HANDLE hSectionOrWnd) InitDefaultConsoleInfo(ConInfo); } - /* Initialize the font support -- additional TrueType fonts cache and current preview font */ + /* Initialize the font support -- additional TrueType font cache and current preview font */ InitTTFontCache(); RefreshFontPreview(&FontPreview, ConInfo); diff --git a/win32ss/user/winsrv/concfg/font.c b/win32ss/user/winsrv/concfg/font.c index 59b12e3ae90..485126a4928 100644 --- a/win32ss/user/winsrv/concfg/font.c +++ b/win32ss/user/winsrv/concfg/font.c @@ -26,8 +26,8 @@ #define TERMINAL_FACENAME L"Terminal" -// RTL_STATIC_LIST_HEAD(TTFontCache); -LIST_ENTRY TTFontCache = {&TTFontCache, &TTFontCache}; +/* TrueType font list cache */ +SINGLE_LIST_ENTRY TTFontCache = { NULL }; // NOTE: Used to tag code that makes sense only with a font cache. // #define FONT_CACHE_PRESENT @@ -952,15 +952,20 @@ IsValidConsoleFont( return Param.IsValidFont; } -/* + +/** + * @brief + * Initializes the console TrueType font cache. + * + * @remark * To install additional TrueType fonts to be available for the console, * add entries of type REG_SZ named "0", "00" etc... in: * HKEY_LOCAL_MACHINE\Software\Microsoft\Windows NT\CurrentVersion\Console\TrueTypeFont * The names of the fonts listed there should match those in: * HKEY_LOCAL_MACHINE\Software\Microsoft\Windows NT\CurrentVersion\Fonts * - * This function initializes the cache of the fonts listed there. - */ + * @return None. + **/ VOID InitTTFontCache(VOID) { @@ -975,9 +980,9 @@ InitTTFontCache(VOID) PTT_FONT_ENTRY FontEntry; PWCHAR pszNext; - if (!IsListEmpty(&TTFontCache)) + if (TTFontCache.Next != NULL) return; - // InitializeListHead(&TTFontCache); + // TTFontCache.Next = NULL; /* Open the Console\TrueTypeFont key */ // "\\Registry\\Machine\\Software\\Microsoft\\Windows NT\\CurrentVersion\\Console\\TrueTypeFont" @@ -1034,7 +1039,7 @@ InitTTFontCache(VOID) pszNext = szValue; /* Check whether bold is disabled for this font */ - if (*pszNext == L'*') + if (*pszNext == BOLD_MARK) { FontEntry->DisableBold = TRUE; ++pszNext; @@ -1054,7 +1059,7 @@ InitTTFontCache(VOID) pszNext += wcslen(pszNext) + 1; /* Check whether bold is disabled for this font */ - if (*pszNext == L'*') + if (*pszNext == BOLD_MARK) { FontEntry->DisableBold = TRUE; ++pszNext; @@ -1066,28 +1071,41 @@ InitTTFontCache(VOID) pszNext, wcslen(pszNext)); } - InsertTailList(&TTFontCache, &FontEntry->Entry); + PushEntryList(&TTFontCache, &FontEntry->Entry); } /* Close the key and quit */ RegCloseKey(hKey); } +/** + * @brief + * Clears the console TrueType font cache. + * + * @return None. + **/ VOID ClearTTFontCache(VOID) { - PLIST_ENTRY Entry; + PSINGLE_LIST_ENTRY Entry; PTT_FONT_ENTRY FontEntry; - while (!IsListEmpty(&TTFontCache)) + while (TTFontCache.Next != NULL) { - Entry = RemoveHeadList(&TTFontCache); + Entry = PopEntryList(&TTFontCache); FontEntry = CONTAINING_RECORD(Entry, TT_FONT_ENTRY, Entry); RtlFreeHeap(RtlGetProcessHeap(), 0, FontEntry); } - InitializeListHead(&TTFontCache); + TTFontCache.Next = NULL; } +/** + * @brief + * Refreshes the console TrueType font cache, + * by clearing and re-initializing it. + * + * @return None. + **/ VOID RefreshTTFontCache(VOID) { @@ -1101,13 +1119,13 @@ FindCachedTTFont( _In_ PCWSTR FaceName, _In_ UINT CodePage) { - PLIST_ENTRY Entry; + PSINGLE_LIST_ENTRY Entry; PTT_FONT_ENTRY FontEntry; /* Search for the font in the cache */ - for (Entry = TTFontCache.Flink; - Entry != &TTFontCache; - Entry = Entry->Flink) + for (Entry = TTFontCache.Next; + Entry != NULL; + Entry = Entry->Next) { FontEntry = CONTAINING_RECORD(Entry, TT_FONT_ENTRY, Entry); diff --git a/win32ss/user/winsrv/concfg/font.h b/win32ss/user/winsrv/concfg/font.h index 6faa59dad33..bff2107a7c8 100644 --- a/win32ss/user/winsrv/concfg/font.h +++ b/win32ss/user/winsrv/concfg/font.h @@ -59,9 +59,19 @@ #define IsBoldFont(Weight) \ ((Weight) >= FW_SEMIBOLD) /* Sometimes, just > FW_MEDIUM */ + +/* + * @struct TrueType font list, cached from + * HKEY_LOCAL_MACHINE\Software\Microsoft\Windows NT\CurrentVersion\Console\TrueTypeFont + * + * See the definition of struct _TT_FONT_LIST + * in https://github.com/microsoft/terminal/blob/main/dep/Console/winconp.h + */ +#define BOLD_MARK L'*' + typedef struct _TT_FONT_ENTRY { - LIST_ENTRY Entry; + SINGLE_LIST_ENTRY Entry; UINT CodePage; BOOL DisableBold; WCHAR FaceName[LF_FACESIZE]; @@ -128,15 +138,7 @@ IsValidConsoleFont( _In_ PCWSTR FaceName, _In_ UINT CodePage); -/* - * To install additional TrueType fonts to be available for the console, - * add entries of type REG_SZ named "0", "00" etc... in: - * HKEY_LOCAL_MACHINE\Software\Microsoft\Windows NT\CurrentVersion\Console\TrueTypeFont - * The names of the fonts listed there should match those in: - * HKEY_LOCAL_MACHINE\Software\Microsoft\Windows NT\CurrentVersion\Fonts - * - * This function initializes the cache of the fonts listed there. - */ + VOID InitTTFontCache(VOID); diff --git a/win32ss/user/winsrv/consrv/frontends/gui/guiterm.c b/win32ss/user/winsrv/consrv/frontends/gui/guiterm.c index 0594ff4a0c8..f272b57acf0 100644 --- a/win32ss/user/winsrv/consrv/frontends/gui/guiterm.c +++ b/win32ss/user/winsrv/consrv/frontends/gui/guiterm.c @@ -301,7 +301,7 @@ GuiInit(IN PCONSOLE_INIT_INFO ConsoleInitInfo, /* Initialize and register the console window class */ if (!RegisterConWndClass(ConSrvDllInstance)) return FALSE; - /* Initialize the font support -- additional TrueType fonts cache */ + /* Initialize the font support -- additional TrueType font cache */ InitTTFontCache(); ConsInitialized = TRUE;

1 0

[reactos] 01/03: [CONCFG:FONT][CONSRV] Use a suitable font when changing console output CP; fail if none found. (#4337)
by Hermès Bélusca-Maïto 08 Feb '22

08 Feb '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=5d3915d0fcc465f65a614… commit 5d3915d0fcc465f65a6146b3d6bd9c050638dc18 Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Sat Jan 29 20:04:16 2022 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Tue Feb 8 15:59:07 2022 +0100 [CONCFG:FONT][CONSRV] Use a suitable font when changing console output CP; fail if none found. (#4337) CORE-12451, CORE-17601, CORE-17803 Replaces PR #4281. When changing the console output code page, check whether the current font can support it. If not, try to find a suitable font for the new code page. If none can be found: - if we are creating a new console, forcefully switch to codepage 437 (OEM USA) and retry finding a font, falling back to "Terminal" if none could be found; - if we were just changing the current CP, just fail and keep the old code page and font. Rework the console font selection/creation functions for this new job (see CreateConsoleFontEx() and friends). Elements of implementation based from https://github.com/microsoft/terminal ; see code for more information. Silence the noisy IsValidConsoleFont2() diagnostic messages. Add Doxygen documentation. [CONSOLE.CPL] Only add "Terminal" to the enumerated list of faces + add a TODO implementation comment. --- dll/cpl/console/font.c | 6 +- win32ss/user/winsrv/concfg/font.c | 703 +++++++++++++++++++-- win32ss/user/winsrv/concfg/font.h | 33 +- win32ss/user/winsrv/consrv/frontends/gui/conwnd.c | 113 +++- .../user/winsrv/consrv/frontends/gui/guisettings.c | 21 +- .../user/winsrv/consrv/frontends/gui/guisettings.h | 2 +- win32ss/user/winsrv/consrv/frontends/gui/guiterm.c | 27 +- win32ss/user/winsrv/consrv/frontends/gui/guiterm.h | 17 +- 8 files changed, 794 insertions(+), 128 deletions(-) diff --git a/dll/cpl/console/font.c b/dll/cpl/console/font.c index 4cb5f4c61d3..888375924bb 100644 --- a/dll/cpl/console/font.c +++ b/dll/cpl/console/font.c @@ -429,9 +429,11 @@ FaceNameList_Initialize( DPRINT1("The ideal console fonts were not found; manually add default ones.\n"); AddFontToList(hWndList, L"Terminal", RASTER_FONTTYPE); +#if 0 + // TODO: insert only the *single* default TT font, that should + // be found in the TT font cache with the codepage number 0. AddFontToList(hWndList, L"Lucida Console", TRUETYPE_FONTTYPE); - if (CodePageToCharSet(CodePage) != DEFAULT_CHARSET) - AddFontToList(hWndList, L"Droid Sans Fallback", TRUETYPE_FONTTYPE); +#endif } static VOID diff --git a/win32ss/user/winsrv/concfg/font.c b/win32ss/user/winsrv/concfg/font.c index 2889d9dd5ef..59b12e3ae90 100644 --- a/win32ss/user/winsrv/concfg/font.c +++ b/win32ss/user/winsrv/concfg/font.c @@ -1,10 +1,9 @@ /* - * COPYRIGHT: See COPYING in the top level directory - * PROJECT: ReactOS Console Server DLL - * FILE: win32ss/user/winsrv/concfg/font.c - * PURPOSE: Console Fonts Management - * PROGRAMMERS: Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) - * Katayama Hirofumi MZ (katayama.hirofumi.mz(a)gmail.com) + * PROJECT: ReactOS Console Server DLL + * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) + * PURPOSE: Console GDI Fonts Management. + * COPYRIGHT: Copyright 2017-2022 Hermès Bélusca-Maïto + * Copyright 2017 Katayama Hirofumi MZ (katayama.hirofumi.mz(a)gmail.com) */ /* INCLUDES *******************************************************************/ @@ -19,85 +18,594 @@ #define NDEBUG #include <debug.h> +#define DBGFNT DPRINT +#define DBGFNT1 DPRINT1 + /* GLOBALS ********************************************************************/ +#define TERMINAL_FACENAME L"Terminal" + // RTL_STATIC_LIST_HEAD(TTFontCache); LIST_ENTRY TTFontCache = {&TTFontCache, &TTFontCache}; +// NOTE: Used to tag code that makes sense only with a font cache. +// #define FONT_CACHE_PRESENT + + /* FUNCTIONS ******************************************************************/ -/* Retrieves the character set associated with a given code page */ +/** + * @brief + * Retrieves the character set associated with a given code page. + * + * @param[in] CodePage + * The code page to convert. + * + * @return + * The character set corresponding to the code page, or @b DEFAULT_CHARSET. + **/ BYTE CodePageToCharSet( _In_ UINT CodePage) { CHARSETINFO CharInfo; if (TranslateCharsetInfo(UlongToPtr(CodePage), &CharInfo, TCI_SRCCODEPAGE)) - return CharInfo.ciCharset; + return (BYTE)CharInfo.ciCharset; else return DEFAULT_CHARSET; } -HFONT -CreateConsoleFontEx( - _In_ LONG Height, - _In_opt_ LONG Width, - _Inout_updates_z_(LF_FACESIZE) - PWSTR FaceName, - _In_ ULONG FontFamily, - _In_ ULONG FontWeight, - _In_ UINT CodePage) +/*****************************************************************************/ + +typedef struct _FIND_SUITABLE_FONT_PROC_PARAM +{ + /* Search criteria */ + _In_reads_or_z_(LF_FACESIZE) PCWSTR AltFaceName; + FONT_DATA SearchFont; + UINT CodePage; + BOOL StrictSearch; // TRUE to do strict search; FALSE for relaxed criteria. + + /* Candidate font data */ + BOOL FontFound; // TRUE/FALSE if we have/haven't found a suitable font. + FONT_DATA CandidateFont; + WCHAR CandidateFaceName[LF_FACESIZE]; +} FIND_SUITABLE_FONT_PROC_PARAM, *PFIND_SUITABLE_FONT_PROC_PARAM; + +#define TM_IS_TT_FONT(x) (((x) & TMPF_TRUETYPE) == TMPF_TRUETYPE) +#define SIZE_EQUAL(s1, s2) (((s1).X == (s2).X) && ((s1).Y == (s2).Y)) + +/** + * @brief EnumFontFamiliesEx() callback helper for FindSuitableFont(). + * + * @remark + * It implements a nearly-identical console-suitable font search + * algorithm based on the one from FindCreateFont() + * https://github.com/microsoft/terminal/blob/main/src/propsheet/fontdlg.cpp#L… + * excepting that for now, it does not support an internal font cache. + **/ +static BOOL CALLBACK +FindSuitableFontProc( + _In_ PLOGFONTW lplf, + _In_ PNEWTEXTMETRICW lpntm, + _In_ DWORD FontType, + _In_ LPARAM lParam) +{ + PFIND_SUITABLE_FONT_PROC_PARAM Param = (PFIND_SUITABLE_FONT_PROC_PARAM)lParam; + PFONT_DATA SearchFont = &Param->SearchFont; + + if (!IsValidConsoleFont2(lplf, lpntm, FontType, Param->CodePage)) + { + /* This font does not suit us; continue enumeration */ + return TRUE; + } + +#ifndef FONT_CACHE_PRESENT + /* + * Since we don't cache all the possible font sizes for TrueType fonts, + * we cannot check our requested size (and weight) against the enumerated + * one; therefore reset the enumerated values to the requested ones. + * On the contrary, Raster fonts get their specific font sizes (and weights) + * enumerated separately, so for them we can keep the enumerated values. + */ + if (FontType == TRUETYPE_FONTTYPE) + { + lplf->lfHeight = SearchFont->Size.Y; + lplf->lfWidth = 0; // SearchFont->Size.X; + lplf->lfWeight = FW_NORMAL; + } +#endif + + if (Param->StrictSearch) + { + /* + * Find whether this is an exact match. + */ + + /* If looking for a particular family, skip non-matches */ + if ((SearchFont->Family != 0) && + ((BYTE)SearchFont->Family != (lplf->lfPitchAndFamily & 0xF0))) + { + /* Continue enumeration */ + return TRUE; + } + + /* Skip non-matching sizes */ +#if 0 + if ((FontInfo[i].SizeWant.Y != Size.Y) && + !SIZE_EQUAL(FontInfo[i].Size, Size)) +#endif + if ((lplf->lfHeight != SearchFont->Size.Y) && + !(lplf->lfWidth == SearchFont->Size.X && + lplf->lfHeight == SearchFont->Size.Y)) + { + /* Continue enumeration */ + return TRUE; + } + + /* Skip non-matching weights */ + if ((SearchFont->Weight != 0) && + (SearchFont->Weight != lplf->lfWeight)) + { + /* Continue enumeration */ + return TRUE; + } + + /* NOTE: We are making the font enumeration at fixed CharSet, + * with the one specified in the parameter block. */ + ASSERT(lplf->lfCharSet == SearchFont->CharSet); + + if ((FontType != TRUETYPE_FONTTYPE) && // !TM_IS_TT_FONT(lpntm->tmPitchAndFamily) + (lplf->lfCharSet != SearchFont->CharSet) && + !(lplf->lfCharSet == OEM_CHARSET && IsCJKCodePage(Param->CodePage))) // g_fEastAsianSystem + { + /* Continue enumeration */ + return TRUE; + } + + /* + * Size (and maybe family) match. If we don't care about the name or + * if it matches, use this font. Otherwise, if name doesn't match and + * it is a raster font, consider it. + * + * NOTE: The font face names are case-sensitive. + */ + if (!SearchFont->FaceName || !*(SearchFont->FaceName) || + (wcscmp(lplf->lfFaceName, SearchFont->FaceName) == 0) || + (wcscmp(lplf->lfFaceName, Param->AltFaceName) == 0)) + { + // FontIndex = i; + + PFONT_DATA CandidateFont = &Param->CandidateFont; + + CandidateFont->FaceName = Param->CandidateFaceName; + StringCchCopyNW(Param->CandidateFaceName, + ARRAYSIZE(Param->CandidateFaceName), + lplf->lfFaceName, ARRAYSIZE(lplf->lfFaceName)); + + CandidateFont->Weight = lplf->lfWeight; + CandidateFont->Family = (lplf->lfPitchAndFamily & 0xF0); + + CandidateFont->Size.X = lplf->lfWidth; + CandidateFont->Size.Y = lplf->lfHeight; + + CandidateFont->CharSet = lplf->lfCharSet; + + /* The font is found, stop enumeration */ + Param->FontFound = TRUE; + return FALSE; + } + else if (FontType != TRUETYPE_FONTTYPE) // !TM_IS_TT_FONT(lpntm->tmPitchAndFamily) + { + // FontIndex = i; + + PFONT_DATA CandidateFont = &Param->CandidateFont; + + CandidateFont->FaceName = Param->CandidateFaceName; + StringCchCopyNW(Param->CandidateFaceName, + ARRAYSIZE(Param->CandidateFaceName), + lplf->lfFaceName, ARRAYSIZE(lplf->lfFaceName)); + + CandidateFont->Weight = lplf->lfWeight; + CandidateFont->Family = (lplf->lfPitchAndFamily & 0xF0); + + CandidateFont->Size.X = lplf->lfWidth; + CandidateFont->Size.Y = lplf->lfHeight; + + CandidateFont->CharSet = lplf->lfCharSet; + + /* A close Raster Font fit was found; only the name doesn't match. + * Continue enumeration to see whether we can find better. */ + Param->FontFound = TRUE; + } + } + else // !Param->StrictSearch + { + /* + * Failed to find exact match, even after enumeration, so now + * try to find a font of same family and same size or bigger. + */ + + if (IsCJKCodePage(Param->CodePage)) // g_fEastAsianSystem + { + if ((SearchFont->Family != 0) && + ((BYTE)SearchFont->Family != (lplf->lfPitchAndFamily & 0xF0))) + { + /* Continue enumeration */ + return TRUE; + } + + if ((FontType != TRUETYPE_FONTTYPE) && // !TM_IS_TT_FONT(lpntm->tmPitchAndFamily) + (lplf->lfCharSet != SearchFont->CharSet)) + { + /* Continue enumeration */ + return TRUE; + } + } + else + { + if (// (SearchFont->Family != 0) && + ((BYTE)SearchFont->Family != (lplf->lfPitchAndFamily & 0xF0))) + { + /* Continue enumeration */ + return TRUE; + } + } + + if ((lplf->lfHeight >= SearchFont->Size.Y) && + (lplf->lfWidth >= SearchFont->Size.X)) + { + /* Same family, size >= desired */ + // FontIndex = i; + + PFONT_DATA CandidateFont = &Param->CandidateFont; + + CandidateFont->FaceName = Param->CandidateFaceName; + StringCchCopyNW(Param->CandidateFaceName, + ARRAYSIZE(Param->CandidateFaceName), + lplf->lfFaceName, ARRAYSIZE(lplf->lfFaceName)); + + CandidateFont->Weight = lplf->lfWeight; + CandidateFont->Family = (lplf->lfPitchAndFamily & 0xF0); + + CandidateFont->Size.X = lplf->lfWidth; + CandidateFont->Size.Y = lplf->lfHeight; + + CandidateFont->CharSet = lplf->lfCharSet; + + /* The font is found, stop enumeration */ + Param->FontFound = TRUE; + return FALSE; + } + } + + /* Continue enumeration */ + return TRUE; +} + +/** + * @brief + * Finds a font suitable for the given code page, based on the current font + * and its characteristics provided in input. + * + * @param[in,out] FontData + * In input: The face name and characteristics of the font to search for, + * possibly getting a best match. + * In output: The face name and characteristics of the suitable font, + * in case of success. + * + * @param[in] CodePage + * The code page the font has to support. + * + * @return + * @b TRUE in case a suitable font has been found. Its name and characteristics + * are returned in @b FontData. @b FALSE if no suitable font has been found. + **/ +static BOOL +FindSuitableFont( + _Inout_ PFONT_DATA FontData, + _In_ UINT CodePage) { + FIND_SUITABLE_FONT_PROC_PARAM Param; + _Inout_updates_z_(LF_FACESIZE) PWSTR FaceName; + HDC hDC; LOGFONTW lf; + PTT_FONT_ENTRY FontEntry; + + /* Save the original FaceName pointer */ + FaceName = FontData->FaceName; + + /* Save our current search criteria */ + RtlZeroMemory(&Param, sizeof(Param)); + Param.SearchFont = *FontData; + + Param.SearchFont.CharSet = CodePageToCharSet(CodePage); + Param.CodePage = CodePage; + + /* Search for a TrueType alternative face name */ + FontEntry = FindCachedTTFont(FaceName, CodePage); + if (FontEntry) + { + /* NOTE: The font face names are case-sensitive */ + if (wcscmp(FontEntry->FaceName, FaceName) == 0) + Param.AltFaceName = FontEntry->FaceNameAlt; + else if (wcscmp(FontEntry->FaceNameAlt, FaceName) == 0) + Param.AltFaceName = FontEntry->FaceName; + } + else + { + Param.AltFaceName = FaceName; + } + + /* Initialize the search: start with a strict search, then a relaxed one */ + Param.FontFound = FALSE; + Param.StrictSearch = TRUE; +SearchAgain: + /* + * Enumerate all fonts with the given character set. + * We will match them with the search criteria. + */ RtlZeroMemory(&lf, sizeof(lf)); + lf.lfCharSet = Param.SearchFont.CharSet; + // lf.lfPitchAndFamily = FIXED_PITCH | FF_MODERN; - lf.lfHeight = Height; - lf.lfWidth = Width; + hDC = GetDC(NULL); + EnumFontFamiliesExW(hDC, &lf, (FONTENUMPROCW)FindSuitableFontProc, (LPARAM)&Param, 0); + ReleaseDC(NULL, hDC); + + /* If we failed to find any font, search again with relaxed criteria */ + if (Param.StrictSearch && !Param.FontFound) + { + Param.StrictSearch = FALSE; + goto SearchAgain; + } + + /* If no font was found again, return failure */ + if (!Param.FontFound) + return FALSE; + + /* Return the font details */ + *FontData = Param.CandidateFont; + FontData->FaceName = FaceName; // Restore the original FaceName pointer. + StringCchCopyNW(FaceName, LF_FACESIZE, + Param.CandidateFaceName, + ARRAYSIZE(Param.CandidateFaceName)); + + return TRUE; +} + +/** + * @brief + * Validates and creates a suitable console font based on the font + * characteristics given in input. + * + * @param[in] FontData + * The face name and characteristics of the font to create. + * + * @param[in] CodePage + * The code page the font has to support. + * + * @return + * A GDI handle to the created font, or @b NULL in case of failure. + **/ +static HFONT +CreateConsoleFontWorker( + _In_ PFONT_DATA FontData, + _In_ UINT CodePage) +{ + LOGFONTW lf; + + RtlZeroMemory(&lf, sizeof(lf)); + + lf.lfHeight = (LONG)(ULONG)FontData->Size.Y; + lf.lfWidth = (LONG)(ULONG)FontData->Size.X; lf.lfEscapement = 0; lf.lfOrientation = 0; // TA_BASELINE; // TA_RTLREADING; when the console supports RTL? // lf.lfItalic = lf.lfUnderline = lf.lfStrikeOut = FALSE; - lf.lfWeight = FontWeight; + lf.lfWeight = FontData->Weight; lf.lfCharSet = CodePageToCharSet(CodePage); lf.lfOutPrecision = OUT_DEFAULT_PRECIS; lf.lfClipPrecision = CLIP_DEFAULT_PRECIS; lf.lfQuality = DEFAULT_QUALITY; /* Set the mandatory flags and remove those that we do not support */ - lf.lfPitchAndFamily = (BYTE)( (FIXED_PITCH | FF_MODERN | FontFamily) & + lf.lfPitchAndFamily = (BYTE)( (FIXED_PITCH | FF_MODERN | FontData->Family) & ~(VARIABLE_PITCH | FF_DECORATIVE | FF_ROMAN | FF_SCRIPT | FF_SWISS)); - if (!IsValidConsoleFont(FaceName, CodePage)) + if (!IsValidConsoleFont(FontData->FaceName, CodePage)) + return NULL; + + StringCchCopyNW(lf.lfFaceName, ARRAYSIZE(lf.lfFaceName), + FontData->FaceName, LF_FACESIZE); + + return CreateFontIndirectW(&lf); +} + +/*****************************************************************************/ + +/** + * @brief + * Validates and creates a suitable console font based on the font + * characteristics given in input. + * + * @param[in] Height + * The font height in cell units (pixels). + * + * @param[in,opt] Width + * The font width in cell units (pixels). + * + * @param[in,out] FaceName + * A pointer to a maximally @b LF_FACESIZE-sized buffer. + * In input: The buffer contains the face name of the font to try to create. + * In output: The buffer receives the face name of the font that has been + * created, in case of success. It may, or may not be, identical to the face + * name provided in input, in case a substitute font has been chosen. + * + * @param[in] FontWeight + * The font weight. + * + * @param[in] FontFamily + * The font family. + * + * @param[in] CodePage + * The code page the font has to support. + * + * @param[in] UseDefaultFallback + * Whether (@b TRUE) or not (@b FALSE) to use a default fallback font in case + * neither the specified font nor any substitute font could be found and + * created for the specified code page. + * + * @param[out] FontData + * The face name and characteristics of the created font. + * + * @return + * A GDI handle to the created font, or @b NULL in case of failure. + * + * @remark + * Similar to FindCreateFont() + * https://github.com/microsoft/terminal/blob/main/src/propsheet/fontdlg.cpp#L… + * but: + * - does not support an internal font cache for now; + * - returns a font handle (and not a font index to the cache). + **/ +HFONT +CreateConsoleFontEx( + _In_ LONG Height, + _In_opt_ LONG Width, + _Inout_updates_z_(LF_FACESIZE) + PWSTR FaceName, + _In_ ULONG FontWeight, + _In_ ULONG FontFamily, + _In_ UINT CodePage, + _In_ BOOL UseDefaultFallback, + _Out_ PFONT_DATA FontData) +{ + HFONT hFont; + + FontData->FaceName = FaceName; + FontData->Weight = FontWeight; + FontData->Family = FontFamily; + /* NOTE: FontSize is always in cell height/width units (pixels) */ + FontData->Size.X = Width; + FontData->Size.Y = Height; + FontData->CharSet = 0; // CodePageToCharSet(CodePage); + + hFont = CreateConsoleFontWorker(FontData, CodePage); + if (hFont) + return hFont; + + DBGFNT1("CreateConsoleFont('%S') failed - Try to find a suitable font...\n", + FaceName); + + /* + * We could not create a font with the default settings. + * Try to find a suitable font and retry. + */ + if (!FindSuitableFont(FontData, CodePage)) { - StringCchCopyW(FaceName, LF_FACESIZE, L"Terminal"); + /* We could not find any suitable font, fall back + * to some default one if required to do so. */ + DBGFNT1("FindSuitableFont could not find anything - %s\n", + UseDefaultFallback ? "Falling back to 'Terminal'" + : "Bailing out"); + + /* No fallback: no font! */ + if (!UseDefaultFallback) + return NULL; + + /* Use "Terminal" as the fallback */ + StringCchCopyW(FaceName, LF_FACESIZE, TERMINAL_FACENAME); +#if 0 + // FIXME: CJK font choose workaround: Don't choose Asian + // charset font if there is no preferred font for CJK. if (IsCJKCodePage(CodePage)) - { - lf.lfCharSet = ANSI_CHARSET; - } + FontData->CharSet = ANSI_CHARSET; +#endif + } + else + { + DBGFNT1("FindSuitableFont found: '%S', size (%d x %d)\n", + FaceName, FontData->Size.X, FontData->Size.Y); } - StringCchCopyNW(lf.lfFaceName, ARRAYSIZE(lf.lfFaceName), - FaceName, LF_FACESIZE); + /* Retry creating the font */ + hFont = CreateConsoleFontWorker(FontData, CodePage); + if (!hFont) + DBGFNT1("CreateConsoleFont('%S') failed\n", FaceName); - return CreateFontIndirectW(&lf); + return hFont; } +/** + * @brief + * A wrapper for CreateConsoleFontEx(). + * + * @param[in] Height + * The font height in cell units (pixels). + * + * @param[in,opt] Width + * The font width in cell units (pixels). + * + * @param[in,out] ConsoleInfo + * A pointer to console settings information, containing in particular + * (in input) the face name and characteristics of the font to create + * with the current console code page. + * In output, the font information gets updated. + * Note that a default fallback font is always being used in case neither + * the specified font nor any substitute font could be found and created + * for the specified code page. + * + * @return + * A GDI handle to the created font, or @b NULL in case of failure. + * + * @see CreateConsoleFontEx(), CreateConsoleFont() + **/ HFONT CreateConsoleFont2( _In_ LONG Height, _In_opt_ LONG Width, _Inout_ PCONSOLE_STATE_INFO ConsoleInfo) { - return CreateConsoleFontEx(Height, - Width, - ConsoleInfo->FaceName, - ConsoleInfo->FontFamily, - ConsoleInfo->FontWeight, - ConsoleInfo->CodePage); + FONT_DATA FontData; + HFONT hFont; + + hFont = CreateConsoleFontEx(Height, + Width, + ConsoleInfo->FaceName, + ConsoleInfo->FontWeight, + ConsoleInfo->FontFamily, + ConsoleInfo->CodePage, + TRUE, // UseDefaultFallback + &FontData); + if (hFont) + { + ConsoleInfo->FontWeight = FontData.Weight; + ConsoleInfo->FontFamily = FontData.Family; + } + + return hFont; } +/** + * @brief + * A wrapper for CreateConsoleFontEx(). + * + * @param[in,out] ConsoleInfo + * A pointer to console settings information, containing in particular + * (in input) the face name and characteristics of the font to create + * with the current console code page. + * In output, the font information gets updated. + * Note that a default fallback font is always being used in case neither + * the specified font nor any substitute font could be found and created + * for the specified code page. + * + * @return + * A GDI handle to the created font, or @b NULL in case of failure. + * + * @see CreateConsoleFontEx(), CreateConsoleFont2() + **/ HFONT CreateConsoleFont( _Inout_ PCONSOLE_STATE_INFO ConsoleInfo) @@ -108,14 +616,30 @@ CreateConsoleFont( * Height = FontSize.Y = HIWORD(FontSize); */ /* NOTE: FontSize is always in cell height/width units (pixels) */ - return CreateConsoleFontEx((LONG)(ULONG)ConsoleInfo->FontSize.Y, - (LONG)(ULONG)ConsoleInfo->FontSize.X, - ConsoleInfo->FaceName, - ConsoleInfo->FontFamily, - ConsoleInfo->FontWeight, - ConsoleInfo->CodePage); + return CreateConsoleFont2((LONG)(ULONG)ConsoleInfo->FontSize.Y, + (LONG)(ULONG)ConsoleInfo->FontSize.X, + ConsoleInfo); } +/** + * @brief + * Retrieves the cell size for a console font. + * + * @param[in,opt] hDC + * An optional GDI device context handle. + * + * @param[in] hFont + * The GDI handle to the font. + * + * @param[out] Height + * In case of success, receives the cell height size (in pixels). + * + * @param[out] Width + * In case of success, receives the cell height size (in pixels). + * + * @return + * @b TRUE if success, @b FALSE in case of failure. + **/ _Success_(return) BOOL GetFontCellSize( @@ -138,7 +662,7 @@ GetFontCellSize( hOldFont = SelectObject(hDC, hFont); if (hOldFont == NULL) { - DPRINT1("GetFontCellSize: SelectObject failed\n"); + DBGFNT1("GetFontCellSize: SelectObject failed\n"); goto Quit; } @@ -152,7 +676,7 @@ GetFontCellSize( */ if (!GetTextMetricsW(hDC, &tm)) { - DPRINT1("GetFontCellSize: GetTextMetrics failed\n"); + DBGFNT1("GetFontCellSize: GetTextMetrics failed\n"); goto Cleanup; } @@ -186,6 +710,31 @@ Quit: return Success; } +/** + * @brief + * Validates whether a given font can be supported in the console, + * under the specified code page. + * + * @param[in] lplf + * @param[in] lpntm + * @param[in] FontType + * The GDI font characteristics of the font to validate. + * + * @param[in] CodePage + * The code page the font has to support. + * + * @return + * @b TRUE if the font is valid and supported in the console, + * @b FALSE if not. + * + * @remark + * Equivalent of the font validation tests in FontEnumForV2Console() + * (or the more restrictive ones in FontEnum()) + * https://github.com/microsoft/terminal/blob/main/src/propsheet/misc.cpp#L465 + * https://github.com/microsoft/terminal/blob/main/src/propsheet/misc.cpp#L607 + * + * @see IsValidConsoleFont() + **/ BOOL IsValidConsoleFont2( _In_ PLOGFONTW lplf, @@ -212,7 +761,7 @@ IsValidConsoleFont2( * - If it is an Asian TrueType font, it must also be an Asian character set. * * See also Raymond Chen's blog: https://devblogs.microsoft.com/oldnewthing/?p=26843 - * and MIT-licensed Microsoft Terminal source code: https://github.com/microsoft/Terminal/blob/master/src/propsheet/misc.cpp + * and MIT-licensed Microsoft Terminal source code: https://github.com/microsoft/terminal/blob/main/src/propsheet/misc.cpp * for other details. * * To install additional TrueType fonts to be available for the console, @@ -241,27 +790,27 @@ IsValidConsoleFont2( /* ... if they are not in the list of additional TrueType fonts to include */ !IsAdditionalTTFont(FaceName) ) { - DPRINT1("Font '%S' rejected because it%s (lfPitchAndFamily = %d)\n", - FaceName, - !(lplf->lfPitchAndFamily & FIXED_PITCH) ? "'s not FIXED_PITCH" - : (!(lpntm->ntmFlags & NTM_NONNEGATIVE_AC) ? " has negative A or C space" - : " is broken"), - lplf->lfPitchAndFamily); + DBGFNT("Font '%S' rejected because it%s (lfPitchAndFamily = %d)\n", + FaceName, + !(lplf->lfPitchAndFamily & FIXED_PITCH) ? "'s not FIXED_PITCH" + : (!(lpntm->ntmFlags & NTM_NONNEGATIVE_AC) ? " has negative A or C space" + : " is broken"), + lplf->lfPitchAndFamily); return FALSE; } /* Reject TrueType fonts that are not FF_MODERN */ if ((FontType == TRUETYPE_FONTTYPE) && ((lplf->lfPitchAndFamily & 0xF0) != FF_MODERN)) { - DPRINT1("TrueType font '%S' rejected because it's not FF_MODERN (lfPitchAndFamily = %d)\n", - FaceName, lplf->lfPitchAndFamily); + DBGFNT("TrueType font '%S' rejected because it's not FF_MODERN (lfPitchAndFamily = %d)\n", + FaceName, lplf->lfPitchAndFamily); return FALSE; } /* Reject vertical fonts (tategaki) */ if (FaceName[0] == L'@') { - DPRINT1("Font '%S' rejected because it's vertical\n", FaceName); + DBGFNT("Font '%S' rejected because it's vertical\n", FaceName); return FALSE; } @@ -276,14 +825,10 @@ IsValidConsoleFont2( * Here we are inclusive and check for any CJK character set, * instead of looking just at the current one via CodePageToCharSet(). */ - if (!IsCJKCharSet(lplf->lfCharSet) -#if 1 // FIXME: Temporary HACK! - && wcscmp(FaceName, L"Terminal") != 0 -#endif - ) + if (!IsCJKCharSet(lplf->lfCharSet)) { - DPRINT1("TrueType font '%S' rejected because it's not Asian charset (lfCharSet = %d)\n", - FaceName, lplf->lfCharSet); + DBGFNT("TrueType font '%S' rejected because it's not Asian charset (lfCharSet = %d)\n", + FaceName, lplf->lfCharSet); return FALSE; } @@ -299,8 +844,8 @@ IsValidConsoleFont2( if (IsAdditionalTTFont(FaceName) && !IsAdditionalTTFontCP(FaceName, 0) && !IsCJKCharSet(lplf->lfCharSet)) { - DPRINT1("Cached TrueType font '%S' rejected because it claims a code page that is not Asian charset (lfCharSet = %d)\n", - FaceName, lplf->lfCharSet); + DBGFNT("Cached TrueType font '%S' rejected because it claims a code page that is not Asian charset (lfCharSet = %d)\n", + FaceName, lplf->lfCharSet); return FALSE; } } @@ -309,15 +854,15 @@ IsValidConsoleFont2( /* Reject non-TrueType fonts that do not have an Asian character set */ if (!IsCJKCharSet(lplf->lfCharSet) && (lplf->lfCharSet != OEM_CHARSET)) { - DPRINT1("Non-TrueType font '%S' rejected because it's not Asian charset or OEM_CHARSET (lfCharSet = %d)\n", - FaceName, lplf->lfCharSet); + DBGFNT("Non-TrueType font '%S' rejected because it's not Asian charset or OEM_CHARSET (lfCharSet = %d)\n", + FaceName, lplf->lfCharSet); return FALSE; } /* Reject non-TrueType fonts that are not Terminal */ - if (wcscmp(FaceName, L"Terminal") != 0) + if (wcscmp(FaceName, TERMINAL_FACENAME) != 0) { - DPRINT1("Non-TrueType font '%S' rejected because it's not 'Terminal'\n", FaceName); + DBGFNT("Non-TrueType font '%S' rejected because it's not 'Terminal'\n", FaceName); return FALSE; } } @@ -332,8 +877,8 @@ IsValidConsoleFont2( (lplf->lfCharSet != DEFAULT_CHARSET) && (lplf->lfCharSet != OEM_CHARSET)) { - DPRINT1("Non-TrueType font '%S' rejected because it's not ANSI_CHARSET or DEFAULT_CHARSET or OEM_CHARSET (lfCharSet = %d)\n", - FaceName, lplf->lfCharSet); + DBGFNT("Non-TrueType font '%S' rejected because it's not ANSI_CHARSET or DEFAULT_CHARSET or OEM_CHARSET (lfCharSet = %d)\n", + FaceName, lplf->lfCharSet); return FALSE; } } @@ -348,6 +893,9 @@ typedef struct _IS_VALID_CONSOLE_FONT_PARAM UINT CodePage; } IS_VALID_CONSOLE_FONT_PARAM, *PIS_VALID_CONSOLE_FONT_PARAM; +/** + * @brief EnumFontFamiliesEx() callback helper for IsValidConsoleFont(). + **/ static BOOL CALLBACK IsValidConsoleFontProc( _In_ PLOGFONTW lplf, @@ -362,6 +910,23 @@ IsValidConsoleFontProc( return FALSE; } +/** + * @brief + * Validates whether a given font can be supported in the console, + * under the specified code page. + * + * @param[in] FaceName + * The face name of the font to validate. + * + * @param[in] CodePage + * The code page the font has to support. + * + * @return + * @b TRUE if the font is valid and supported in the console, + * @b FALSE if not. + * + * @see IsValidConsoleFont2() + **/ BOOL IsValidConsoleFont( // _In_reads_or_z_(LF_FACESIZE) @@ -460,7 +1025,7 @@ InitTTFontCache(VOID) FontEntry = RtlAllocateHeap(RtlGetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*FontEntry)); if (!FontEntry) { - DPRINT1("InitTTFontCache: Failed to allocate memory, continuing...\n"); + DBGFNT1("InitTTFontCache: Failed to allocate memory, continuing...\n"); continue; } diff --git a/win32ss/user/winsrv/concfg/font.h b/win32ss/user/winsrv/concfg/font.h index 870b3805a63..6faa59dad33 100644 --- a/win32ss/user/winsrv/concfg/font.h +++ b/win32ss/user/winsrv/concfg/font.h @@ -1,10 +1,9 @@ /* - * COPYRIGHT: See COPYING in the top level directory - * PROJECT: ReactOS Console Server DLL - * FILE: win32ss/user/winsrv/concfg/font.h - * PURPOSE: Console Fonts Management - * PROGRAMMERS: Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) - * Katayama Hirofumi MZ (katayama.hirofumi.mz(a)gmail.com) + * PROJECT: ReactOS Console Server DLL + * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) + * PURPOSE: Console GDI Fonts Management. + * COPYRIGHT: Copyright 2017-2022 Hermès Bélusca-Maïto + * Copyright 2017 Katayama Hirofumi MZ (katayama.hirofumi.mz(a)gmail.com) */ #pragma once @@ -13,6 +12,12 @@ #define INVALID_CP ((UINT)-1) +#ifndef CP_UTF8 +#define CP_UTF8 65001 +#endif + +#define CP_USA 437 // United States (OEM) + #define CP_SHIFTJIS 932 // Japanese Shift-JIS #define CP_HANGUL 949 // Korean Hangul/Wansung #define CP_JOHAB 1361 // Korean Johab @@ -70,15 +75,27 @@ BYTE CodePageToCharSet( _In_ UINT CodePage); +// FIXME: Will be redefined once we support a font cache. +typedef struct _FONT_DATA +{ + _Inout_updates_z_(LF_FACESIZE) PWSTR FaceName; + ULONG Weight; + ULONG Family; + COORD Size; + BYTE CharSet; +} FONT_DATA, *PFONT_DATA; + HFONT CreateConsoleFontEx( _In_ LONG Height, _In_opt_ LONG Width, _Inout_updates_z_(LF_FACESIZE) PWSTR FaceName, - _In_ ULONG FontFamily, _In_ ULONG FontWeight, - _In_ UINT CodePage); + _In_ ULONG FontFamily, + _In_ UINT CodePage, + _In_ BOOL UseDefaultFallback, + _Out_ PFONT_DATA FontData); HFONT CreateConsoleFont2( diff --git a/win32ss/user/winsrv/consrv/frontends/gui/conwnd.c b/win32ss/user/winsrv/consrv/frontends/gui/conwnd.c index a1d180d5f93..8b6f67c7e9d 100644 --- a/win32ss/user/winsrv/consrv/frontends/gui/conwnd.c +++ b/win32ss/user/winsrv/consrv/frontends/gui/conwnd.c @@ -3,7 +3,7 @@ * PROJECT: ReactOS Console Server DLL * FILE: win32ss/user/winsrv/consrv/frontends/gui/conwnd.c * PURPOSE: GUI Console Window Class - * PROGRAMMERS: G� van Geldorp + * PROGRAMMERS: Gé van Geldorp * Johannes Anderwald * Jeffrey Morlan * Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) @@ -518,31 +518,54 @@ CreateDerivedFont(HFONT OrgFont, } BOOL -InitFonts(PGUI_CONSOLE_DATA GuiData, - LPWSTR FaceName, // Points to a WCHAR array of LF_FACESIZE elements. - ULONG FontFamily, - COORD FontSize, - ULONG FontWeight) +InitFonts( + _Inout_ PGUI_CONSOLE_DATA GuiData, + _In_reads_or_z_(LF_FACESIZE) + PCWSTR FaceName, + _In_ ULONG FontWeight, + _In_ ULONG FontFamily, + _In_ COORD FontSize, + _In_opt_ UINT CodePage, + _In_ BOOL UseDefaultFallback) { HDC hDC; HFONT hFont; + FONT_DATA FontData; + UINT OldCharWidth = GuiData->CharWidth; + UINT OldCharHeight = GuiData->CharHeight; + COORD OldFontSize = GuiData->GuiInfo.FontSize; + WCHAR NewFaceName[LF_FACESIZE]; + + /* Default to current code page if none has been provided */ + if (!CodePage) + CodePage = GuiData->Console->OutputCodePage; /* - * Initialize a new NORMAL font and get its character cell size. + * Initialize a new NORMAL font. */ + + /* Copy the requested face name into the local buffer. + * It will be modified in output by CreateConsoleFontEx() + * to hold a possible fallback font face name. */ + StringCchCopyNW(NewFaceName, ARRAYSIZE(NewFaceName), + FaceName, LF_FACESIZE); + /* NOTE: FontSize is always in cell height/width units (pixels) */ hFont = CreateConsoleFontEx((LONG)(ULONG)FontSize.Y, (LONG)(ULONG)FontSize.X, - FaceName, - FontFamily, + NewFaceName, FontWeight, - GuiData->Console->OutputCodePage); - if (hFont == NULL) + FontFamily, + CodePage, + UseDefaultFallback, + &FontData); + if (!hFont) { - DPRINT1("InitFonts: CreateConsoleFontEx failed\n"); + DPRINT1("InitFonts: CreateConsoleFontEx('%S') failed\n", NewFaceName); return FALSE; } + /* Retrieve its character cell size */ hDC = GetDC(GuiData->hWindow); if (!GetFontCellSize(hDC, hFont, &GuiData->CharHeight, &GuiData->CharWidth)) { @@ -561,35 +584,43 @@ InitFonts(PGUI_CONSOLE_DATA GuiData, GuiData->Font[FONT_NORMAL] = hFont; /* - * Now build the other fonts (bold, underlined, mixed). + * Now build the optional fonts (bold, underlined, mixed). + * Do not error in case they fail to be created. */ GuiData->Font[FONT_BOLD] = CreateDerivedFont(GuiData->Font[FONT_NORMAL], - FontWeight < FW_BOLD ? FW_BOLD : FontWeight, + max(FW_BOLD, FontData.Weight), FALSE, FALSE); GuiData->Font[FONT_UNDERLINE] = CreateDerivedFont(GuiData->Font[FONT_NORMAL], - FontWeight, + FontData.Weight, TRUE, FALSE); GuiData->Font[FONT_BOLD | FONT_UNDERLINE] = CreateDerivedFont(GuiData->Font[FONT_NORMAL], - FontWeight < FW_BOLD ? FW_BOLD : FontWeight, + max(FW_BOLD, FontData.Weight), TRUE, FALSE); /* - * Save the settings. + * Save the new font characteristics. */ - if (FaceName != GuiData->GuiInfo.FaceName) + StringCchCopyNW(GuiData->GuiInfo.FaceName, + ARRAYSIZE(GuiData->GuiInfo.FaceName), + NewFaceName, ARRAYSIZE(NewFaceName)); + GuiData->GuiInfo.FontWeight = FontData.Weight; + GuiData->GuiInfo.FontFamily = FontData.Family; + GuiData->GuiInfo.FontSize = FontData.Size; + + /* Resize the terminal, in case the new font has a different size */ + if ((OldCharWidth != GuiData->CharWidth) || + (OldCharHeight != GuiData->CharHeight) || + (OldFontSize.X != FontData.Size.X || + OldFontSize.Y != FontData.Size.Y)) { - StringCchCopyNW(GuiData->GuiInfo.FaceName, ARRAYSIZE(GuiData->GuiInfo.FaceName), - FaceName, LF_FACESIZE); + TermResizeTerminal(GuiData->Console); } - GuiData->GuiInfo.FontFamily = FontFamily; - GuiData->GuiInfo.FontSize = FontSize; - GuiData->GuiInfo.FontWeight = FontWeight; return TRUE; } @@ -615,14 +646,40 @@ OnNcCreate(HWND hWnd, LPCREATESTRUCTW Create) /* Initialize the fonts */ if (!InitFonts(GuiData, GuiData->GuiInfo.FaceName, + GuiData->GuiInfo.FontWeight, GuiData->GuiInfo.FontFamily, GuiData->GuiInfo.FontSize, - GuiData->GuiInfo.FontWeight)) + 0, FALSE)) { - DPRINT1("GuiConsoleNcCreate: InitFonts failed\n"); - GuiData->hWindow = NULL; - NtSetEvent(GuiData->hGuiInitEvent, NULL); - return FALSE; + /* Reset only the output code page if we don't have a suitable + * font for it, possibly falling back to "United States (OEM)". */ + UINT AltCodePage = GetOEMCP(); + + if (AltCodePage == Console->OutputCodePage) + AltCodePage = CP_USA; + + DPRINT1("Could not initialize font '%S' for code page %d - Resetting CP to %d\n", + GuiData->GuiInfo.FaceName, Console->OutputCodePage, AltCodePage); + + CON_SET_OUTPUT_CP(Console, AltCodePage); + + /* We will use a fallback font if we cannot find + * anything for this replacement code page. */ + if (!InitFonts(GuiData, + GuiData->GuiInfo.FaceName, + GuiData->GuiInfo.FontWeight, + GuiData->GuiInfo.FontFamily, + GuiData->GuiInfo.FontSize, + 0, TRUE)) + { + DPRINT1("Failed to initialize font '%S' for code page %d\n", + GuiData->GuiInfo.FaceName, Console->OutputCodePage); + + DPRINT1("GuiConsoleNcCreate: InitFonts failed\n"); + GuiData->hWindow = NULL; + NtSetEvent(GuiData->hGuiInitEvent, NULL); + return FALSE; + } } /* Initialize the terminal framebuffer */ diff --git a/win32ss/user/winsrv/consrv/frontends/gui/guisettings.c b/win32ss/user/winsrv/consrv/frontends/gui/guisettings.c index 1ee89ed8a47..38dec0bbd75 100644 --- a/win32ss/user/winsrv/consrv/frontends/gui/guisettings.c +++ b/win32ss/user/winsrv/consrv/frontends/gui/guisettings.c @@ -163,9 +163,9 @@ GuiConsoleShowConsoleProperties(PGUI_CONSOLE_DATA GuiData, /* GUI Information */ StringCchCopyNW(pSharedInfo->FaceName, ARRAYSIZE(pSharedInfo->FaceName), GuiData->GuiInfo.FaceName, ARRAYSIZE(GuiData->GuiInfo.FaceName)); + pSharedInfo->FontWeight = GuiData->GuiInfo.FontWeight; pSharedInfo->FontFamily = GuiData->GuiInfo.FontFamily; pSharedInfo->FontSize = GuiData->GuiInfo.FontSize; - pSharedInfo->FontWeight = GuiData->GuiInfo.FontWeight; pSharedInfo->FullScreen = GuiData->GuiInfo.FullScreen; pSharedInfo->AutoPosition = GuiData->GuiInfo.AutoPosition; pSharedInfo->WindowPosition = GuiData->GuiInfo.WindowOrigin; @@ -312,28 +312,29 @@ GuiApplyUserSettings(PGUI_CONSOLE_DATA GuiData, // TODO: Check that GuiData->hWindow == pConInfo->hWnd - /* Retrieve terminal informations */ - /* Console information */ /* * Apply the settings */ - /* Set the console informations */ + /* Refresh the additional TrueType fonts cache now, + * as ConSrvApplyUserSettings() could change the output + * code page and trigger a font change in the terminal. */ + RefreshTTFontCache(); + + /* Apply the generic console settings */ ConSrvApplyUserSettings(Console, pConInfo); - /* Set the terminal informations */ + /* Set the terminal settings */ - /* Refresh the additional TrueType fonts cache and change the font */ - RefreshTTFontCache(); + /* Now, attempt to change the font to what the user specified */ InitFonts(GuiData, pConInfo->FaceName, + pConInfo->FontWeight, pConInfo->FontFamily, pConInfo->FontSize, - pConInfo->FontWeight); - // HACK, needed because changing font may change the size of the window - /**/TermResizeTerminal(Console);/**/ + 0, FALSE); /* Move the window to the user's values */ GuiData->GuiInfo.AutoPosition = !!pConInfo->AutoPosition; diff --git a/win32ss/user/winsrv/consrv/frontends/gui/guisettings.h b/win32ss/user/winsrv/consrv/frontends/gui/guisettings.h index e67c537ee8f..71e968a57a6 100644 --- a/win32ss/user/winsrv/consrv/frontends/gui/guisettings.h +++ b/win32ss/user/winsrv/consrv/frontends/gui/guisettings.h @@ -16,9 +16,9 @@ typedef struct _GUI_CONSOLE_INFO { WCHAR FaceName[LF_FACESIZE]; + ULONG FontWeight; ULONG FontFamily; COORD FontSize; - ULONG FontWeight; BOOL FullScreen; /* Whether the console is displayed in full-screen or windowed mode */ // ULONG HardwareState; /* _GDI_MANAGED, _DIRECT */ diff --git a/win32ss/user/winsrv/consrv/frontends/gui/guiterm.c b/win32ss/user/winsrv/consrv/frontends/gui/guiterm.c index 778b11073c1..0594ff4a0c8 100644 --- a/win32ss/user/winsrv/consrv/frontends/gui/guiterm.c +++ b/win32ss/user/winsrv/consrv/frontends/gui/guiterm.c @@ -3,7 +3,7 @@ * PROJECT: ReactOS Console Server DLL * FILE: win32ss/user/winsrv/consrv/frontends/gui/guiterm.c * PURPOSE: GUI Terminal Front-End - * PROGRAMMERS: G� van Geldorp + * PROGRAMMERS: Gé van Geldorp * Johannes Anderwald * Jeffrey Morlan * Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) @@ -1038,9 +1038,28 @@ static BOOL NTAPI GuiSetCodePage(IN OUT PFRONTEND This, UINT CodePage) { - // TODO: Find a suitable console font for the given code page, - // and set it if found; otherwise fail the call, or fall back - // to some default font... + PGUI_CONSOLE_DATA GuiData = This->Context; + + /* + * Attempt to reinitialize the current font for the new code page, + * trying to keep the current font with the same characteristics. + * If the current font does not support the new code page, switch + * to a different font supporting the code page but having similar + * characteristics. + * If no font can be found for this code page, stay using the + * original font and refuse changing the code page. + */ + if (!InitFonts(GuiData, + GuiData->GuiInfo.FaceName, + GuiData->GuiInfo.FontWeight, + GuiData->GuiInfo.FontFamily, + GuiData->GuiInfo.FontSize, + CodePage, FALSE)) + { + DPRINT1("Failed to initialize font '%S' for code page %d - Refuse CP change\n", + GuiData->GuiInfo.FaceName, CodePage); + return FALSE; + } return TRUE; } diff --git a/win32ss/user/winsrv/consrv/frontends/gui/guiterm.h b/win32ss/user/winsrv/consrv/frontends/gui/guiterm.h index 4ed6c1e8c19..e83f2a0cf96 100644 --- a/win32ss/user/winsrv/consrv/frontends/gui/guiterm.h +++ b/win32ss/user/winsrv/consrv/frontends/gui/guiterm.h @@ -3,7 +3,7 @@ * PROJECT: ReactOS Console Server DLL * FILE: win32ss/user/winsrv/consrv/frontends/gui/guiterm.h * PURPOSE: GUI Terminal Front-End - * PROGRAMMERS: G� van Geldorp + * PROGRAMMERS: Gé van Geldorp * Johannes Anderwald * Jeffrey Morlan * Hermes Belusca-Maito (hermes.belusca(a)sfr.fr) @@ -65,11 +65,16 @@ GuiConsoleMoveWindow(PGUI_CONSOLE_DATA GuiData); /* conwnd.c */ BOOL -InitFonts(PGUI_CONSOLE_DATA GuiData, - LPWSTR FaceName, // Points to a WCHAR array of LF_FACESIZE elements. - ULONG FontFamily, - COORD FontSize, - ULONG FontWeight); +InitFonts( + _Inout_ PGUI_CONSOLE_DATA GuiData, + _In_reads_or_z_(LF_FACESIZE) + PCWSTR FaceName, + _In_ ULONG FontWeight, + _In_ ULONG FontFamily, + _In_ COORD FontSize, + _In_opt_ UINT CodePage, + _In_ BOOL UseDefaultFallback); + VOID DeleteFonts(PGUI_CONSOLE_DATA GuiData);

1 0

[reactos] 03/03: [KERNEL32] SetThreadUILanguage(): Validate the parameter. Use LANGIDFROMLCID() macro. (#4301)
by Hermès Bélusca-Maïto 08 Feb '22

08 Feb '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=a94436b5194ca86fedc75… commit a94436b5194ca86fedc759c63f79653c85e281a0 Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Fri Jan 21 18:01:29 2022 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Tue Feb 8 15:58:04 2022 +0100 [KERNEL32] SetThreadUILanguage(): Validate the parameter. Use LANGIDFROMLCID() macro. (#4301) Both ReactOS and Win2k3 only support '0' for the parameter. --- dll/win32/kernel32/client/thread.c | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/dll/win32/kernel32/client/thread.c b/dll/win32/kernel32/client/thread.c index be33021fcf3..6d31a1b394d 100644 --- a/dll/win32/kernel32/client/thread.c +++ b/dll/win32/kernel32/client/thread.c @@ -935,8 +935,19 @@ LANGID WINAPI SetThreadUILanguage(IN LANGID LangId) { +#if (NTDDI_VERSION < NTDDI_LONGHORN) + /* We only support LangId == 0, for selecting a language + * identifier that best supports the NT Console. */ + if (LangId != 0) + { + BaseSetLastNTError(STATUS_NOT_SUPPORTED); + return 0; + } +#endif + UNIMPLEMENTED; - return (LANGID)NtCurrentTeb()->CurrentLocale; + + return LANGIDFROMLCID(NtCurrentTeb()->CurrentLocale); } /*

1 0

[reactos] 02/03: [KERNEL32][CONSRV] Retrieve the best-suited language ID corresponding to the active console output code page. (#4301)
by Hermès Bélusca-Maïto 08 Feb '22

08 Feb '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=3e44a5d71c6ef6f0d6662… commit 3e44a5d71c6ef6f0d66625369f2633318a73a493 Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Fri Jan 14 22:00:55 2022 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Tue Feb 8 15:58:02 2022 +0100 [KERNEL32][CONSRV] Retrieve the best-suited language ID corresponding to the active console output code page. (#4301) CORE-17601, CORE-17803 Replaces PR #4281. Implement SrvGetConsoleLangId() (server-side) and set the new current thread's locale after connecting to a console, or changing its output code page. Based on API tracing on Windows 2003, as well as on comments and code gathered from: https://github.com/microsoft/terminal Tests results are listed in PR #4301. --- dll/win32/kernel32/client/console/console.c | 57 +++++++++++++++++++++---- dll/win32/kernel32/client/console/init.c | 9 ++-- dll/win32/kernel32/include/console.h | 3 +- sdk/include/reactos/subsys/win/conmsg.h | 7 ++++ win32ss/user/winsrv/concfg/font.h | 9 ++++ win32ss/user/winsrv/consrv/console.c | 64 +++++++++++++++++++++++++++-- 6 files changed, 131 insertions(+), 18 deletions(-) diff --git a/dll/win32/kernel32/client/console/console.c b/dll/win32/kernel32/client/console/console.c index 5c687dfde1d..7a5b3b61c34 100644 --- a/dll/win32/kernel32/client/console/console.c +++ b/dll/win32/kernel32/client/console/console.c @@ -1375,8 +1375,6 @@ AllocConsole(VOID) ULONG AppNameLength = 128 * sizeof(WCHAR); ULONG CurDirLength = (MAX_PATH + 1) * sizeof(WCHAR); - LCID lcid; - RtlEnterCriticalSection(&ConsoleLock); if (NtCurrentPeb()->ProcessParameters->ConsoleHandle) @@ -1427,8 +1425,8 @@ AllocConsole(VOID) /* Initialize Console Ctrl Handling */ InitializeCtrlHandling(); - /* Sets the current console locale for this thread */ - SetTEBLangID(lcid); + /* Sync the current thread's LangId with the console's one */ + SetTEBLangID(); } Quit: @@ -2500,6 +2498,9 @@ SetConsoleOutputCP(UINT wCodePageID) return FALSE; } + /* Sync the current thread's LangId with the console's one */ + SetTEBLangID(); + return TRUE; } @@ -2676,9 +2677,7 @@ AttachConsole(DWORD dwProcessId) { BOOL Success; CONSOLE_START_INFO ConsoleStartInfo; - DWORD dummy; - LCID lcid; RtlEnterCriticalSection(&ConsoleLock); @@ -2711,8 +2710,8 @@ AttachConsole(DWORD dwProcessId) /* Initialize Console Ctrl Handling */ InitializeCtrlHandling(); - /* Sets the current console locale for this thread */ - SetTEBLangID(lcid); + /* Sync the current thread's LangId with the console's one */ + SetTEBLangID(); } Quit: @@ -3067,6 +3066,48 @@ UnregisterConsoleIME(VOID) return FALSE; } +/** + * @brief + * Internal helper function used to synchronize the current + * thread's language ID with the one from the console. + **/ +VOID +SetTEBLangID(VOID) +{ + CONSOLE_API_MESSAGE ApiMessage; + PCONSOLE_GETLANGID LangIdRequest = &ApiMessage.Data.LangIdRequest; + + /* Retrieve the "best-suited" language ID corresponding + * to the active console output code page. */ + LangIdRequest->ConsoleHandle = NtCurrentPeb()->ProcessParameters->ConsoleHandle; + + CsrClientCallServer((PCSR_API_MESSAGE)&ApiMessage, + NULL, + CSR_CREATE_API_NUMBER(CONSRV_SERVERDLL_INDEX, ConsolepGetLangId), + sizeof(*LangIdRequest)); + if (!NT_SUCCESS(ApiMessage.Status)) + { + /* + * No best console language ID: keep the current thread's one. + * Since this internal function only modifies an optional setting, + * don't set any last error, as it could otherwise mess with the + * main last error set by the caller. + */ + return; + } + + /* + * We succeeded, set the current thread's language ID by + * modifying its locale -- Windows <= 2003 does not have + * the concept of a separate thread UI language. + * Ignore the returned value. + */ + if (!SetThreadLocale(MAKELCID(LangIdRequest->LangId, SORT_DEFAULT))) + { + DPRINT1("SetTEBLangID: Could not set thread locale to console lang ID %lu\n", + LangIdRequest->LangId); + } +} static BOOL diff --git a/dll/win32/kernel32/client/console/init.c b/dll/win32/kernel32/client/console/init.c index 3186bf4e71f..eb97ac73ded 100644 --- a/dll/win32/kernel32/client/console/init.c +++ b/dll/win32/kernel32/client/console/init.c @@ -342,14 +342,13 @@ ConDllInitialize(IN ULONG Reason, PRTL_USER_PROCESS_PARAMETERS Parameters = NtCurrentPeb()->ProcessParameters; BOOLEAN InServerProcess = FALSE; CONSRV_API_CONNECTINFO ConnectInfo; - LCID lcid; if (Reason != DLL_PROCESS_ATTACH) { if ((Reason == DLL_THREAD_ATTACH) && IsConsoleApp()) { - /* Sets the current console locale for the new thread */ - SetTEBLangID(lcid); + /* Sync the new thread's LangId with the console's one */ + SetTEBLangID(); } else if (Reason == DLL_PROCESS_DETACH) { @@ -522,8 +521,8 @@ ConDllInitialize(IN ULONG Reason, InputWaitHandle = ConnectInfo.ConsoleStartInfo.InputWaitHandle; - /* Sets the current console locale for this thread */ - SetTEBLangID(lcid); + /* Sync the current thread's LangId with the console's one */ + SetTEBLangID(); } DPRINT("Console setup: 0x%p, 0x%p, 0x%p, 0x%p\n", diff --git a/dll/win32/kernel32/include/console.h b/dll/win32/kernel32/include/console.h index b6462af46b6..5fe84cb304c 100644 --- a/dll/win32/kernel32/include/console.h +++ b/dll/win32/kernel32/include/console.h @@ -60,7 +60,8 @@ GetConsoleInputWaitHandle(VOID); HANDLE TranslateStdHandle(HANDLE hHandle); -#define SetTEBLangID(p) (p) +VOID +SetTEBLangID(VOID); VOID SetUpConsoleInfo(IN BOOLEAN CaptureTitle, diff --git a/sdk/include/reactos/subsys/win/conmsg.h b/sdk/include/reactos/subsys/win/conmsg.h index b96441dd9aa..dd50cb11437 100644 --- a/sdk/include/reactos/subsys/win/conmsg.h +++ b/sdk/include/reactos/subsys/win/conmsg.h @@ -859,6 +859,12 @@ typedef struct _CONSOLE_SETINPUTOUTPUTCP HANDLE EventHandle; } CONSOLE_SETINPUTOUTPUTCP, *PCONSOLE_SETINPUTOUTPUTCP; +typedef struct _CONSOLE_GETLANGID +{ + HANDLE ConsoleHandle; + LANGID LangId; +} CONSOLE_GETLANGID, *PCONSOLE_GETLANGID; + typedef struct _CONSOLE_GETKBDLAYOUTNAME { HANDLE ConsoleHandle; @@ -991,6 +997,7 @@ typedef struct _CONSOLE_API_MESSAGE /* Input and Output Code Pages; keyboard */ CONSOLE_GETINPUTOUTPUTCP GetConsoleCPRequest; CONSOLE_SETINPUTOUTPUTCP SetConsoleCPRequest; + CONSOLE_GETLANGID LangIdRequest; CONSOLE_GETKBDLAYOUTNAME GetKbdLayoutNameRequest; /* Virtual DOS Machine */ diff --git a/win32ss/user/winsrv/concfg/font.h b/win32ss/user/winsrv/concfg/font.h index 15224b3f9d6..870b3805a63 100644 --- a/win32ss/user/winsrv/concfg/font.h +++ b/win32ss/user/winsrv/concfg/font.h @@ -19,6 +19,15 @@ #define CP_GB2312 936 // Chinese Simplified (GB2312) #define CP_BIG5 950 // Chinese Traditional (Big5) +/* + * "Human-understandable" names for the previous standard code pages. + * Taken from https://github.com/microsoft/terminal/blob/main/src/inc/unicode.hpp + */ +#define CP_JAPANESE CP_SHIFTJIS +#define CP_KOREAN CP_HANGUL +#define CP_CHINESE_SIMPLIFIED CP_GB2312 +#define CP_CHINESE_TRADITIONAL CP_BIG5 + /* IsFarEastCP(CodePage) */ #define IsCJKCodePage(CodePage) \ ((CodePage) == CP_SHIFTJIS || (CodePage) == CP_HANGUL || \ diff --git a/win32ss/user/winsrv/consrv/console.c b/win32ss/user/winsrv/consrv/console.c index 1e35fdea127..cc269604ba7 100644 --- a/win32ss/user/winsrv/consrv/console.c +++ b/win32ss/user/winsrv/consrv/console.c @@ -16,7 +16,7 @@ #define COBJMACROS #include <shlobj.h> - +#include "../concfg/font.h" #include <alias.h> #include <history.h> #include "procinit.h" @@ -2014,10 +2014,66 @@ CSR_API(SrvSetConsoleNlsMode) } /* API_NUMBER: ConsolepGetLangId */ -CSR_API(SrvGetConsoleLangId) +CON_API(SrvGetConsoleLangId, + CONSOLE_GETLANGID, LangIdRequest) { - DPRINT1("%s not yet implemented\n", __FUNCTION__); - return STATUS_NOT_IMPLEMENTED; + /* + * Quoting MS Terminal, see function GetConsoleLangId() at + * https://github.com/microsoft/terminal/blob/main/src/host/srvinit.cpp#L655 + * "Only attempt to return the Lang ID if the Windows ACP on console + * launch was an East Asian Code Page." + * + * The underlying logic is as follows: + * + * - When the current user's UI language is *not* CJK, the user expects + * to not see any CJK output to the console by default, even if its + * output has been set to a CJK code page (this is possible when CJK + * fonts are installed on the system). That is, of course, unless if + * the attached console program chooses to actually output CJK text. + * Whatever current language of the running program's thread should + * be kept: STATUS_NOT_SUPPORTED is returned. + * + * - When the current user's UI language *is* CJK, the user expects to + * see CJK output to the console by default when its code page is CJK. + * A valid LangId is returned in this case to ensure this. + * However, if the console code page is not CJK, then it is evident + * that CJK text will not be able to be correctly shown, and therefore + * we should fall back to a standard language that can be shown, namely + * en-US english, instead of keeping the current language. + */ + + BYTE UserCharSet = CodePageToCharSet(GetACP()); + if (!IsCJKCharSet(UserCharSet)) + return STATUS_NOT_SUPPORTED; + + /* Return a "best-suited" language ID corresponding + * to the active console output code page. */ + switch (Console->OutputCodePage) + { +/** ReactOS-specific: do nothing if the code page is UTF-8. This will allow + ** programs to naturally output in whatever current language they are. **/ + case CP_UTF8: + return STATUS_NOT_SUPPORTED; +/** End ReactOS-specific **/ + case CP_JAPANESE: + LangIdRequest->LangId = MAKELANGID(LANG_JAPANESE, SUBLANG_DEFAULT); + break; + case CP_KOREAN: + LangIdRequest->LangId = MAKELANGID(LANG_KOREAN, SUBLANG_KOREAN); + break; + case CP_CHINESE_SIMPLIFIED: + LangIdRequest->LangId = MAKELANGID(LANG_CHINESE, SUBLANG_CHINESE_SIMPLIFIED); + break; + case CP_CHINESE_TRADITIONAL: + LangIdRequest->LangId = MAKELANGID(LANG_CHINESE, SUBLANG_CHINESE_TRADITIONAL); + break; + default: + /* Default to en-US english otherwise */ + LangIdRequest->LangId = MAKELANGID(LANG_ENGLISH, SUBLANG_ENGLISH_US); + break; + } + + return STATUS_SUCCESS; } /* EOF */

1 0

[reactos] 01/03: [KERNEL32_APITESTS] Add a test showing thread locale/LangID sync with console (#4301)
by Hermès Bélusca-Maïto 08 Feb '22

08 Feb '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=ba516ff0f5c83f86d76b1… commit ba516ff0f5c83f86d76b1383dbc58b6ef03f9031 Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Sat Feb 5 18:44:02 2022 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Tue Feb 8 15:58:00 2022 +0100 [KERNEL32_APITESTS] Add a test showing thread locale/LangID sync with console (#4301) This test shows how a console program's thread's language ID changes, or not, when changing the console output code page (or indirectly when starting on/connecting to a new console), depending on whether or not the console is running on a CJK/on a non-CJK system, and the selected code page is a CJK/a non-CJK one. By that, the test indirectly tests the (non-exposed) consoles GetLangId + SetThreadLocale() calls being made under the hood. --- modules/rostests/apitests/kernel32/ConsoleCP.c | 215 +++++++++++++++++++++---- 1 file changed, 184 insertions(+), 31 deletions(-) diff --git a/modules/rostests/apitests/kernel32/ConsoleCP.c b/modules/rostests/apitests/kernel32/ConsoleCP.c index 6a997681ac5..31fc45767f3 100644 --- a/modules/rostests/apitests/kernel32/ConsoleCP.c +++ b/modules/rostests/apitests/kernel32/ConsoleCP.c @@ -1,9 +1,9 @@ /* * PROJECT: ReactOS api tests - * LICENSE: GPL-2.0+ (https://spdx.org/licenses/GPL-2.0+) + * LICENSE: GPL-2.0-or-later (https://spdx.org/licenses/GPL-2.0-or-later) * PURPOSE: Tests for i18n console. * COPYRIGHT: Copyright 2017-2020 Katayama Hirofumi MZ - * Copyright 2020 Hermes Belusca-Maito + * Copyright 2020-2022 Hermès Bélusca-Maïto */ #include "precomp.h" @@ -24,30 +24,189 @@ static const WCHAR u9580[] = {0x9580, 0}; /* 門 */ static const WCHAR space[] = {L' ', 0}; static const WCHAR ideograph_space = (WCHAR)0x3000; /* fullwidth space */ static const WCHAR s_str[] = {L'A', 0x9580, 'B', 0}; -static LCID lcidJapanese = MAKELCID(MAKELANGID(LANG_JAPANESE, SUBLANG_DEFAULT), SORT_DEFAULT); -static LCID lcidRussian = MAKELCID(MAKELANGID(LANG_RUSSIAN , SUBLANG_DEFAULT), SORT_DEFAULT); +static const LCID lcidJapanese = MAKELCID(MAKELANGID(LANG_JAPANESE, SUBLANG_DEFAULT), SORT_DEFAULT); +static const LCID lcidRussian = MAKELCID(MAKELANGID(LANG_RUSSIAN , SUBLANG_DEFAULT), SORT_DEFAULT); + +static UINT s_uOEMCP; static BOOL s_bIs8Plus; -static BOOL IsCJKCodePage(void) +static BOOL IsCJKCodePage(_In_ UINT CodePage) { - switch (GetOEMCP()) + switch (CodePage) { - case 936: // Chinese PRC case 932: // Japanese case 949: // Korean case 1361: // Korean (Johab) + case 936: // Chinese PRC case 950: // Taiwan return TRUE; } return FALSE; } +static __inline +LANGID MapCJKCPToLangId(_In_ UINT CodePage) +{ + switch (CodePage) + { + case 932: // Japanese (Shift-JIS) + return MAKELANGID(LANG_JAPANESE, SUBLANG_DEFAULT); + case 949: // Korean (Hangul/Wansung) + // case 1361: // Korean (Johab) + return MAKELANGID(LANG_KOREAN, SUBLANG_KOREAN); + case 936: // Chinese PRC (Chinese Simplified) + return MAKELANGID(LANG_CHINESE, SUBLANG_CHINESE_SIMPLIFIED); + case 950: // Taiwan (Chinese Traditional) + return MAKELANGID(LANG_CHINESE, SUBLANG_CHINESE_TRADITIONAL); + default: + return MAKELANGID(LANG_ENGLISH, SUBLANG_ENGLISH_US); + } +} + +static BOOL ChangeOutputCP_( + _In_ const char* file, + _In_ int line, + _In_ UINT CodePage) +{ + BOOL bSuccess; + + /* Validate the code page */ + bSuccess = IsValidCodePage(CodePage); + if (!bSuccess) + { + skip_(file, line)("Code page %d not available\n", CodePage); + return FALSE; + } + + /* Set the new code page */ + SetLastError(0xdeadbeef); + bSuccess = SetConsoleOutputCP(CodePage); + if (!bSuccess) + skip_(file, line)("SetConsoleOutputCP(%d) failed with last error %lu\n", CodePage, GetLastError()); + return bSuccess; +} + +#define ChangeOutputCP(CodePage) \ + ChangeOutputCP_(__FILE__, __LINE__, CodePage) + + +#define cmpThreadLangId(file, line, ExpectedLangId) \ +do { \ + LANGID ThreadLangId; \ + ThreadLangId = LANGIDFROMLCID(NtCurrentTeb()->CurrentLocale); \ + trace_((file), (line))("Thread LangId %d, expecting %d...\n", \ + ThreadLangId, (ExpectedLangId)); \ + ok_((file), (line))(ThreadLangId == (ExpectedLangId), \ + "Thread LangId %d, expected %d\n", \ + ThreadLangId, (ExpectedLangId)); \ +} while (0) + +static BOOL +doTest_CP_ThreadLang_( + _In_ const char* file, + _In_ int line, + _In_ UINT CodePage, + _In_ LANGID ExpectedLangId) +{ + UINT newcp; + + /* Verify and set the new code page */ + if (!ChangeOutputCP_(file, line, CodePage)) + { + skip_(file, line)("Code page %d expected to be valid!\n", CodePage); + return FALSE; + } + + newcp = GetConsoleOutputCP(); + ok_(file, line)(newcp == CodePage, "Console output CP is %d, expected %d\n", newcp, CodePage); + + /* Verify that the thread lang ID is the expected one */ + cmpThreadLangId(file, line, ExpectedLangId); + return TRUE; +} + +#define doTest_CP_ThreadLang(...) \ + doTest_CP_ThreadLang_(__FILE__, __LINE__, ##__VA_ARGS__) + +static VOID test_CP_ThreadLang(VOID) +{ + /* Save the initial current thread locale. It is (re)initialized after + * attaching to a console. Don't use GetThreadLocale() as the latter + * can return a replacement value in case CurrentLocale is 0. */ + LANGID ThreadLangId = LANGIDFROMLCID(NtCurrentTeb()->CurrentLocale); + UINT oldcp = GetConsoleOutputCP(); + + if (IsCJKCodePage(s_uOEMCP)) + { + /* We are on a CJK system */ + + /* Is the console in CJK? If so, current thread should be in CJK language */ + if (!IsCJKCodePage(oldcp)) + { + skip("CJK system but console CP not in CJK\n"); + } + else + { + /* Check that the thread lang ID matches what the console set */ + LANGID LangId = MapCJKCPToLangId(oldcp); + cmpThreadLangId(__FILE__, __LINE__, LangId); + } + + /* Set the code page to OEM USA (non-CJK codepage that is supported). + * Verify that the thread lang ID has changed to non-CJK language. */ + doTest_CP_ThreadLang(437, MapCJKCPToLangId(437)); + + /* Set the code page to the default system CJK codepage. + * Check that the thread lang ID matches what the console set. */ + doTest_CP_ThreadLang(s_uOEMCP, MapCJKCPToLangId(s_uOEMCP)); + } + else + { + /* We are on a non-CJK system */ + + /* Code pages: Japanese, Korean, Chinese Simplified/Traditional */ + UINT CJKCodePages[] = {932, 949, 936, 950}; + UINT newcp; + USHORT i; + + /* Switch to a different code page (OEM USA) than the current one. + * In such setup, the current thread lang ID should not change. */ + newcp = (s_uOEMCP == 437 ? 850 : 437); + doTest_CP_ThreadLang(newcp, ThreadLangId); + + /* Try switching to a CJK codepage, if possible, but + * the thread lang ID should not change either... */ + + /* Retry as long as no valid CJK codepage has been found */ + for (i = 0; i < ARRAYSIZE(CJKCodePages); ++i) + { + newcp = CJKCodePages[i]; + if (IsValidCodePage(newcp)) + break; // Found a valid one. + } + if (i >= ARRAYSIZE(CJKCodePages)) + { + /* No valid CJK code pages on the system */ + skip("CJK system but console CP not in CJK\n"); + } + else + { + /* Verify that the thread lang ID remains the same */ + doTest_CP_ThreadLang(newcp, ThreadLangId); + } + } + + /* Restore code page */ + SetConsoleOutputCP(oldcp); +} + + /* Russian Code Page 855 */ // NOTE that CP 866 can also be used static void test_cp855(HANDLE hConOut) { BOOL ret; - DWORD oldcp; + UINT oldcp; int n; DWORD len; COORD c; @@ -56,19 +215,11 @@ static void test_cp855(HANDLE hConOut) WCHAR str[32]; WORD attrs[16]; - if (!IsValidCodePage(855)) - { - skip("Codepage 855 not available\n"); - return; - } - /* Set code page */ oldcp = GetConsoleOutputCP(); - SetLastError(0xdeadbeef); - ret = SetConsoleOutputCP(855); - if (!ret) + if (!ChangeOutputCP(855)) { - skip("SetConsoleOutputCP failed with last error %lu\n", GetLastError()); + skip("Codepage 855 not available\n"); return; } @@ -269,7 +420,7 @@ static void test_cp855(HANDLE hConOut) static void test_cp932(HANDLE hConOut) { BOOL ret; - DWORD oldcp; + UINT oldcp; int n; DWORD len; COORD c, buffSize; @@ -280,19 +431,11 @@ static void test_cp932(HANDLE hConOut) CHAR_INFO buff[16]; SMALL_RECT sr; - if (!IsValidCodePage(932)) - { - skip("Codepage 932 not available\n"); - return; - } - /* Set code page */ oldcp = GetConsoleOutputCP(); - SetLastError(0xdeadbeef); - ret = SetConsoleOutputCP(932); - if (!ret) + if (!ChangeOutputCP(932)) { - skip("SetConsoleOutputCP failed with last error %lu\n", GetLastError()); + skip("Codepage 932 not available\n"); return; } @@ -1364,6 +1507,7 @@ static void test_cp932(HANDLE hConOut) SetConsoleOutputCP(oldcp); } + START_TEST(ConsoleCP) { HANDLE hConIn, hConOut; @@ -1382,9 +1526,18 @@ START_TEST(ConsoleCP) ok(hConIn != INVALID_HANDLE_VALUE, "Opening ConIn\n"); ok(hConOut != INVALID_HANDLE_VALUE, "Opening ConOut\n"); + /* Retrieve the system OEM code page */ + s_uOEMCP = GetOEMCP(); + trace("Running on %s system (codepage %d)\n", + IsCJKCodePage(s_uOEMCP) ? "CJK" : "Non-CJK", + s_uOEMCP); + + /* Test thread lang ID syncing with console code page */ + test_CP_ThreadLang(); + if (IsValidLocale(lcidRussian, LCID_INSTALLED)) { - if (!IsCJKCodePage()) + if (!IsCJKCodePage(s_uOEMCP)) test_cp855(hConOut); else skip("Russian testcase is skipped because of CJK\n"); @@ -1396,7 +1549,7 @@ START_TEST(ConsoleCP) if (IsValidLocale(lcidJapanese, LCID_INSTALLED)) { - if (IsCJKCodePage()) + if (IsCJKCodePage(s_uOEMCP)) test_cp932(hConOut); else skip("Japanese testcase is skipped because of not CJK\n");

1 0

[reactos] 02/02: [NTOS:EX] ExpGet/SetCurrentUserUILanguage() take pointer to const string. Add an implementation note to NtQueryDefaultUILanguage().
by Hermès Bélusca-Maïto 08 Feb '22

08 Feb '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=20e23bbfcdcc1bb4efc3b… commit 20e23bbfcdcc1bb4efc3b0a92bbd3053c5eb9c6f Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Fri Jan 21 17:09:53 2022 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Tue Feb 8 15:56:39 2022 +0100 [NTOS:EX] ExpGet/SetCurrentUserUILanguage() take pointer to const string. Add an implementation note to NtQueryDefaultUILanguage(). --- ntoskrnl/ex/locale.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/ntoskrnl/ex/locale.c b/ntoskrnl/ex/locale.c index 9e0f57aa458..b8abb886031 100644 --- a/ntoskrnl/ex/locale.c +++ b/ntoskrnl/ex/locale.c @@ -21,14 +21,14 @@ LCID PsDefaultSystemLocaleId = 0x00000409; LANGID PsInstallUILanguageId = LANGIDFROMLCID(0x00000409); /* UI/Thread IDs: Same as system */ -LANGID PsDefaultUILanguageId = 0x00000409; -LCID PsDefaultThreadLocaleId = LANGIDFROMLCID(0x00000409); +LCID PsDefaultThreadLocaleId = 0x00000409; +LANGID PsDefaultUILanguageId = LANGIDFROMLCID(0x00000409); /* PRIVATE FUNCTIONS *********************************************************/ NTSTATUS NTAPI -ExpGetCurrentUserUILanguage(IN PWSTR MuiName, +ExpGetCurrentUserUILanguage(IN PCWSTR MuiName, OUT LANGID* LanguageId) { UCHAR ValueBuffer[256]; @@ -101,7 +101,7 @@ ExpGetCurrentUserUILanguage(IN PWSTR MuiName, NTSTATUS NTAPI -ExpSetCurrentUserUILanguage(IN PWSTR MuiName, +ExpSetCurrentUserUILanguage(IN PCWSTR MuiName, IN LANGID LanguageId) { OBJECT_ATTRIBUTES ObjectAttributes; @@ -426,6 +426,7 @@ NtQueryDefaultUILanguage(OUT LANGID* LanguageId) else { /* Failed, use fallback value */ + // NOTE: Windows doesn't use PsDefaultUILanguageId. *LanguageId = PsInstallUILanguageId; } }

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