[reactos] 01/01: [CONSRV] Introduce a CON_SET_OUTPUT_CP() macro to simplify setting the console's output CP + associated changes. Simplify ConDrvSetConsoleCP() as well.

[Hermès Bélusca-Maïto]

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=f79c02bd8a9797ab180788...

commit f79c02bd8a9797ab1807884862928ffb817d2d54
Author:     Hermès Bélusca-Maïto [e-mail]
AuthorDate: Thu Jan 27 21:36:16 2022 +0100
Commit:     Hermès Bélusca-Maïto [e-mail]
CommitDate: Fri Feb 4 21:35:15 2022 +0100

[CONSRV] Introduce a CON_SET_OUTPUT_CP() macro to simplify setting the console's output CP + associated changes. Simplify ConDrvSetConsoleCP() as well.
---
 win32ss/user/winsrv/consrv/condrv/console.c | 24 +++++++++++-------------
 win32ss/user/winsrv/consrv/include/conio.h  |  6 ++++++
 win32ss/user/winsrv/consrv/settings.c       |  4 ++--
 3 files changed, 19 insertions(+), 15 deletions(-)

diff --git a/win32ss/user/winsrv/consrv/condrv/console.c b/win32ss/user/winsrv/consrv/condrv/console.c
index fae7beb7de3..74d2229b4db 100644
--- a/win32ss/user/winsrv/consrv/condrv/console.c
+++ b/win32ss/user/winsrv/consrv/condrv/console.c
@@ -116,9 +116,10 @@ ConDrvInitConsole(

/* Set-up the code page */
     if (IsValidCodePage(ConsoleInfo->CodePage))
-        Console->InputCodePage = Console->OutputCodePage = ConsoleInfo->CodePage;
-
-    Console->IsCJK = IsCJKCodePage(Console->OutputCodePage);
+    {
+        CON_SET_OUTPUT_CP(Console, ConsoleInfo->CodePage);
+        Console->InputCodePage = ConsoleInfo->CodePage;
+    }

/* Initialize a new text-mode screen buffer with default settings */
     ScreenBufferInfo.ScreenBufferSize = ConsoleInfo->ScreenBufferSize;
@@ -424,27 +425,24 @@ ConDrvSetConsoleCP(IN PCONSOLE Console,
                    IN UINT CodePage,
                    IN BOOLEAN OutputCP)
 {
-    BOOL Success = TRUE;
-
     if (Console == NULL || !IsValidCodePage(CodePage))
         return STATUS_INVALID_PARAMETER;

if (OutputCP)
     {
         /* Request the terminal to change its code page support */
-        Success = TermSetCodePage(Console, CodePage);
-        if (Success)
-        {
-            Console->OutputCodePage = CodePage;
-            Console->IsCJK = IsCJKCodePage(CodePage);
-        }
+        if (!TermSetCodePage(Console, CodePage))
+            return STATUS_UNSUCCESSFUL;
+
+        /* All is fine, actually set the output code page */
+        CON_SET_OUTPUT_CP(Console, CodePage);
+        return STATUS_SUCCESS;
     }
     else
     {
         Console->InputCodePage = CodePage;
+        return STATUS_SUCCESS;
     }
-
-    return (Success ? STATUS_SUCCESS : STATUS_UNSUCCESSFUL);
 }

/* EOF */
diff --git a/win32ss/user/winsrv/consrv/include/conio.h b/win32ss/user/winsrv/consrv/include/conio.h
index ead041e51df..0c8ae92631c 100644
--- a/win32ss/user/winsrv/consrv/include/conio.h
+++ b/win32ss/user/winsrv/consrv/include/conio.h
@@ -320,6 +320,12 @@ ConDrvUnpause(PCONSOLE Console);
 #define GetConsoleInputBufferMode(Console)  \
     (Console)->InputBuffer.Mode

+#define CON_SET_OUTPUT_CP(Console, CodePage)\
+do { \
+    (Console)->OutputCodePage = (CodePage); \
+    (Console)->IsCJK = IsCJKCodePage((Console)->OutputCodePage); \
+} while (0)
+

/* conoutput.c */
 PCHAR_INFO ConioCoordToPointer(PTEXTMODE_SCREEN_BUFFER Buff, ULONG X, ULONG Y);
diff --git a/win32ss/user/winsrv/consrv/settings.c b/win32ss/user/winsrv/consrv/settings.c
index 22cccc2d81c..4594d6e8e6d 100644
--- a/win32ss/user/winsrv/consrv/settings.c
+++ b/win32ss/user/winsrv/consrv/settings.c
@@ -68,8 +68,8 @@ ConSrvApplyUserSettings(
         // ConDrvSetConsoleCP(Console, ConsoleInfo->CodePage, FALSE);   // Input
         if (TermSetCodePage(Console, ConsoleInfo->CodePage))
         {
-            Console->InputCodePage = Console->OutputCodePage = ConsoleInfo->CodePage;
-            Console->IsCJK = IsCJKCodePage(Console->OutputCodePage);
+            CON_SET_OUTPUT_CP(Console, ConsoleInfo->CodePage);
+            Console->InputCodePage = ConsoleInfo->CodePage;
         }
     }