Ros-diffs

ros-diffs@reactos.org

81390 discussions

[reactos] 02/04: [SMSS][NTOS:MM] Get rid of the remaining sprintf_nt hacks. (#4799)
by Hermès Bélusca-Maïto 25 Oct '22

25 Oct '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=0997e9023c2242974026c… commit 0997e9023c2242974026cc397805318b96425714 Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Fri Oct 21 19:33:57 2022 +0200 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Wed Oct 26 00:41:23 2022 +0200 [SMSS][NTOS:MM] Get rid of the remaining sprintf_nt hacks. (#4799) --- base/system/smss/smss.c | 19 ++++--------------- ntoskrnl/mm/ARM3/sysldr.c | 36 ++++++++++++++---------------------- 2 files changed, 18 insertions(+), 37 deletions(-) diff --git a/base/system/smss/smss.c b/base/system/smss/smss.c index 6e9fb6081c0..27b8306b87e 100644 --- a/base/system/smss/smss.c +++ b/base/system/smss/smss.c @@ -25,18 +25,6 @@ HANDLE SmpInitialCommandProcessId; /* FUNCTIONS ******************************************************************/ -/* GCC's incompetence strikes again */ -VOID -sprintf_nt(IN PCHAR Buffer, - IN PCHAR Format, - IN ...) -{ - va_list ap; - va_start(ap, Format); - sprintf(Buffer, Format, ap); - va_end(ap); -} - NTSTATUS NTAPI SmpExecuteImage(IN PUNICODE_STRING FileName, @@ -176,9 +164,10 @@ SmpInvokeAutoChk(IN PUNICODE_STRING FileName, if (Flags & SMP_INVALID_PATH) { /* It wasn't, so create an error message to print on the screen */ - sprintf_nt(MessageBuffer, - "%wZ program not found - skipping AUTOCHECK\r\n", - FileName); + RtlStringCbPrintfA(MessageBuffer, + sizeof(MessageBuffer), + "%wZ program not found - skipping AUTOCHECK\r\n", + FileName); RtlInitAnsiString(&MessageString, MessageBuffer); if (NT_SUCCESS(RtlAnsiStringToUnicodeString(&Destination, &MessageString, diff --git a/ntoskrnl/mm/ARM3/sysldr.c b/ntoskrnl/mm/ARM3/sysldr.c index 31181b69fc9..35b538259f3 100644 --- a/ntoskrnl/mm/ARM3/sysldr.c +++ b/ntoskrnl/mm/ARM3/sysldr.c @@ -16,19 +16,6 @@ #define MODULE_INVOLVED_IN_ARM3 #include <mm/ARM3/miarm.h> -static -inline -VOID -sprintf_nt(IN PCHAR Buffer, - IN PCHAR Format, - IN ...) -{ - va_list ap; - va_start(ap, Format); - vsprintf(Buffer, Format, ap); - va_end(ap); -} - /* GLOBALS ********************************************************************/ LIST_ENTRY PsLoadedModuleList; @@ -2908,7 +2895,7 @@ MmLoadSystemImage(IN PUNICODE_STRING FileName, OBJECT_ATTRIBUTES ObjectAttributes; IO_STATUS_BLOCK IoStatusBlock; PIMAGE_NT_HEADERS NtHeader; - UNICODE_STRING BaseName, BaseDirectory, PrefixName, UnicodeTemp; + UNICODE_STRING BaseName, BaseDirectory, PrefixName; PLDR_DATA_TABLE_ENTRY LdrEntry = NULL; ULONG EntrySize, DriverSize; PLOAD_IMPORTS LoadedImports = MM_SYSLDR_NO_IMPORTS; @@ -2920,7 +2907,7 @@ MmLoadSystemImage(IN PUNICODE_STRING FileName, BOOLEAN LockOwned = FALSE; PLIST_ENTRY NextEntry; IMAGE_INFO ImageInfo; - STRING AnsiTemp; + PAGED_CODE(); /* Detect session-load */ @@ -3364,6 +3351,9 @@ LoaderScan: if (MiCacheImageSymbols(LdrEntry->DllBase)) #endif { + UNICODE_STRING UnicodeTemp; + STRING AnsiTemp; + /* Check if the system root is present */ if ((PrefixName.Length > (11 * sizeof(WCHAR))) && !(_wcsnicmp(PrefixName.Buffer, L"\\SystemRoot", 11))) @@ -3372,18 +3362,20 @@ LoaderScan: UnicodeTemp = PrefixName; UnicodeTemp.Buffer += 11; UnicodeTemp.Length -= (11 * sizeof(WCHAR)); - sprintf_nt(Buffer, - "%ws%wZ", - &SharedUserData->NtSystemRoot[2], - &UnicodeTemp); + RtlStringCbPrintfA(Buffer, + MAXIMUM_FILENAME_LENGTH, + "%ws%wZ", + &SharedUserData->NtSystemRoot[2], + &UnicodeTemp); } else { /* Build the name */ - sprintf_nt(Buffer, "%wZ", &BaseName); + RtlStringCbPrintfA(Buffer, MAXIMUM_FILENAME_LENGTH, + "%wZ", &BaseName); } - /* Setup the ansi string */ + /* Setup the ANSI string */ RtlInitString(&AnsiTemp, Buffer); /* Notify the debugger */ @@ -3523,7 +3515,7 @@ MmGetSystemRoutineAddress(IN PUNICODE_STRING SystemRoutineName) UNICODE_STRING HalName = RTL_CONSTANT_STRING(L"hal.dll"); ULONG Modules = 0; - /* Convert routine to ansi name */ + /* Convert routine to ANSI name */ Status = RtlUnicodeStringToAnsiString(&AnsiRoutineName, SystemRoutineName, TRUE);

1 0

[reactos] 01/04: [CSRSRV] Remove deprecated dependency on smdll. (#4799)
by Hermès Bélusca-Maïto 25 Oct '22

25 Oct '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=426598c6f71d88c3774de… commit 426598c6f71d88c3774de4f8ab62c7e968cf4981 Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Fri Oct 21 16:13:22 2022 +0200 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Wed Oct 26 00:41:13 2022 +0200 [CSRSRV] Remove deprecated dependency on smdll. (#4799) --- subsystems/win32/csrsrv/CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/subsystems/win32/csrsrv/CMakeLists.txt b/subsystems/win32/csrsrv/CMakeLists.txt index fc7dae1bb40..3ee512e5dd9 100644 --- a/subsystems/win32/csrsrv/CMakeLists.txt +++ b/subsystems/win32/csrsrv/CMakeLists.txt @@ -21,7 +21,7 @@ add_library(csrsrv MODULE set_module_type(csrsrv nativedll) target_link_libraries(csrsrv ${PSEH_LIB} smlib) -add_importlibs(csrsrv smdll ntdll) +add_importlibs(csrsrv ntdll) add_pch(csrsrv srv.h SOURCE) add_dependencies(csrsrv psdk bugcodes) add_cd_file(TARGET csrsrv DESTINATION reactos/system32 FOR all)

1 0

[reactos] 03/03: [NTUSER] Fix issue when cursor/caret is static
by Jose Carlos Jesus 25 Oct '22

25 Oct '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=9701bbcf6d6a4bbcdd6d0… commit 9701bbcf6d6a4bbcdd6d0cdbe9b639dfa6cadb43 Author: Jose Carlos Jesus <zecarlos1957(a)hotmail.com> AuthorDate: Fri Jan 14 19:32:52 2022 +0000 Commit: Stanislav Motylkov <x86corez(a)gmail.com> CommitDate: Tue Oct 25 23:54:31 2022 +0300 [NTUSER] Fix issue when cursor/caret is static Also remove the check that Windows doesn't do. CORE-17929 --- win32ss/user/ntuser/caret.c | 24 ++++++++++-------------- 1 file changed, 10 insertions(+), 14 deletions(-) diff --git a/win32ss/user/ntuser/caret.c b/win32ss/user/ntuser/caret.c index 24fadf6e16e..52ebb1bdb22 100644 --- a/win32ss/user/ntuser/caret.c +++ b/win32ss/user/ntuser/caret.c @@ -10,11 +10,6 @@ #include <win32k.h> DBG_DEFAULT_CHANNEL(UserCaret); -/* DEFINES *****************************************************************/ - -#define MIN_CARETBLINKRATE 100 -#define MAX_CARETBLINKRATE 10000 - /* FUNCTIONS *****************************************************************/ VOID FASTCALL @@ -189,13 +184,6 @@ IntSetCaretBlinkTime(UINT uMSeconds) { /* Don't save the new value to the registry! */ - /* Windows doesn't do this check */ - if((uMSeconds < MIN_CARETBLINKRATE) || (uMSeconds > MAX_CARETBLINKRATE)) - { - EngSetLastError(ERROR_INVALID_PARAMETER); - return FALSE; - } - gpsi->dtCaretBlink = uMSeconds; return TRUE; @@ -300,7 +288,15 @@ BOOL FASTCALL co_UserShowCaret(PWND Window OPTIONAL) { IntNotifyWinEvent(EVENT_OBJECT_SHOW, pWnd, OBJID_CARET, OBJID_CARET, 0); } - IntSetTimer(pWnd, IDCARETTIMER, gpsi->dtCaretBlink, CaretSystemTimerProc, TMRF_SYSTEM); + if ((INT)gpsi->dtCaretBlink > 0) + { + IntSetTimer(pWnd, IDCARETTIMER, gpsi->dtCaretBlink, CaretSystemTimerProc, TMRF_SYSTEM); + } + else if (ThreadQueue->CaretInfo.Visible) + { + ThreadQueue->CaretInfo.Showing = 1; + co_IntDrawCaret(pWnd, &ThreadQueue->CaretInfo); + } } return TRUE; } @@ -366,7 +362,7 @@ NtUserCreateCaret( ThreadQueue->CaretInfo.Visible = 0; ThreadQueue->CaretInfo.Showing = 0; - IntSetTimer( Window, IDCARETTIMER, gpsi->dtCaretBlink, CaretSystemTimerProc, TMRF_SYSTEM ); + IntSetTimer(Window, IDCARETTIMER, gpsi->dtCaretBlink, CaretSystemTimerProc, TMRF_SYSTEM); IntNotifyWinEvent(EVENT_OBJECT_CREATE, Window, OBJID_CARET, CHILDID_SELF, 0);

1 0

[reactos] 02/03: [ACCESS][MAIN] Fix setting cursor/caret blinking rate
by Jose Carlos Jesus 25 Oct '22

25 Oct '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=c0eb40713051eab0c38c6… commit c0eb40713051eab0c38c6663c9d762e0505ac863 Author: Jose Carlos Jesus <zecarlos1957(a)hotmail.com> AuthorDate: Fri Dec 31 12:27:42 2021 +0000 Commit: Stanislav Motylkov <x86corez(a)gmail.com> CommitDate: Tue Oct 25 23:54:28 2022 +0300 [ACCESS][MAIN] Fix setting cursor/caret blinking rate - Save uCaretBlinkTime to CursorBlinkRate registry key - Also fix blinking rate never reaching None CORE-17929 --- dll/cpl/access/display.c | 52 ++++++++++++++++++++++++++++++++++++++--- dll/cpl/main/keyboard.c | 61 +++++++++++++++++++++++++++++++++++++++++++++--- 2 files changed, 107 insertions(+), 6 deletions(-) diff --git a/dll/cpl/access/display.c b/dll/cpl/access/display.c index 843e4a1024e..03e1959b6ea 100644 --- a/dll/cpl/access/display.c +++ b/dll/cpl/access/display.c @@ -13,6 +13,31 @@ #define ID_BLINK_TIMER 346 +static VOID +UpdateCaretBlinkTimeReg( + _In_ UINT uCaretBlinkTime) +{ + HKEY hKey; + WCHAR szBuffer[12]; + + if (RegOpenKeyExW(HKEY_CURRENT_USER, + L"Control Panel\\Desktop", + 0, KEY_SET_VALUE, + &hKey) != ERROR_SUCCESS) + { + return; + } + + wsprintf(szBuffer, L"%d", uCaretBlinkTime); + + RegSetValueExW(hKey, L"CursorBlinkRate", + 0, REG_SZ, + (CONST BYTE*)szBuffer, + (wcslen(szBuffer) + 1) * sizeof(WCHAR)); + + RegCloseKey(hKey); +} + static VOID FillColorSchemeComboBox(HWND hwnd) { @@ -164,7 +189,16 @@ DisplayPageProc(HWND hwndDlg, SendDlgItemMessage(hwndDlg, IDC_CURSOR_WIDTH_TRACK, TBM_SETPOS, (WPARAM)TRUE, (LPARAM)(pGlobalData->uCaretWidth - 1)); /* Start the blink timer */ - SetTimer(hwndDlg, ID_BLINK_TIMER, pGlobalData->uCaretBlinkTime, NULL); + pGlobalData->uCaretBlinkTime = pGlobalData->uCaretBlinkTime >= 1200 ? -1 : pGlobalData->uCaretBlinkTime; + if ((INT)pGlobalData->uCaretBlinkTime > 0) + { + SetTimer(hwndDlg, ID_BLINK_TIMER, pGlobalData->uCaretBlinkTime, NULL); + } + else + { + PostMessage(hwndDlg, WM_TIMER, ID_BLINK_TIMER, 0); + } + return TRUE; case WM_COMMAND: @@ -196,7 +230,15 @@ DisplayPageProc(HWND hwndDlg, i = SendDlgItemMessage(hwndDlg, IDC_CURSOR_BLINK_TRACK, TBM_GETPOS, 0, 0); pGlobalData->uCaretBlinkTime = (12 - (UINT)i) * 100; KillTimer(hwndDlg, ID_BLINK_TIMER); - SetTimer(hwndDlg, ID_BLINK_TIMER, pGlobalData->uCaretBlinkTime, NULL); + pGlobalData->uCaretBlinkTime = pGlobalData->uCaretBlinkTime >= 1200 ? -1 : pGlobalData->uCaretBlinkTime; + if ((INT)pGlobalData->uCaretBlinkTime > 0) + { + SetTimer(hwndDlg, ID_BLINK_TIMER, pGlobalData->uCaretBlinkTime, NULL); + } + else if (pGlobalData->fShowCaret) + { + SendMessage(hwndDlg, WM_TIMER, ID_BLINK_TIMER, 0); + } PropSheet_Changed(GetParent(hwndDlg), hwndDlg); break; @@ -243,7 +285,11 @@ DisplayPageProc(HWND hwndDlg, lppsn = (LPPSHNOTIFY)lParam; if (lppsn->hdr.code == PSN_APPLY) { - SetCaretBlinkTime(pGlobalData->uCaretBlinkTime); + if (SetCaretBlinkTime(pGlobalData->uCaretBlinkTime)) + { + UpdateCaretBlinkTimeReg(pGlobalData->uCaretBlinkTime); + } + SystemParametersInfo(SPI_SETCARETWIDTH, 0, IntToPtr(pGlobalData->uCaretWidth), diff --git a/dll/cpl/main/keyboard.c b/dll/cpl/main/keyboard.c index c3f7b1ce69c..59af3bb8cb0 100644 --- a/dll/cpl/main/keyboard.c +++ b/dll/cpl/main/keyboard.c @@ -39,6 +39,31 @@ typedef struct _SPEED_DATA RECT rcCursor; } SPEED_DATA, *PSPEED_DATA; +static VOID +UpdateCaretBlinkTimeReg( + _In_ UINT uCaretBlinkTime) +{ + HKEY hKey; + WCHAR szBuffer[12]; + + if (RegOpenKeyExW(HKEY_CURRENT_USER, + L"Control Panel\\Desktop", + 0, KEY_SET_VALUE, + &hKey) != ERROR_SUCCESS) + { + return; + } + + wsprintf(szBuffer, L"%d", uCaretBlinkTime); + + RegSetValueExW(hKey, L"CursorBlinkRate", + 0, REG_SZ, + (CONST BYTE*)szBuffer, + (wcslen(szBuffer) + 1) * sizeof(WCHAR)); + + RegCloseKey(hKey); +} + /* Property page dialog callback */ static INT_PTR CALLBACK KeyboardSpeedProc(IN HWND hwndDlg, @@ -97,7 +122,16 @@ KeyboardSpeedProc(IN HWND hwndDlg, SendDlgItemMessage(hwndDlg, IDC_SLIDER_CURSOR_BLINK, TBM_SETPOS, (WPARAM)TRUE, (LPARAM)(12 - (pSpeedData->uCaretBlinkTime / 100))); /* Start the blink timer */ - SetTimer(hwndDlg, ID_BLINK_TIMER, pSpeedData->uCaretBlinkTime, NULL); + pSpeedData->uCaretBlinkTime = pSpeedData->uCaretBlinkTime >= 1200 ? -1 : pSpeedData->uCaretBlinkTime; + if ((INT)pSpeedData->uCaretBlinkTime > 0) + { + SetTimer(hwndDlg, ID_BLINK_TIMER, pSpeedData->uCaretBlinkTime, NULL); + } + else + { + PostMessage(hwndDlg, WM_TIMER, ID_BLINK_TIMER, 0); + } + break; case WM_HSCROLL: @@ -172,7 +206,15 @@ KeyboardSpeedProc(IN HWND hwndDlg, case TB_ENDTRACK: pSpeedData->uCaretBlinkTime = (12 - (UINT)SendDlgItemMessage(hwndDlg, IDC_SLIDER_CURSOR_BLINK, TBM_GETPOS, 0, 0)) * 100; KillTimer(hwndDlg, ID_BLINK_TIMER); - SetTimer(hwndDlg, ID_BLINK_TIMER, pSpeedData->uCaretBlinkTime, NULL); + pSpeedData->uCaretBlinkTime = pSpeedData->uCaretBlinkTime >= 1200 ? -1 : pSpeedData->uCaretBlinkTime; + if ((INT)pSpeedData->uCaretBlinkTime > 0) + { + SetTimer(hwndDlg, ID_BLINK_TIMER, pSpeedData->uCaretBlinkTime, NULL); + } + else if (pSpeedData->fShowCursor) + { + SendMessage(hwndDlg, WM_TIMER, ID_BLINK_TIMER, 0); + } SetCaretBlinkTime(pSpeedData->uCaretBlinkTime); PropSheet_Changed(GetParent(hwndDlg), hwndDlg); break; @@ -180,7 +222,15 @@ KeyboardSpeedProc(IN HWND hwndDlg, case TB_THUMBTRACK: pSpeedData->uCaretBlinkTime = (12 - (UINT)HIWORD(wParam)) * 100; KillTimer(hwndDlg, ID_BLINK_TIMER); - SetTimer(hwndDlg, ID_BLINK_TIMER, pSpeedData->uCaretBlinkTime, NULL); + pSpeedData->uCaretBlinkTime = pSpeedData->uCaretBlinkTime >= 1200 ? -1 : pSpeedData->uCaretBlinkTime; + if ((INT)pSpeedData->uCaretBlinkTime > 0) + { + SetTimer(hwndDlg, ID_BLINK_TIMER, pSpeedData->uCaretBlinkTime, NULL); + } + else if (pSpeedData->fShowCursor) + { + SendMessage(hwndDlg, WM_TIMER, ID_BLINK_TIMER, 0); + } SetCaretBlinkTime(pSpeedData->uCaretBlinkTime); PropSheet_Changed(GetParent(hwndDlg), hwndDlg); break; @@ -215,6 +265,11 @@ KeyboardSpeedProc(IN HWND hwndDlg, { case PSN_APPLY: /* Set the new keyboard settings */ + if (pSpeedData->uOrigCaretBlinkTime != pSpeedData->uCaretBlinkTime) + { + UpdateCaretBlinkTimeReg(pSpeedData->uCaretBlinkTime); + } + SystemParametersInfo(SPI_SETKEYBOARDDELAY, pSpeedData->nKeyboardDelay, 0,

1 0

[reactos] 01/03: [ACCESS][MAIN] Do not delete system stock object
by Jose Carlos Jesus 25 Oct '22

25 Oct '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=be014129a578b41adf7b8… commit be014129a578b41adf7b8fce5ee406cde3a4aa3e Author: Jose Carlos Jesus <zecarlos1957(a)hotmail.com> AuthorDate: Thu Dec 30 22:22:18 2021 +0000 Commit: Stanislav Motylkov <x86corez(a)gmail.com> CommitDate: Tue Oct 25 17:42:39 2022 +0300 [ACCESS][MAIN] Do not delete system stock object --- dll/cpl/access/display.c | 2 -- dll/cpl/main/keyboard.c | 1 - 2 files changed, 3 deletions(-) diff --git a/dll/cpl/access/display.c b/dll/cpl/access/display.c index 05e88f1e691..843e4a1024e 100644 --- a/dll/cpl/access/display.c +++ b/dll/cpl/access/display.c @@ -209,7 +209,6 @@ DisplayPageProc(HWND hwndDlg, HDC hDC = GetDC(hwndDlg); HBRUSH hBrush = GetSysColorBrush(COLOR_BTNTEXT); FillRect(hDC, &pGlobalData->rcCaret, hBrush); - DeleteObject(hBrush); ReleaseDC(hwndDlg, hDC); } else @@ -229,7 +228,6 @@ DisplayPageProc(HWND hwndDlg, HDC hDC = GetDC(hwndDlg); HBRUSH hBrush = GetSysColorBrush(COLOR_BTNTEXT); FillRect(hDC, &pGlobalData->rcCaret, hBrush); - DeleteObject(hBrush); ReleaseDC(hwndDlg, hDC); } else diff --git a/dll/cpl/main/keyboard.c b/dll/cpl/main/keyboard.c index 813c9996863..c3f7b1ce69c 100644 --- a/dll/cpl/main/keyboard.c +++ b/dll/cpl/main/keyboard.c @@ -196,7 +196,6 @@ KeyboardSpeedProc(IN HWND hwndDlg, HDC hDC = GetDC(hwndDlg); HBRUSH hBrush = GetSysColorBrush(COLOR_BTNTEXT); FillRect(hDC, &pSpeedData->rcCursor, hBrush); - DeleteObject(hBrush); ReleaseDC(hwndDlg, hDC); } else

1 0

[reactos] 01/01: [DISKPART] Update Polish translation for message table (#4548)
by Piotr Hetnarowicz 24 Oct '22

24 Oct '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=126afdaa6f0c03ca71b0d… commit 126afdaa6f0c03ca71b0dad0ce2698252661560b Author: Piotr Hetnarowicz <piotrhwz(a)gmail.com> AuthorDate: Mon Oct 24 16:08:35 2022 +0200 Commit: GitHub <noreply(a)github.com> CommitDate: Mon Oct 24 17:08:35 2022 +0300 [DISKPART] Update Polish translation for message table (#4548) Reviewed-by: Adam Stachowicz <saibamenppl(a)gmail.com> Reviewed-by: Stanislav Motylkov <x86corez(a)gmail.com> --- base/system/diskpart/diskpart_msg.mc | 734 ++++++++++++++++++----------------- 1 file changed, 383 insertions(+), 351 deletions(-) diff --git a/base/system/diskpart/diskpart_msg.mc b/base/system/diskpart/diskpart_msg.mc index 7c3dabd3761..5fe3d2deeac 100644 --- a/base/system/diskpart/diskpart_msg.mc +++ b/base/system/diskpart/diskpart_msg.mc @@ -77,26 +77,26 @@ Beispiel: ACTIVE . Language=Polish - On disks with master boot record (MBR) disk formatting, marks - the partition with focus as active. + Na dyskach zawierających formatowanie z głównym rekordem rozruchowym + (MBR) oznacza partycję, na której jest ustawiony fokus, jako aktywną. -Syntax: ACTIVE +Składnia: ACTIVE - Writes a value to the disk which is read by the basic input/output - system (BIOS) at boot. This value specifies that the partition is - a valid system partition. + Zapisuje na dysku wartość odczytywaną podczas rozruchu przez podstawowy + system wejścia/wyjścia (BIOS). Ta wartość oznacza, że dana partycja jest + prawidłową partycją systemową. - A partition must be selected for this operation to succeed. + Aby ta operacja powiodła się, partycja musi być wybrana. - Caution: + Uwaga: - DiskPart verifies only that the partition is capable of containing the - operating system startup files. DiskPart does not check the contents of - the partition. If you mistakenly mark a partition as active and it does - not contain the operating system startup files, your computer might not - start. + Program DiskPart sprawdza tylko, czy partycja może zawierać pliki + startowe systemu operacyjnego. Program DiskPart nie sprawdza zawartości + partycji. Jeśli przez pomyłkę jako aktywna zostanie oznaczona partycja + niezawierająca plików startowych systemu operacyjnego, komputer może się + nie uruchomić. -Example: +Przykład: ACTIVE . @@ -534,23 +534,23 @@ Beispiel: CLEAN . Language=Polish - Removes any and all partition or volume formatting from the disk with - focus. + Usuwa wszelkie formatowanie partycji lub woluminów z dysku mającego + fokus. -Syntax: CLEAN [ALL] +Składnia: CLEAN [ALL] - ALL Specifies that each and every byte\sector on the disk is set to - zero, which completely deletes all data contained on the disk. + ALL Określa, że każdy bajt/sektor na dysku zostaje ustawiony + na zero, co całkowicie usuwa wszystkie dane zawarte na dysku. - On master boot record (MBR) disks, only the MBR partitioning information - and hidden sector information are overwritten. On GUID partition table - (GPT) disks, the GPT partitioning information, including the Protective - MBR, is overwritten. If the ALL parameter is not used, the first 1MB - and the last 1MB of the disk are zeroed. This erases any disk formatting - that had been previously applied to the disk. The disk's state after - cleaning the disk is 'UNINITIALIZED'. + Na dyskach z głównym rekordem rozruchowym (MBR) są zastępowane tylko + informacje o partycjonowaniu MBR i ukrytych sektorach. Na dyskach + z tablicą partycji GUID (GPT) są zastępowane informacje + o partycjonowaniu GPT, łącznie z ochronnym rekordem MBR. Jeśli parametr + ALL nie zostanie użyty, 1 MB na początku i na końcu dysku jest zerowany. + Powoduje to wymazanie formatowania poprzednio stosowanego na dysku. + Stanem dysku po jego czyszczeniu jest stan 'NIEZAINICJOWANY'. -Example: +Przykład: CLEAN . @@ -895,39 +895,42 @@ Beispiel: CREATE PARTITION EXTENDED SIZE=1000 . Language=Polish - Creates an extended partition on the disk with focus. - Applies to master boot record (MBR) disks only. + Tworzy partycję rozszerzoną na dysku mającym fokus. + Dotyczy tylko dysków z głównym rekordem rozruchowym (MBR). -Syntax: CREATE PARTITION EXTENDED [SIZE=<N>] [OFFSET=<N>] [ALIGN=<N>] [NOERR] +Składnia: CREATE PARTITION EXTENDED [SIZE=<N>] [OFFSET=<N>] [ALIGN=<N>] [NOERR] - SIZE=<N> The size of the partition in megabytes (MB). If no size is - given, the partition continues until there is no more free - space in the extended partition. + SIZE=<N> Rozmiar partycji w megabajtach (MB). Jeśli nie podano + rozmiaru, partycja zajmuje całe wolne miejsce na partycji + rozszerzonej. - OFFSET=<N> The offset, in kilobytes (KB), at which the partition is - created. If no offset is given, the partition will start - at the beginning of the first free space on the disk that - is large enough to hold the new partition. + OFFSET=<N> Przesunięcie w kilobajtach (KB), przy którym zostanie + utworzona partycja. Jeśli nie podano przesunięcia, partycja + zostanie umieszczona na początku pierwszego wolnego miejsca + na dysku, w którym się zmieści. - ALIGN=<N> Typically used with hardware RAID Logical Unit Number (LUN) - arrays to improve performance. The partition offset will be - a multiple of <N>. If the OFFSET parameter is specified, it - will be rounded to the closest multiple of <N>. + ALIGN=<N> Zazwyczaj używany z numerami jednostek logicznych (LUN) + sprzętowych macierzy RAID w celu poprawienia wydajności. + Przesunięcie partycji będzie wielokrotnością liczby <N>. + Jeśli nie określono parametru OFFSET, jest ono zaokrąglane + do najbliższej wielokrotności liczby <N>. - NOERR For scripting only. When an error is encountered, DiskPart - continues to process commands as if the error did not occur. - Without the NOERR parameter, an error causes DiskPart to exit - with an error code. + NOERR Tylko do obsługi skryptów. Po wystąpieniu błędu program + DiskPart kontynuuje przetwarzanie poleceń tak, jakby błąd + nie wystąpił. W przypadku braku parametru NOERR błąd powoduje + zakończenie działania programu DiskPart i zwrócenie + kodu błędu. - After the partition has been created, the focus automatically shifts to the - new partition. Only one extended partition can be created per disk. This - command fails if you attempt to create an extended partition within another - extended partition. You must create an extended partition before you can - create logical partitions. + Po utworzeniu nowej partycji automatycznie otrzymuje ona fokus. Na każdym + dysku można utworzyć tylko jedną partycję rozszerzoną. To polecenie + zakończy się niepowodzeniem w przypadku próby utworzenia partycji + rozszerzonej w obrębie innej partycji rozszerzonej. Konieczne jest + utworzenie partycji rozszerzonej przed utworzeniem partycji logicznych. - A basic MBR disk must be selected for this operation to succeed. + Aby operacja została wykonana pomyślnie, musi być wybrany podstawowy + dysk MBR. -Example: +Przykład: CREATE PARTITION EXTENDED SIZE=1000 . @@ -1268,38 +1271,38 @@ Beispiel: CREATE PARTITION LOGICAL SIZE=1000 . Language=Polish - Creates a logical partition in an extended partition. - Applies to master boot record (MBR) disks only. + Tworzy partycję logiczną w obrębie partycji rozszerzonej. + Dotyczy tylko dysków z głównym rekordem rozruchowym (MBR). -Syntax: CREATE PARTITION LOGICAL [SIZE=<N>] [OFFSET=<N>] [ALIGN=<N>] [NOERR] +Składnia: CREATE PARTITION LOGICAL [SIZE=<N>] [OFFSET=<N>] [ALIGN=<N>] [NOERR] - SIZE=<N> The size of the partition in megabytes (MB). The partition is - at least as big in bytes as the number specified by N. If you - specify a size for the logical partition, it must be smaller - than the extended partition. If no size is given, the partition - continues until there is no more free space in the extended - partition. + SIZE=<N> Rozmiar partycji w megabajtach (MB). Partycja ma przynajmniej + taki rozmiar w bajtach, jak podana liczba N. + Określany rozmiar partycji logicznej musi być mniejszy od + rozmiaru partycji rozszerzonej. Jeśli nie podano rozmiaru, + partycja zajmuje całe wolne miejsce w bieżącym obszarze. - OFFSET=<N> The offset, in kilobytes (KB), at which the partition is created. - If no offset is given, the partition is placed in the first disk - extent that is large enough to hold it. + OFFSET=<N> Przesunięcie w kilobajtach (KB), przy którym zostanie utworzona + partycja. Jeśli nie podano przesunięcia, partycja zostanie + umieszczona w pierwszym zakresie dysku, w którym się zmieści. - ALIGN=<N> Typically used with hardware RAID Logical Unit Number (LUN) - arrays to improve performance. The partition offset will be - a multiple of <N>. If the OFFSET parameter is specified, it - will be rounded to the closest multiple of <N>. + ALIGN=<N> Zazwyczaj używany z numerami jednostek logicznych (LUN) + sprzętowych macierzy RAID w celu poprawienia wydajności. + Przesunięcie partycji będzie wielokrotnością liczby <N>. + Jeśli nie określono parametru OFFSET, jest ono zaokrąglane + do najbliższej wielokrotności liczby <N>. - NOERR For scripting only. When an error is encountered, DiskPart - continues to process commands as if the error did not occur. - Without the NOERR parameter, an error causes DiskPart to exit - with an error code. + NOERR Tylko do obsługi skryptów. Po wystąpieniu błędu program DiskPart + kontynuuje przetwarzanie poleceń tak, jakby błąd nie wystąpił. + W przypadku braku parametru NOERR błąd powoduje, zakończenie + działania programu DiskPart i zwrócenie kodu błędu. - After the partition has been created, the focus automatically shifts to the - new logical partition. + Po utworzeniu nowej partycji logicznej automatycznie otrzymuje ona fokus. - A basic MBR disk must be selected for this operation to succeed. + Aby operacja została wykonana pomyślnie, musi być wybrany podstawowy + dysk MBR. -Example: +Przykład: CREATE PARTITION LOGICAL SIZE=1000 . @@ -1814,105 +1817,113 @@ Beispiel: CREATE PARTITION PRIMARY SIZE=10000 ID=DE . Language=Polish - Creates a primary partition on the basic disk with focus. + Tworzy partycję podstawową na dysku podstawowym mającym fokus. -Syntax: CREATE PARTITION PRIMARY [SIZE=<N>] [OFFSET=<N>] [ID={<BYTE> | <GUID>}] +Składnia: CREATE PARTITION PRIMARY [SIZE=<N>] [OFFSET=<N>] [ID={<BYTE> | <GUID>}] [ALIGN=<N>] [NOERR] - SIZE=<N> The size of the partition in megabytes (MB). If no size is - given, the partition continues until there is no more - unallocated space in the current region. + SIZE=<N> Rozmiar partycji w megabajtach (MB). Jeśli nie podano + rozmiaru, partycja zajmuje całe nieprzydzielone miejsce w + bieżącym obszarze. - OFFSET=<N> The offset, in kilobytes (KB), at which the partition is created. - If no offset is given, the partition is placed in the first disk - extent that is large enough to hold it. + OFFSET=<N> Przesunięcie w kilobajtach, przy którym zostanie utworzona. + partycja. Jeśli nie podano przesunięcia, partycja zostanie + umieszczona w pierwszym zakresie dysku, w którym się zmieści. ID={<BYTE> | <GUID>} - Specifies the partition type. + Określa typ partycji. - Intended for Original Equipment Manufacturer (OEM) use only. + Parametr przeznaczony tylko do użytku przez producentów + oryginalnego sprzętu (OEM). - For master boot record (MBR) disks, you can specify a partition - type byte, in hexadecimal form, for the partition. If this - parameter is not specified for an MBR disk, the command creates - a partition of type 0x06 (specifies no file system is installed). + W przypadku dysków z głównym rekordem rozruchowym (MBR) można + dla partycji określić bajt typu partycji (szesnastkowo). Jeśli + ten parametr nie zostanie określony dla dysku MBR, polecenie + powoduje utworzenie partycji typu 0x06 (określa, że brak jest + zainstalowanego systemu plików). - LDM data partition: + Partycja danych LDM: 0x42 - Recovery partition: + Partycja odzyskiwania: 0x27 - Recognized OEM Ids: + Rozpoznawane identyfikatory OEM: 0x12 0x84 0xDE 0xFE 0xA0 - For GUID partition table (GPT) disks you can specify a - partition type GUID for the partition you want to create. - Recognized GUIDs include: + W przypadku dysków z tablicą partycji GUID (GPT) można + określić identyfikator GUID typu partycji, która ma zostać + utworzona. Rozpoznawane identyfikatory GUID są następujące: - EFI System partition: + Partycja systemowa EFI: c12a7328-f81f-11d2-ba4b-00a0c93ec93b - Microsoft Reserved partition: + Partycja zastrzeżona firmy Microsoft: e3c9e316-0b5c-4db8-817d-f92df00215ae - Basic data partition: + Podstawowa partycja danych: ebd0a0a2-b9e5-4433-87c0-68b6b72699c7 - LDM Metadata partition on a dynamic disk: + Partycja metadanych LDM na dysku dynamicznym: 5808c8aa-7e8f-42e0-85d2-e1e90434cfb3 - LDM Data partition on a dynamic disk: + Partycja danych LDM na dysku dynamicznym: af9b60a0-1431-4f62-bc68-3311714a69ad - Recovery partition: + Partycja odzyskiwania: de94bba4-06d1-4d40-a16a-bfd50179d6ac - If this parameter is not specified for a GPT disk, the command - creates a basic data partition. - - Any partition type byte or GUID can be specified with this - parameter. DiskPart does not check the partition type for - validity except to ensure that it is a byte in hexadecimal form - or a GUID. - - Caution: - - Creating partitions with this parameter might cause your - computer to fail or be unable to start up. Unless you are - an OEM or an IT professional experienced with GPT disks, do - not create partitions on GPT disks using this parameter. - Instead, always use the CREATE PARTITION EFI command to - create EFI System partitions, the CREATE PARTITION MSR - command to create Microsoft Reserved partitions, and the - CREATE PARTITION PRIMARY command without this parameter to - create primary partitions on GPT disks. - - ALIGN=<N> Typically used with hardware RAID Logical Unit Number (LUN) - arrays to improve performance. The partition offset will be - a multiple of <N>. If the OFFSET parameter is specified, it - will be rounded to the closest multiple of <N>. - - NOERR For scripting only. When an error is encountered, DiskPart - continues to process commands as if the error did not occur. - Without the NOERR parameter, an error causes DiskPart to exit - with an error code. - - After you create the partition, the focus automatically shifts to the new - partition. The partition does not receive a drive letter. You must use the - assign command to assign a drive letter to the partition. - - A basic disk must be selected for this operation to succeed. - - If a partition type is not specified, the disk is uninitialized and disk - size is greater than 2TB, it will be initialized to GPT. - -Example: + Jeśli ten parametr nie zostanie określony dla dysku GPT, + polecenie utworzy podstawową partycję danych. + + Za pomocą tego parametru można określić dowolny bajt + typu partycji lub identyfikator GUID. Program DiskPart nie + sprawdza poprawności typu partycji. Sprawdza jedynie, czy + podana wartość ma postać szesnastkową lub jest + identyfikatorem GUID. + + Uwaga: + + Tworzenie partycji z tym parametrem może sprawić, że + komputer ulegnie awarii lub nie będzie można go + uruchomić. Jeśli użytkownik nie jest producentem OEM ani + informatykiem dysponującym doświadczeniem w zakresie + dysków GPT, nie powinien tworzyć partycji na dyskach GPT + przy użyciu tego parametru. Zamiast tego należy zawsze + używać polecenia CREATE PARTITION EFI do tworzenia + partycji systemowych EFI, polecenia CREATE PARTITION MSR + do tworzenia partycji zastrzeżonej firmy Microsoft + i polecenia CREATE PARTITION PRIMARY (bez opisywanego + parametru) do tworzenia partycji podstawowych + na dyskach GPT. + + ALIGN=<N> Zazwyczaj używany z numerami jednostek logicznych (LUN) + sprzętowych macierzy RAID w celu poprawienia wydajności. + Przesunięcie partycji będzie wielokrotnością liczby N. + Jeśli nie określono parametru OFFSET, jest ono zaokrąglane + do najbliższej wielokrotności liczby N. + + NOERR Tylko do obsługi skryptów. Po wystąpieniu błędu program + DiskPart kontynuuje przetwarzanie poleceń tak, jakby błąd + nie wystąpił. W przypadku braku parametru NOERR błąd powoduje + zakończenie działania programu DiskPart i zwrócenie + kodu błędu. + + Po utworzeniu partycji automatycznie otrzymuje ona fokus. Partycja nie + otrzymuje litery dysku. Aby przypisać partycji literę dysku, + należy użyć polecenia ASSIGN. + + Aby ta operacja się powiodła, musi być wybrany dysk podstawowy. + + Jeśli typ partycji nie jest określony, dysk jest niezainicjowany a rozmiar + rozmiar większy niż 2 TB, zostanie zainicjowany do GPT. + +Przykład: CREATE PARTITION PRIMARY SIZE=1000 CREATE PARTITION PRIMARY SIZE=128 ID=c12a7328-f81f-11d2-ba4b-00a0c93ec93b @@ -2751,28 +2762,30 @@ Beispiel: DELETE PARTITION . Language=Polish -Deletes the partition with focus. +Usuwa partycję, na której jest ustawiony fokus. -Syntax: DELETE PARTITION [NOERR] [OVERRIDE] +Składnia: DELETE PARTITION [NOERR] [OVERRIDE] - NOERR For scripting only. When an error is encountered, DiskPart - continues to process commands as if the error did not occur. - Without the NOERR parameter, an error causes DiskPart to exit - with an error code. + NOERR Tylko do obsługi skryptów. Po wystąpieniu błędu program + DiskPart kontynuuje przetwarzanie poleceń tak, jakby błąd + nie wystąpił. W przypadku braku parametru NOERR błąd powoduje + zakończenie działania programu DiskPart i zwrócenie + kodu błędu. - OVERRIDE Enables DiskPart to delete any partition regardless of type. - Typically, DiskPart only permits you to delete known data - partitions. + OVERRIDE Umożliwia usunięcie przez program DiskPart każdej partycji bez + względu na jej typ. Zazwyczaj program DiskPart zezwala + na usunięcie wyłącznie znanych partycji zawierających dane. - You cannot delete the system partition, boot partition, or any partition - that contains the active paging file or crash dump (memory dump) filed. + Nie można usunąć partycji systemowej, partycji rozruchowej ani żadnej + partycji zawierającej aktywny plik stronicowania lub plik zrzutu + awaryjnego (zrzutu pamięci). - A partition must be selected for this operation to succeed. + Aby ta operacja się powiodła, musi być wybrana partycja. - Partitions cannot be deleted from dynamic disks or created on dynamic - disks. + Partycji nie można usuwać z dysków dynamicznych ani tworzyć ich na takich + dyskach. -Example: +Przykład: DELETE PARTITION . @@ -3023,14 +3036,14 @@ Beispiel: DETAIL DISK . Language=Polish - Displays the properties of the selected disk and the list of volumes on - the disk. + Wyświetla właściwości wybranego dysku oraz listę znajdujących się na nim + woluminów. -Syntax: DETAIL DISK +Składnia: DETAIL DISK - A disk must be selected for this operation to succeed. + Do pomyślnego ukończenia operacji konieczne jest wybranie dysku. -Example: +Przykład: DETAIL DISK . @@ -3146,13 +3159,13 @@ Beispiel: DETAIL PARTITION . Language=Polish - Displays the properties for the selected partition. + Wyświetla właściwości wybranej partycji. -Syntax: DETAIL PARTITION +Składnia: DETAIL PARTITION - A partition must be selected for this operation to succeed. + Do pomyślnego ukończenia operacji konieczne jest wybranie partycji. -Example: +Przykład: DETAIL PARTITION . @@ -3264,14 +3277,14 @@ Beispiel: DETAIL VOLUME . Language=Polish - Displays the properties for the selected volume and the list of disks on - which the volume resides. + Wyświetla właściwości wybranego woluminu oraz listę dysków, na których + wolumin się znajduje. -Syntax: DETAIL VOLUME +Składnia: DETAIL VOLUME - A volume must be selected for this operation to succeed. + Do pomyślnego ukończenia operacji konieczne jest wybranie woluminu. -Example: +Przykład: DETAIL VOLUME . @@ -3419,11 +3432,11 @@ Beispiel: EXIT . Language=Polish - Exits the DiskPart command interpreter. + Kończy działanie interpretera poleceń programu DiskPart. -Syntax: EXIT +Składnia: EXIT -Example: +Przykład: EXIT . @@ -3595,14 +3608,14 @@ Beispiel: FILESYSTEMS . Language=Polish - Displays information about the current file system for the selected - volume, and the supported file systems for formatting the volume. + Wyświetla informacje o bieżącym systemie plików w wybranym woluminie + i systemach plików obsługiwanych przy formatowaniu woluminu. -Syntax: FILESYSTEMS +Składnia: FILESYSTEMS - A volume must be selected for this operation to succeed. + Aby ta operacja się powiodła, musi być wybrany wolumin. -Example: +Przykład: FILESYSTEMS . @@ -4420,16 +4433,18 @@ Beispiel: HELP CREATE PARTITION PRIMARY . Language=Polish - Displays a list of the available commands or detailed help information for a - specified command. + Wyświetla listę dostępnych poleceń lub szczegółową pomoc dotyczącą + danego polecenia. -Syntax: HELP [<COMMAND>] +Składnia: HELP [<POLECENIE>] - <COMMAND> The command for which to display detail help. + <POLECENIE> Polecenie, dla którego ma zostać wyświetlona szczegółowa + pomoc. - If no command is specified, HELP will display all possible commands. + Jeśli polecenie nie jest określone, polecenie HELP wyświetla + wszystkie możliwe polecenia. -Example: +Przykład: HELP HELP CREATE PARTITION PRIMARY @@ -4630,25 +4645,26 @@ Beispiel: INACTIVE . Language=Polish - On disks with master boot record (MBR) disk formatting, marks - the partition with focus as inactive. + Na dyskach zawierających formatowanie z głównym rekordem rozruchowym (MBR) + oznacza partycję, na której jest ustawiony fokus, jako nieaktywną. -Syntax: INACTIVE +Składnia: INACTIVE - The computer may start from the next option specified in the BIOS such as a - CD-ROM drive or a Pre-Boot eXecution Environment (PXE)-based boot - environment (such as Remote Installation Services (RIS)) when you restart - the computer. + Podczas ponownego uruchamiania komputer może zostać uruchomiony według + następnej opcji określonej w systemie BIOS, takiej jak stacja dysków + CD-ROM lub środowisko rozruchowe oparte na środowisku PXE (Pre-Boot + eXecution Environment), na przykład usługi instalacji zdalnej (RIS). - A partition must be selected for this operation to succeed. + Aby operacja się powiodła, musi być wybrana partycja. - Caution: + Uwaga: - Your computer might not start without an active partition. Do not mark - a system or boot partition as inactive unless you are an experienced - user with a thorough understanding of ReactOS storage management. + Komputer może nie zostać uruchomiony bez aktywnej partycji. Jeśli + użytkownik nie dysponuje doświadczeniem w zakresie zarządzania + magazynami systemu Windows, nie powinien oznaczać partycji systemowej + ani rozruchowej jako nieaktywnej. -Example: +Przykład: INACTIVE . @@ -4872,27 +4888,28 @@ Beispiel: LIST DISK . Language=Polish - Displays a list of disks. + Wyświetla listę dysków. -Syntax: LIST DISK +Składnia: LIST DISK - Displays a list of disks and information about them, such as their - size, amount of available free space, whether the disk is a basic - or dynamic disk, and whether the disk uses the master boot record - (MBR) or GUID partition table (GPT) partition style. The disk marked - with an asterisk (*) has focus. + Wyświetla listę dysków oraz dotyczące ich informacje, takie jak rozmiar, + ilość dostępnego wolnego miejsca, informację, czy dysk jest podstawowy + lub dynamiczny oraz czy dysk używa głównego rekordu rozruchowego (MBR) + lub stylu partycji tabeli partycji identyfikatora GUID (GPT). Fokus jest + ustawiony na dysku oznaczonym gwiazdką (*). - Note that the FREE column does not display the total amount of free - space on the disk, but rather the amount of usable free space left - on the disk. For example, if you have a 10GB disk with 4 primary - partitions covering 5GB, there is no usable free space left (no - more partitions may be created). Another example would be you have - a 10GB disk with 3 primary partitions and an extended partition - covering 8GB. The exended partition is of size 3GB with one logical - drive of size 2GB. The disk will show only 1GB as free - the - 1GB of free space in the extended partition. + Należy zauważyć, że w kolumnie WOLNE nie jest wyświetlana całkowita + ilość wolnego miejsca na dysku, ale ilość pozostałego wolnego miejsca + na dysku zdatnego do użytku. Jeśli na przykład dysk o pojemności 10 GB + zawiera 4 partycje główne zajmujące 5 GB, na dysku nie ma wolnego + miejsca (nie można utworzyć następnych partycji). W innym przykładzie + dysk o pojemności 10 GB zawiera 3 partycje główne oraz partycję + rozszerzoną zajmujące 8 GB. Partycja rozszerzona ma rozmiar 3 GB + przy jednym dysku logicznym o rozmiarze 2 GB. Na dysku zostanie + pokazany tylko 1 GB wolnego miejsca i jest to 1 GB wolnego miejsca + w partycji rozszerzonej. -Example: +Przykład: LIST DISK . @@ -5106,17 +5123,18 @@ Beispiel: LIST PARTITION . Language=Polish - Displays a list of partitions in the partition table for the selected disk. + Wyświetla listę partycji znajdujących się w tabeli partycji wybranego + dysku. -Syntax: LIST PARTITION +Składnia: LIST PARTITION - On dynamic disks, the partitions do not neccessarily correspond to the - dynamic volumes on the disk. Partitions may not be created or deleted - on dynamic disks. + Na dyskach dynamicznych partycje nie muszą być zgodne z woluminami + dynamicznymi na dysku. Partycji nie można tworzyć na dyskach dynamicznych + ani ich usuwać z takich dysków. - A disk must be selected for this operation to succeed. + Aby ta operacja się powiodła, musi być wybrany dysk. -Example: +Przykład: LIST PARTITION . @@ -5251,12 +5269,12 @@ Beispiel: LIST VOLUME . Language=Polish - Displays a list of basic and dynamic volumes which are installed on the local - machine. + Wyświetla listę woluminów podstawowych i dynamicznych zainstalowanych + w komputerze lokalnym. -Syntax: LIST VOLUME +Składnia: LIST VOLUME -Example: +Przykład: LIST VOLUME . @@ -5554,16 +5572,16 @@ Beispiel: ASSIGN f: . Language=Polish - Provides a way to add comments to a script. + Umożliwia dodawanie komentarzy do skryptu. -Syntax: REM +Składnia: REM -Example: +Przykład: - In this example script, REM is used to provide a comment about what the - script does. + W tym przykładowym skrypcie polecenie REM zostało użyte w celu dodania + komentarza opisującego działanie skryptu. - REM These commands set up 3 drives. + REM Te polecenia konfigurują trzy dyski. CREATE PARTITION PRIMARY SIZE=2048 ASSIGN d: CREATE PARTITION EXTEND @@ -5802,11 +5820,11 @@ Beispiel: RESCAN . Language=Polish - Locates new disks that may have been added to the computer. + Lokalizuje nowe dyski, które można dodać do komputera. -Syntax: RESCAN +Składnia: RESCAN -Example: +Przykład: RESCAN . @@ -6039,40 +6057,44 @@ Beispiele: SELECT DISK=PCIROOT(0)#PCI(0100)#ATA(C00T00L01) . Language=Polish - Selects the specified disk and shifts the focus to it. + Wybiera określony dysk i przenosi na niego fokus. -Syntax: SELECT DISK=<N> - SELECT DISK=SYSTEM - SELECT DISK=NEXT - SELECT DISK=<Path> +Składnia: SELECT DISK=<N> + SELECT DISK=SYSTEM + SELECT DISK=NEXT + SELECT DISK=<Ścieżka> DISK=<N> - The DiskPart disk index number of the disk to receive - focus. + Numer indeksu dysku w programie DiskPart odpowiadający + dyskowi, na którym ma zostać ustawiony fokus. - DISK=<Path> - The location path of the disk to receive focus. + DISK=<Ścieżka> + Ścieżka lokalizacji dysku, na którym ma zostać + ustawiony fokus. DISK=SYSTEM - On BIOS machines, BIOS disk 0 will receive focus. - On EFI machines, the disk containing the ESP partition - used for the current boot will receive focus. On EFI - machines, if there is no ESP, or there is more than - one ESP present, or the machine is booted from Windows PE, - the command will fail. + Na komputerach z systemem BIOS fokus zostanie ustawiony na + dysku 0 systemu BIOS. Na komputerach z interfejsem EFI + fokus zostanie ustawiony na dysku zawierającym partycję ESP + użytą do wykonania bieżącego rozruchu. Wykonanie polecenia + nie powiedzie się na komputerach z interfejsem EFI, + na których nie ma partycji ESP lub jest kilka partycji ESP, + oraz gdy rozruch komputera nastąpił w środowisku Windows PE. DISK=NEXT - Once a disk is selected, this command is used to iterate - over all disks in the disk list. The next disk in the list - will receive focus. If the next disk is the start of the - enumeration, the command will fail and no disk will have - focus. + Po wybraniu dysku to polecenie jest używane do przejścia + przez wszystkie dyski znajdujące się na liście dysków. + Fokus zostanie ustawiony na następnym dysku na liście. + Jeśli następny dysk jest na początku wyliczania, wykonanie + polecenia nie powiedzie się i fokus nie zostanie ustawiony + na żadnym dysku. - If no options are specified, the select command lists the disk that - currently has the focus. You can view the DiskPart index numbers - for all disks on the computer by using the LIST DISK command. + Jeśli żadne opcje nie są określone, polecenie SELECT wyświetla dysk, + na którym obecnie jest ustawiony fokus. Numery indeksu w programie + DiskPart odpowiadające wszystkim dyskom w komputerze można wyświetlić + za pomocą polecenia LIST DISK. -Example: +Przykłady: SELECT DISK=1 SELECT DISK=SYSTEM @@ -6413,23 +6435,23 @@ Beispiel: SELECT PARTITION=1 . Language=Polish - Selects the specified partition and shifts the focus to it. + Wybiera określoną partycję i przenosi na nią fokus. -Syntax: SELECT PARTITION=<N> +Składnia: SELECT PARTITION=<N> PARTITION=<N> - The number of the partition to receive the focus. + Numer partycji, na której ma zostać ustawiony fokus. - If no partition is specified, the select command lists the current - partition with focus. You can specify the partition by its number. You can - view the numbers of all partitions on the current disk by using the list - partition command. + Jeśli żadna partycja nie jest określona, polecenie SELECT wyświetla + partycję, na której obecnie jest ustawiony fokus. Partycję można + określić, podając jej numer. Wszystkie numery partycji na bieżącym dysku + można wyświetlić za pomocą polecenia LIST PARTITION. - You must first select a disk using the DiskPart select disk command before - you can select a partition. + Przed wybraniem partycji należy wybrać dysk przy użyciu polecenia + SELECT DISK programu DiskPart. -Example: +Przykład: SELECT PARTITION=1 . @@ -6632,22 +6654,23 @@ Beispiel: SELECT VOLUME=C:\\MountH . Language=Polish - Selects the specified volume and shifts the focus to it. + Wybiera określony wolumin i ustawia na nim fokus. -Syntax: SELECT VOLUME={<N> | <D>} +Składnia: SELECT VOLUME={<N> | <D>} - VOLUME=<N> The number of the volume to receive the focus. + VOLUME=<N> Numer woluminu, na którym ma zostać ustawiony fokus. - VOLUME=<D> The drive letter or mounted folder path of the volume - to receive the focus. + VOLUME=<D> Litera dysku lub ścieżka zainstalowanego folderu woluminu, + na którym ma zostać ustawiony fokus. - If no volume is specified, the select command lists the current volume with - focus. You can specify the volume by number, drive letter, or mounted folder - path. On a basic disk, selecting a volume also gives the corresponding - partition focus. You can view the numbers of all volumes on the computer by - using the list volume command. + Jeśli żaden wolumin nie jest określony, polecenie SELECT wyświetla + wolumin, na którym obecnie jest ustawiony fokus. Wolumin można określić, + podając jego numer, literę dysku lub ścieżkę zainstalowanego woluminu. + Na dysku podstawowym fokus jest ustawiany również na partycji + odpowiadającej woluminowi. Numery wszystkich woluminów na komputerze można + wyświetlić przy użyciu polecenia LIST VOLUME. -Example: +Przykład: SELECT VOLUME=1 SELECT VOLUME=C @@ -7024,84 +7047,92 @@ Beispiel: SET ID=ebd0a0a2-b9e5-4433-87c0-68b6b72699c7 . Language=Polish - Changes the partition type field for the partition with focus. + Zmienia zawartość pola typu partycji dla partycji, na której ustawiony + jest fokus. -Syntax: SET ID={<BYTE> | <GUID>} [OVERRIDE] [NOERR] +Składnia: SET ID={<BYTE> | <GUID>} [OVERRIDE] [NOERR] ID={<BYTE> | <GUID>} - Specifies the new partition type. - For master boot record (MBR) disks, you can specify a partition - type byte, in hexadecimal form, for the partition. Any - partition type byte can be specified with this parameter except - for type 0x42 (LDM partition). Note that the leading '0x' is - omitted when specifying the hexadecimal partition type. + Określa nowy typ partycji. + W przypadku dysków z głównym rekordem rozruchowym (MBR) + można określić bajt typu partycji w formie szesnastkowej. + Za pomocą tego parametru można określić dowolny bajt typu + partycji z wyjątkiem typu 0x42 (partycja LDM). W przypadku + określania szesnastkowego typu partycji znaki '0x' + na początku są pomijane. - For GUID partition table (GPT) disks you can specify a - partition type GUID for the partition. Recognized GUIDs - include: + W przypadku dysków z tablicą partycji GUID (GPT) można + określić identyfikator GUID typu partycji. Rozpoznawane + identyfikatory GUID są następujące: - EFI System partition: + Partycja systemowa EFI: c12a7328-f81f-11d2-ba4b-00a0c93ec93b - Basic data partition: + Podstawowa partycja danych: ebd0a0a2-b9e5-4433-87c0-68b6b72699c7 - Any partition type GUID can be specified with this parameter - except for the following: + Za pomocą tego parametru można określić dowolny identyfikator + GUID typu partycji z wyjątkiem następujących: - Microsoft Reserved partition: + Partycja zastrzeżona firmy Microsoft: e3c9e316-0b5c-4db8-817d-f92df00215a - LDM Metadata partition on a dynamic disk: + Partycja metadanych LDM na dysku dynamicznym: 5808c8aa-7e8f-42e0-85d2-e1e90434cfb3 - LDM Data partition on a dynamic disk: + Partycja danych LDM na dysku dynamicznym: af9b60a0-1431-4f62-bc68-3311714a69ad - Cluster Metadata partition: + Partycja metadanych klastra: db97dba9-0840-4bae-97f0-ffb9a327c7e1 - Other than the limitations mentioned, DiskPart otherwise does - not check the partition type for validity except to ensure that - it is a byte in hexadecimal form or a GUID. - - OVERRIDE Enables DiskPart to force the file system on the volume to - dismount before changing the partition type. When changing - the partition type, DiskPart will attempt to lock and dismount - the file system on the volume. If this parameter is not specified, - and the call to lock the file system fails, (because some other - application has an open handle to the volume), the entire - operation will fail. When this parameter is specified, the - dismount is forced even if the call to lock the file system - fails. When a file system is dismounted, all opened handles to - the volume will become invalid. - - NOERR For scripting only. When an error is encountered, DiskPart - continues to process commands as if the error did not occur. - Without the NOERR parameter, an error causes DiskPart to exit - with an error code. - - Intended for Original Equipment Manufacturer (OEM) use only. - - A partition must be selected for this operation to succeed. - - Caution: - - Changing partition type fields with this parameter might cause your - computer to fail or be unable to start up. Unless you are an OEM or an - IT professional experienced with GPT disks, do not change partition - type fields on GPT disks using this parameter. Instead, always use the - CREATE PARTITION EFI command to create EFI System partitions, the - CREATE PARTITION MSR command to create Microsoft Reserved partitions, - and the CREATE PARTITION PRIMARY command without the ID parameter to - create primary partitions on GPT disks. - - This command does not work on dynamic disks nor on Microsoft Reserved - partitions. - -Example: + Oprócz wspomnianych ograniczeń program DiskPart nie sprawdza + poprawności typu partycji. Sprawdza jedynie, czy podana + wartość ma postać szesnastkową lub jest identyfikatorem GUID. + + OVERRIDE Powoduje, że przed zmianą typu partycji program DiskPart + wymusza odinstalowanie systemu plików w woluminie. Podczas + zmiany typu partycji program DiskPart podejmie próbę + zablokowania i odinstalowania systemu plików w woluminie. + Jeśli nie określono tego parametru, a wywołanie w celu + zablokowania systemu plików nie powiedzie się (ponieważ inna + aplikacja korzysta z otwartego dojścia do woluminu), cała + operacja zakończy się niepowodzeniem. Jeśli określono ten + parametr, odinstalowanie jest wymuszane nawet wtedy, + gdy wywołanie w celu zablokowania systemu plików nie + powiedzie się. Po odinstalowaniu systemu plików wszystkie + otwarte dojścia do woluminu staną się nieprawidłowe. + + NOERR Tylko do obsługi skryptów. Po wystąpieniu błędu program + DiskPart kontynuuje przetwarzanie poleceń, tak jakby błąd. + nie wystąpił. W przypadku braku parametru NOERR błąd + powoduje zakończenie działania programu DiskPart + i zwrócenie kodu błędu. + + Polecenie przeznaczone tylko do użytku przez producentów oryginalnego + sprzętu (OEM). + + Aby ta operacja się powiodła, musi być wybrana partycja. + + Uwaga: + + Zmiana pól typu partycji z tym parametrem może sprawić, że komputer + ulegnie awarii lub nie będzie można go uruchomić. Jeśli użytkownik + nie jest producentem OEM ani informatykiem dysponującym + doświadczeniem w zakresie dysków GPT, nie powinien zmieniać pól typu + partycji na dyskach GPT przy użyciu tego parametru. Zamiast tego + należy zawsze używać polecenia CREATE PARTITION EFI do tworzenia + partycji systemowych EFI, polecenia CREATE PARTITION MSR + do tworzenia partycji zastrzeżonej firmy Microsoft oraz polecenia + CREATE PARTITION PRIMARY bez parametru ID do tworzenia partycji + podstawowych na dyskach GPT. + + To polecenie nie działa na dyskach dynamicznych ani na partycjach + zastrzeżonych firmy Microsoft. + +Przykłady: SET ID=07 OVERRIDE SET ID=ebd0a0a2-b9e5-4433-87c0-68b6b72699c7 . @@ -7770,27 +7801,28 @@ Beispiel: UNIQUEID DISK ID=baf784e7-6bbd-4cfb-aaac-e86c96e166ee . Language=Polish - Displays or sets the GUID partition table (GPT) identifier or master boot - record (MBR) signature for the disk with focus. + Wyświetla lub ustawia identyfikator tabeli partycji GUID (GPT) lub podpis + głównego rekordu rozruchowego (MBR) dla dysku z fokusem. -Syntax: UNIQUEID DISK [ID={<DWORD> | <GUID>}] [NOERR] +Składnia: UNIQUEID DISK [ID={<DWORD> | <GUID>}] [NOERR] ID={<DWORD> | <GUID>} - For MBR disks, you can specify a four-byte (DWORD) value in - hexadecimal form for the signature. + Dla dysków MBR można określić czterobajtową wartość (DWORD) + podpisu w postaci szesnastkowej. - For GPT disks, specify a GUID for the identifier. + Dla dysków GPT należy określić ident. GUID jako identyfikator. - NOERR For scripting only. When an error is encountered, DiskPart - continues to process commands as if the error did not occur. - Without the NOERR parameter, an error causes DiskPart to exit - with an error code. + NOERR Tylko dla skryptów. W przypadku wystąpienia błędu program + DiskPart kontynuuje przetwarzanie poleceń, tak jakby błąd + nie wystąpił. W przypadku braku parametru NOERR błąd + powoduje zakończenie działania programu DiskPart i zwrócenie + kodu błędu. - A disk must be selected for this operation to succeed. This command works - on basic and dynamic disks. + Aby ta operacja powiodła się, musi być wybrany dysk. To polecenie działa + dla dysków podstawowych i dynamicznych. -Example: +Przykład: UNIQUEID DISK UNIQUEID DISK ID=5f1b2c36

1 0

[reactos] 01/01: [DISKPART] Update Polish (pl-PL) translation v2 (#4547)
by Piotr Hetnarowicz 23 Oct '22

23 Oct '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=bb406c4c1861f4d02d8d0… commit bb406c4c1861f4d02d8d00854e61d42b35b2599d Author: Piotr Hetnarowicz <piotrhwz(a)gmail.com> AuthorDate: Sun Oct 23 13:00:51 2022 +0200 Commit: GitHub <noreply(a)github.com> CommitDate: Sun Oct 23 14:00:51 2022 +0300 [DISKPART] Update Polish (pl-PL) translation v2 (#4547) Reviewed-by: Adam Stachowicz <saibamenppl(a)gmail.com> Reviewed-by: Stanislav Motylkov <x86corez(a)gmail.com> --- base/system/diskpart/lang/pl-PL.rc | 112 ++++++++++++++++++------------------- 1 file changed, 56 insertions(+), 56 deletions(-) diff --git a/base/system/diskpart/lang/pl-PL.rc b/base/system/diskpart/lang/pl-PL.rc index d67404e7812..e54a1faac37 100644 --- a/base/system/diskpart/lang/pl-PL.rc +++ b/base/system/diskpart/lang/pl-PL.rc @@ -17,28 +17,28 @@ END STRINGTABLE BEGIN - IDS_ACTIVE_FAIL "\nDiskPart was unable to mark the partition active.\nMake sure the partition is valid.\n" - IDS_ACTIVE_SUCCESS "\nDiskPart marked the current partition as active.\n" - IDS_ACTIVE_ALREADY "\nThe current partition is already marked as active.\n" + IDS_ACTIVE_FAIL "\nDiskPart nie mógł oznaczyć partycji jako aktywnej.\nUpewnij się, że partycja jest prawidłowa.\n" + IDS_ACTIVE_SUCCESS "\nDiskPart oznaczył bieżącą partycję jako aktywną.\n" + IDS_ACTIVE_ALREADY "\nBieżąca partycja jest już oznaczona jako aktywna.\n" END STRINGTABLE BEGIN - IDS_CLEAN_FAIL "\nDiskPart was unable to clean the disk.\nThe data on this disk may be unrecoverable.\n" - IDS_CLEAN_SUCCESS "\nDiskPart succeeded in cleaning the disk.\n" - IDS_CLEAN_SYSTEM "\nThe selected disk is neccessary to the operation of your computer, and may not be cleaned.\n" + IDS_CLEAN_FAIL "\nDiskPart nie może wyczyścić dysku.\nDane na dysku mogą nie nadawać się do odzyskania.\n" + IDS_CLEAN_SUCCESS "\nDiskPart pomyślnie oczyścił dysk.\n" + IDS_CLEAN_SYSTEM "\nWybrany dysk jest niezbędny do działania tego komputera, nie może być wyczyszczony.\n" END STRINGTABLE BEGIN - IDS_CREATE_PARTITION_FAIL "\nDiskPart was unable to create the specified partition.\n" - IDS_CREATE_PARTITION_SUCCESS "\nDiskPart succeeded in creating the specified partition.\n" + IDS_CREATE_PARTITION_FAIL "\nDiskPart nie może utworzyć określonej partycji.\n" + IDS_CREATE_PARTITION_SUCCESS "\nDiskPart pomyślnie utworzył określoną partycję.\n" END STRINGTABLE BEGIN - IDS_DELETE_PARTITION_FAIL "\nDiskPart failed to delete the selected partition.\nPlease make sure the selected partition is valid to delete.\n" - IDS_DELETE_PARTITION_SUCCESS "\nDiskPart successfully deleted the selected partition.\n" + IDS_DELETE_PARTITION_FAIL "\nDiskPart nie może usunąć wybranej partycji.\nUpewnij się, że wybrana partycja nadaje się do usunięcia.\n" + IDS_DELETE_PARTITION_SUCCESS "\nDiskPart pomyślnie usunął wybraną partycję.\n" END /* Disk Information Labels */ @@ -69,10 +69,10 @@ END STRINGTABLE BEGIN - IDS_FILESYSTEMS_CURRENT "Current Filesystem\n" - IDS_FILESYSTEMS_FORMATTING "Filesystems available for formatting\n" - IDS_FILESYSTEMS_TYPE "Type : %s\n" - IDS_FILESYSTEMS_CLUSTERSIZE "Cluster size: %lu\n" + IDS_FILESYSTEMS_CURRENT "Bieżący system plików\n" + IDS_FILESYSTEMS_FORMATTING "Systemy plików obsługiwane przy formatowaniu\n" + IDS_FILESYSTEMS_TYPE "Typ : %s\n" + IDS_FILESYSTEMS_CLUSTERSIZE "Rozmiar klastra: %lu\n" END STRINGTABLE @@ -82,9 +82,9 @@ END STRINGTABLE BEGIN - IDS_INACTIVE_FAIL "\nDiskPart was unable to mark the partition inactive.\nMake sure the partition is valid.\n" - IDS_INACTIVE_SUCCESS "\nDiskPart marked the current partition as inactive.\n" - IDS_INACTIVE_ALREADY "\nThe current partition is already marked as inactive.\n" + IDS_INACTIVE_FAIL "\nDiskPart nie może oznaczyć partycji jako nieaktywnej.\nUpewnij się, że partycja jest prawidłowa.\n" + IDS_INACTIVE_SUCCESS "\nDiskPart oznaczył bieżącą partycję jako nieaktywną.\n" + IDS_INACTIVE_ALREADY "\nBieżąca partycja jest już oznaczona jako nieaktywna.\n" END /* Detail header titles */ @@ -126,10 +126,10 @@ END STRINGTABLE BEGIN - IDS_SETID_FAIL "\nDiskPart was unable to change the partition type.\n" - IDS_SETID_SUCCESS "\nThe partition type was changed successfully.\n" - IDS_SETID_INVALID_FORMAT "\nThe format of the partition type is invalid.\n" - IDS_SETID_INVALID_TYPE "\nThe partition type is invalid.\n" + IDS_SETID_FAIL "\nDiskPart nie może zmienić typu partycji.\n" + IDS_SETID_SUCCESS "\nTyp partycji został pomyślnie zmieniony.\n" + IDS_SETID_INVALID_FORMAT "\nFormat typu partycji jest nieprawidłowy.\n" + IDS_SETID_INVALID_TYPE "\nTyp partycji jest nieprawidłowy.\n" END /* Disk Status */ @@ -154,54 +154,54 @@ END /* Help Command Descriptions */ STRINGTABLE BEGIN - IDS_HELP_ACTIVE "Oznacza bieżącą partycję podstawową jako aktywną.\n" + IDS_HELP_ACTIVE "Oznacz wybraną partycję jako aktywną.\n" IDS_HELP_ADD "Dodaje dublowanie do woluminu prostego.\n" IDS_HELP_ASSIGN "Przypisz literę dysku lub punkt instalacji dla wybranego\n woluminu.\n" IDS_HELP_ATTACH "Dołącza wirtualny dysk.\n" IDS_HELP_ATTRIBUTES "Manipuluje atrybutami woluminu lub dysku.\n" - IDS_HELP_AUTOMOUNT "Włącza i wyłącza automatyczne instalowanie woluminów\n podstawowych.\n" + IDS_HELP_AUTOMOUNT "Włącz i wyłącz automatyczne instalowanie woluminów\n podstawowych.\n" IDS_HELP_BREAK "Dzieli zestaw dublowania.\n" IDS_HELP_CLEAN "Usuń informacje o konfiguracji lub wszystkie informacje\n z dysku.\n" IDS_HELP_COMPACT "Próbuje zmniejszyć fizyczny rozmiaru pliku.\n" IDS_HELP_CONVERT "Konwertuje między różnymi formatami dysków.\n" IDS_HELP_CREATE "Tworzy wolumin, partycję, lub dysk wirtualny.\n" - IDS_HELP_CREATE_PARTITION "Create a partition.\n" - IDS_HELP_CREATE_PARTITION_EFI "Create an EFI system partition.\n" - IDS_HELP_CREATE_PARTITION_EXTENDED "Create an extended partition.\n" - IDS_HELP_CREATE_PARTITION_LOGICAL "Create a logical drive.\n" - IDS_HELP_CREATE_PARTITION_MSR "Create an MSR partition.\n" - IDS_HELP_CREATE_PARTITION_PRIMARY "Create a primary partition.\n" - IDS_HELP_CREATE_VOLUME "Create a volume.\n" - IDS_HELP_CREATE_VDISK "Create a virtual disk file.\n" - - IDS_HELP_DELETE "Usuń obiekt.\n" - IDS_HELP_DELETE_DISK "Delete a disk.\n" - IDS_HELP_DELETE_PARTITION "Delete a partition.\n" - IDS_HELP_DELETE_VOLUME "Delete a volume.\n" + IDS_HELP_CREATE_PARTITION "Tworzy partycję.\n" + IDS_HELP_CREATE_PARTITION_EFI "Tworzy partycję systemową EFI.\n" + IDS_HELP_CREATE_PARTITION_EXTENDED "Tworzy partycję rozszerzoną.\n" + IDS_HELP_CREATE_PARTITION_LOGICAL "Tworzy dysk logiczny.\n" + IDS_HELP_CREATE_PARTITION_MSR "Tworzy zastrzeżoną partycję MSR.\n" + IDS_HELP_CREATE_PARTITION_PRIMARY "Tworzy partycję podstawową.\n" + IDS_HELP_CREATE_VOLUME "Tworzy wolumin.\n" + IDS_HELP_CREATE_VDISK "Tworzy plik dysku wirtualnego.\n" + + IDS_HELP_DELETE "Usuwanie obiektu.\n" + IDS_HELP_DELETE_DISK "Usuwanie dysku.\n" + IDS_HELP_DELETE_PARTITION "Usuwanie wybranej partycji.\n" + IDS_HELP_DELETE_VOLUME "Usuwanie wybranego woluminu.\n" IDS_HELP_DETACH "Odłącza wirtualny dysk.\n" IDS_HELP_DETAIL "Podaj szczegółowe informacje o obiekcie.\n" - IDS_HELP_DETAIL_DISK "Print disk details.\n" - IDS_HELP_DETAIL_PARTITION "Print partition details.\n" - IDS_HELP_DETAIL_VOLUME "Print volume details.\n" + IDS_HELP_DETAIL_DISK "Wyświetl właściwości wybranego dysku.\n" + IDS_HELP_DETAIL_PARTITION "Wyświetl właściwości wybranej partycji.\n" + IDS_HELP_DETAIL_VOLUME "Wyświetl właściwości wybranego woluminu.\n" IDS_HELP_EXIT "Zakończ program DiskPart.\n" IDS_HELP_EXPAND "Zwiększa maksymalny rozmiar dysku wirtualnego.\n" IDS_HELP_EXTEND "Rozszerz wolumin.\n" - IDS_HELP_FILESYSTEMS "Drukuj obecne i obsługiwane systemy plików na tym woluminie.\n" - IDS_HELP_FORMAT "Formatuje wolumin lub partycję.\n" - IDS_HELP_GPT "Przypisuje atrybuty do wybranej partycji GPT.\n" - IDS_HELP_HELP "Drukuje listę poleceń.\n" - IDS_HELP_IMPORT "Importuje grupę dysków.\n" - IDS_HELP_INACTIVE "Oznacza bieżącą partycję podstawową jako nieaktywną.\n" - - IDS_HELP_LIST "Drukuje listę obiektów.\n" - IDS_HELP_LIST_DISK "List disks.\n" - IDS_HELP_LIST_PARTITION "List partitions.\n" - IDS_HELP_LIST_VOLUME "List volumes.\n" - IDS_HELP_LIST_VDISK "List virtual disk files.\n" + IDS_HELP_FILESYSTEMS "Wyświetl obecne i obsługiwane systemy plików w woluminie.\n" + IDS_HELP_FORMAT "Formatuj wolumin lub partycję.\n" + IDS_HELP_GPT "Przypisz atrybuty do wybranej partycji GPT.\n" + IDS_HELP_HELP "Wyświetl listę poleceń.\n" + IDS_HELP_IMPORT "Importuj grupę dysków.\n" + IDS_HELP_INACTIVE "Oznacza wybraną partycję jako nieaktywną.\n" + + IDS_HELP_LIST "Wyświetl listę obiektów.\n" + IDS_HELP_LIST_DISK "Wyświetl listę dysków.\n" + IDS_HELP_LIST_PARTITION "Wyświetl listę partycji na wybranym dysku.\n" + IDS_HELP_LIST_VOLUME "Wyświetl listę woluminów.\n" + IDS_HELP_LIST_VDISK "Wyświetl listę dysków wirtualnych.\n" IDS_HELP_MERGE "Scala dysk podrzędny z dyskami nadrzędnymi.\n" IDS_HELP_OFFLINE "Przęłącz do trybu offline obiekt będący w trybie online.\n" @@ -214,11 +214,11 @@ BEGIN IDS_HELP_RETAIN "Umieść zachowaną partycję w woluminie prostym.\n" IDS_HELP_SAN "Wyświetla lub ustawia zasady SAN dla aktualnie uruchomionego\n systemu operacyjnego.\n" - IDS_HELP_SELECT "Przenieś fokus na jeden z obiektów.\n" - IDS_HELP_SELECT_DISK "Moves the focus to the disk.\n" - IDS_HELP_SELECT_PARTITION "Moves the focus to the partition.\n" - IDS_HELP_SELECT_VOLUME "Moves the focus to the volume.\n" - IDS_HELP_SELECT_VDISK "Moves the focus to the virtual disk.\n" + IDS_HELP_SELECT "Przenieś fokus na obiekt.\n" + IDS_HELP_SELECT_DISK "Przenieś fokus na dysk.\n" + IDS_HELP_SELECT_PARTITION "Przenieś fokus na partycję.\n" + IDS_HELP_SELECT_VOLUME "Przenieś fokus na wolumin.\n" + IDS_HELP_SELECT_VDISK "Przenieś fokus na dysk wirtualny.\n" IDS_HELP_SETID "Zmien typ partycji.\n" IDS_HELP_SHRINK "Zmniejsz rozmiar wybranego woluminu.\n"

1 0

[reactos] 01/01: [USER32] Fix UpdatePerUserImmEnabling tracing bug
by Katayama Hirofumi MZ 23 Oct '22

23 Oct '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=67d8ffb60b6ea94d37695… commit 67d8ffb60b6ea94d37695d7ad7acdcfa53c0afeb Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Sun Oct 23 13:02:37 2022 +0900 Commit: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> CommitDate: Sun Oct 23 13:02:37 2022 +0900 [USER32] Fix UpdatePerUserImmEnabling tracing bug CORE-11700 --- win32ss/user/user32/misc/imm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/win32ss/user/user32/misc/imm.c b/win32ss/user/user32/misc/imm.c index 857ae50ae1f..2d861540314 100644 --- a/win32ss/user/user32/misc/imm.c +++ b/win32ss/user/user32/misc/imm.c @@ -1105,9 +1105,9 @@ BOOL WINAPI UpdatePerUserImmEnabling(VOID) return TRUE; imm32 = LoadLibraryW(L"imm32.dll"); - if (imm32) + if (imm32 == NULL) { - ERR("UPUIE: Imm32 not installed!\n"); + ERR("Imm32 not installed!\n"); ret = FALSE; }

1 0

[reactos] 01/01: [IMM32] Add UNEXPECTED() macro for detailed debugging
by Katayama Hirofumi MZ 23 Oct '22

23 Oct '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=a65014e3a0cc73ba1dbb6… commit a65014e3a0cc73ba1dbb6a39fdd614c4d3f06f07 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Sun Oct 23 12:49:37 2022 +0900 Commit: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> CommitDate: Sun Oct 23 12:49:37 2022 +0900 [IMM32] Add UNEXPECTED() macro for detailed debugging CORE-11700 --- dll/win32/imm32/precomp.h | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/dll/win32/imm32/precomp.h b/dll/win32/imm32/precomp.h index 629eaba4e4b..6db1f8d6e4d 100644 --- a/dll/win32/imm32/precomp.h +++ b/dll/win32/imm32/precomp.h @@ -100,6 +100,12 @@ BOOL APIENTRY Imm32IsCrossThreadAccess(HIMC hIMC); BOOL APIENTRY Imm32IsCrossProcessAccess(HWND hWnd); BOOL WINAPI Imm32IsImcAnsi(HIMC hIMC); +#if 0 + #define UNEXPECTED() ASSERT(FALSE) +#else + #define UNEXPECTED() 0 +#endif + /* * Unexpected Condition Checkers * --- Examine the condition, and then generate trace log if necessary. @@ -113,19 +119,19 @@ BOOL WINAPI Imm32IsImcAnsi(HIMC hIMC); #else /* on Debug */ #define IS_NULL_UNEXPECTEDLY(p) \ (!(p) ? (ros_dbg_log(__WINE_DBCL_ERR, __wine_dbch___default, \ - __FILE__, __FUNCTION__, __LINE__, "%s was NULL\n", #p), TRUE) \ + __FILE__, __FUNCTION__, __LINE__, "%s was NULL\n", #p), UNEXPECTED(), TRUE) \ : FALSE) #define IS_ZERO_UNEXPECTEDLY(p) \ (!(p) ? (ros_dbg_log(__WINE_DBCL_ERR, __wine_dbch___default, \ - __FILE__, __FUNCTION__, __LINE__, "%s was zero\n", #p), TRUE) \ + __FILE__, __FUNCTION__, __LINE__, "%s was zero\n", #p), UNEXPECTED(), TRUE) \ : FALSE) #define IS_TRUE_UNEXPECTEDLY(x) \ ((x) ? (ros_dbg_log(__WINE_DBCL_ERR, __wine_dbch___default, \ - __FILE__, __FUNCTION__, __LINE__, "%s was non-zero\n", #x), TRUE) \ + __FILE__, __FUNCTION__, __LINE__, "%s was non-zero\n", #x), UNEXPECTED(), TRUE) \ : FALSE) #define IS_FALSE_UNEXPECTEDLY(x) \ ((!(x)) ? (ros_dbg_log(__WINE_DBCL_ERR, __wine_dbch___default, \ - __FILE__, __FUNCTION__, __LINE__, "%s was FALSE\n", #x), TRUE) \ + __FILE__, __FUNCTION__, __LINE__, "%s was FALSE\n", #x), UNEXPECTED(), TRUE) \ : FALSE) #define IS_ERROR_UNEXPECTEDLY(x) \ ((x) != ERROR_SUCCESS ? (ros_dbg_log(__WINE_DBCL_ERR, __wine_dbch___default, \

1 0

[reactos] 01/01: [IMM32] Improve debuggability at all (#4798)
by Katayama Hirofumi MZ 23 Oct '22

23 Oct '22

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=c2a943653e1f4881a10d2… commit c2a943653e1f4881a10d20054ae951bd53cc1a1c Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Sun Oct 23 09:51:24 2022 +0900 Commit: GitHub <noreply(a)github.com> CommitDate: Sun Oct 23 09:51:24 2022 +0900 [IMM32] Improve debuggability at all (#4798) - Strengthen tracing. - Introduce IS_NULL_UNEXPECTEDLY, IS_ZERO_UNEXPECTEDLY etc. macros to manage tracing. CORE-11700 --- dll/win32/imm32/candidate.c | 36 ++++-- dll/win32/imm32/compstr.c | 69 +++++----- dll/win32/imm32/ctf.c | 6 +- dll/win32/imm32/guideline.c | 6 +- dll/win32/imm32/ime.c | 300 ++++++++++++++++++++++++++++++-------------- dll/win32/imm32/imm.c | 169 +++++++++++++++++-------- dll/win32/imm32/keymsg.c | 126 +++++++++++-------- dll/win32/imm32/precomp.h | 68 ++++++---- dll/win32/imm32/regword.c | 75 ++++++----- dll/win32/imm32/utils.c | 161 ++++++++++++++++++------ dll/win32/imm32/win3.c | 23 ++-- 11 files changed, 685 insertions(+), 354 deletions(-) diff --git a/dll/win32/imm32/candidate.c b/dll/win32/imm32/candidate.c index f816500d75d..a1df77e7b33 100644 --- a/dll/win32/imm32/candidate.c +++ b/dll/win32/imm32/candidate.c @@ -161,19 +161,19 @@ ImmGetCandidateListAW(HIMC hIMC, DWORD dwIndex, LPCANDIDATELIST lpCandList, DWOR LPCANDIDATELIST pCL; pClientImc = ImmLockClientImc(hIMC); - if (!pClientImc) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return 0; uCodePage = pClientImc->uCodePage; pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) { ImmUnlockClientImc(pClientImc); return 0; } pCI = ImmLockIMCC(pIC->hCandInfo); - if (pCI == NULL) + if (IS_NULL_UNEXPECTEDLY(pCI)) { ImmUnlockIMC(hIMC); ImmUnlockClientImc(pClientImc); @@ -181,7 +181,10 @@ ImmGetCandidateListAW(HIMC hIMC, DWORD dwIndex, LPCANDIDATELIST lpCandList, DWOR } if (pCI->dwSize < sizeof(CANDIDATEINFO) || pCI->dwCount <= dwIndex) + { + ERR("\n"); goto Quit; + } /* get required size */ pCL = (LPCANDIDATELIST)((LPBYTE)pCI + pCI->dwOffset[dwIndex]); @@ -245,17 +248,17 @@ ImmGetCandidateListCountAW(HIMC hIMC, LPDWORD lpdwListCount, BOOL bAnsi) const CANDIDATELIST *pCL; const DWORD *pdwOffsets; - if (lpdwListCount == NULL) + if (IS_NULL_UNEXPECTEDLY(lpdwListCount)) return 0; *lpdwListCount = 0; pClientImc = ImmLockClientImc(hIMC); - if (pClientImc == NULL) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return 0; pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) { ImmUnlockClientImc(pClientImc); return 0; @@ -264,7 +267,7 @@ ImmGetCandidateListCountAW(HIMC hIMC, LPDWORD lpdwListCount, BOOL bAnsi) uCodePage = pClientImc->uCodePage; pCI = ImmLockIMCC(pIC->hCandInfo); - if (pCI == NULL) + if (IS_NULL_UNEXPECTEDLY(pCI)) { ImmUnlockIMC(hIMC); ImmUnlockClientImc(pClientImc); @@ -272,7 +275,10 @@ ImmGetCandidateListCountAW(HIMC hIMC, LPDWORD lpdwListCount, BOOL bAnsi) } if (pCI->dwSize < sizeof(CANDIDATEINFO)) + { + ERR("\n"); goto Quit; + } *lpdwListCount = pCI->dwCount; /* the number of candidate lists */ @@ -371,10 +377,13 @@ ImmGetCandidateWindow(HIMC hIMC, DWORD dwIndex, LPCANDIDATEFORM lpCandidate) TRACE("(%p, %lu, %p)\n", hIMC, dwIndex, lpCandidate); if (dwIndex >= MAX_CANDIDATEFORM) /* Windows didn't check but we do for security reason */ + { + ERR("\n"); return FALSE; + } pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; pCF = &pIC->cfCandForm[dwIndex]; @@ -383,6 +392,10 @@ ImmGetCandidateWindow(HIMC hIMC, DWORD dwIndex, LPCANDIDATEFORM lpCandidate) *lpCandidate = *pCF; ret = TRUE; } + else + { + ERR("\n"); + } ImmUnlockIMC(hIMC); TRACE("ret: %d\n", ret); @@ -400,13 +413,16 @@ BOOL WINAPI ImmSetCandidateWindow(HIMC hIMC, LPCANDIDATEFORM lpCandidate) TRACE("(%p, %p)\n", hIMC, lpCandidate); if (lpCandidate->dwIndex >= MAX_CANDIDATEFORM) + { + ERR("\n"); return FALSE; + } - if (Imm32IsCrossThreadAccess(hIMC)) + if (IS_CROSS_THREAD_HIMC(hIMC)) return FALSE; pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; hWnd = pIC->hWnd; diff --git a/dll/win32/imm32/compstr.c b/dll/win32/imm32/compstr.c index 2230a0858bc..980e904e1c6 100644 --- a/dll/win32/imm32/compstr.c +++ b/dll/win32/imm32/compstr.c @@ -23,11 +23,11 @@ Imm32OpenICAndCS(HIMC hIMC, LPINPUTCONTEXT *ppIC, LPCOMPOSITIONSTRING *ppCS) *ppCS = NULL; pIC = ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; pCS = ImmLockIMCC(pIC->hCompStr); - if (!pCS) + if (IS_NULL_UNEXPECTEDLY(pCS)) { ImmUnlockIMC(hIMC); return FALSE; @@ -272,7 +272,7 @@ Imm32GetCompStrA(HIMC hIMC, const COMPOSITIONSTRING *pCS, DWORD dwIndex, break; default: - FIXME("TODO:\n"); + FIXME("\n"); return IMM_ERROR_GENERAL; } } @@ -357,7 +357,7 @@ Imm32GetCompStrA(HIMC hIMC, const COMPOSITIONSTRING *pCS, DWORD dwIndex, break; default: - FIXME("TODO:\n"); + FIXME("\n"); return IMM_ERROR_GENERAL; } } @@ -449,7 +449,7 @@ Imm32GetCompStrW(HIMC hIMC, const COMPOSITIONSTRING *pCS, DWORD dwIndex, break; default: - FIXME("TODO:\n"); + FIXME("\n"); return IMM_ERROR_GENERAL; } } @@ -506,7 +506,7 @@ Imm32GetCompStrW(HIMC hIMC, const COMPOSITIONSTRING *pCS, DWORD dwIndex, break; default: - FIXME("TODO:\n"); + FIXME("\n"); return IMM_ERROR_GENERAL; } } @@ -531,11 +531,14 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe dwThreadId = (DWORD)NtUserQueryInputContext(hIMC, QIC_INPUTTHREADID); if (dwThreadId != GetCurrentThreadId()) + { + ERR("\n"); return FALSE; + } hKL = GetKeyboardLayout(dwThreadId); pImeDpi = ImmLockImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return FALSE; uCodePage = pImeDpi->uCodePage; @@ -551,11 +554,12 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe break; /* FALL THROUGH */ default: + ERR("\n"); ImmUnlockImeDpi(pImeDpi); return FALSE; } - if (bAnsiAPI == bAnsiClient || (!pComp && !pRead)) + if (bAnsiAPI == bAnsiClient || (!pComp && !pRead)) /* No conversion needed */ { ret = pImeDpi->ImeSetCompositionString(hIMC, dwIndex, pComp, dwCompLen, pRead, dwReadLen); @@ -565,6 +569,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe if (!Imm32OpenICAndCS(hIMC, &pIC, &pCS)) { + ERR("\n"); ImmUnlockImeDpi(pImeDpi); return FALSE; } @@ -585,7 +590,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe { cbCompNew = Imm32CompStrWideToAnsi(pComp, dwCompLen, NULL, 0, uCodePage); pCompNew = ImmLocalAlloc(0, cbCompNew); - if (!pCompNew) + if (IS_NULL_UNEXPECTEDLY(pCompNew)) goto Quit; Imm32CompStrWideToAnsi(pComp, dwCompLen, pCompNew, cbCompNew, uCodePage); @@ -594,7 +599,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe { cbCompNew = Imm32CompStrAnsiToWide(pComp, dwCompLen, NULL, 0, uCodePage); pCompNew = ImmLocalAlloc(0, cbCompNew); - if (!pCompNew) + if (IS_NULL_UNEXPECTEDLY(pCompNew)) goto Quit; Imm32CompStrAnsiToWide(pComp, dwCompLen, pCompNew, cbCompNew, uCodePage); @@ -607,7 +612,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe { cbReadNew = Imm32CompStrWideToAnsi(pRead, dwReadLen, NULL, 0, uCodePage); pReadNew = ImmLocalAlloc(0, cbReadNew); - if (!pReadNew) + if (IS_NULL_UNEXPECTEDLY(pReadNew)) goto Quit; Imm32CompStrWideToAnsi(pRead, dwReadLen, pReadNew, cbReadNew, uCodePage); @@ -616,7 +621,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe { cbReadNew = Imm32CompStrAnsiToWide(pRead, dwReadLen, NULL, 0, uCodePage); pReadNew = ImmLocalAlloc(0, cbReadNew); - if (!pReadNew) + if (IS_NULL_UNEXPECTEDLY(pReadNew)) goto Quit; Imm32CompStrAnsiToWide(pRead, dwReadLen, pReadNew, cbReadNew, uCodePage); @@ -634,7 +639,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe CS_SizeW(pCS, CompStr), NULL, 0, uCodePage); pCompNew = ImmLocalAlloc(0, cbCompNew); - if (!pCompNew) + if (IS_NULL_UNEXPECTEDLY(pCompNew)) goto Quit; Imm32CompAttrWideToAnsi(pComp, dwCompLen, @@ -648,7 +653,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe CS_SizeA(pCS, CompStr), NULL, 0, uCodePage); pCompNew = ImmLocalAlloc(0, cbCompNew); - if (!pCompNew) + if (IS_NULL_UNEXPECTEDLY(pCompNew)) goto Quit; Imm32CompAttrAnsiToWide(pComp, dwCompLen, @@ -666,7 +671,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe CS_SizeW(pCS, CompReadStr), NULL, 0, uCodePage); pReadNew = ImmLocalAlloc(0, cbReadNew); - if (!pReadNew) + if (IS_NULL_UNEXPECTEDLY(pReadNew)) goto Quit; Imm32CompAttrWideToAnsi(pRead, dwReadLen, @@ -680,7 +685,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe CS_SizeA(pCS, CompReadStr), NULL, 0, uCodePage); pReadNew = ImmLocalAlloc(0, cbReadNew); - if (!pReadNew) + if (IS_NULL_UNEXPECTEDLY(pReadNew)) goto Quit; Imm32CompAttrAnsiToWide(pRead, dwReadLen, @@ -698,7 +703,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe cbCompNew = Imm32CompClauseWideToAnsi(pComp, dwCompLen, CS_StrW(pCS, CompStr), NULL, 0, uCodePage); pCompNew = ImmLocalAlloc(0, cbCompNew); - if (!pCompNew) + if (IS_NULL_UNEXPECTEDLY(pCompNew)) goto Quit; Imm32CompClauseWideToAnsi(pComp, dwCompLen, CS_StrW(pCS, CompStr), @@ -709,7 +714,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe cbCompNew = Imm32CompClauseAnsiToWide(pComp, dwCompLen, CS_StrA(pCS, CompStr), NULL, 0, uCodePage); pCompNew = ImmLocalAlloc(0, cbCompNew); - if (!pCompNew) + if (IS_NULL_UNEXPECTEDLY(pCompNew)) goto Quit; Imm32CompClauseAnsiToWide(pComp, dwCompLen, CS_StrA(pCS, CompStr), @@ -724,7 +729,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe cbReadNew = Imm32CompClauseWideToAnsi(pRead, dwReadLen, CS_StrW(pCS, CompReadStr), NULL, 0, uCodePage); pReadNew = ImmLocalAlloc(0, cbReadNew); - if (!pReadNew) + if (IS_NULL_UNEXPECTEDLY(pReadNew)) goto Quit; Imm32CompClauseWideToAnsi(pRead, dwReadLen, @@ -736,7 +741,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe cbReadNew = Imm32CompClauseAnsiToWide(pRead, dwReadLen, CS_StrA(pCS, CompReadStr), NULL, 0, uCodePage); pReadNew = ImmLocalAlloc(0, cbReadNew); - if (!pReadNew) + if (IS_NULL_UNEXPECTEDLY(pReadNew)) goto Quit; Imm32CompClauseAnsiToWide(pRead, dwReadLen, CS_StrA(pCS, CompReadStr), @@ -753,7 +758,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe { cbCompNew = Imm32ReconvertAnsiFromWide(NULL, pComp, uCodePage); pCompNew = ImmLocalAlloc(0, cbCompNew); - if (!pCompNew) + if (IS_NULL_UNEXPECTEDLY(pCompNew)) goto Quit; pRS = pCompNew; @@ -765,7 +770,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe { cbCompNew = Imm32ReconvertWideFromAnsi(NULL, pComp, uCodePage); pCompNew = ImmLocalAlloc(0, cbCompNew); - if (!pCompNew) + if (IS_NULL_UNEXPECTEDLY(pCompNew)) goto Quit; pRS = pCompNew; @@ -781,7 +786,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe { cbReadNew = Imm32ReconvertAnsiFromWide(NULL, pRead, uCodePage); pReadNew = ImmLocalAlloc(0, cbReadNew); - if (!pReadNew) + if (IS_NULL_UNEXPECTEDLY(pReadNew)) goto Quit; pRS = pReadNew; @@ -793,7 +798,7 @@ ImmSetCompositionStringAW(HIMC hIMC, DWORD dwIndex, LPVOID pComp, DWORD dwCompLe { cbReadNew = Imm32ReconvertWideFromAnsi(NULL, pRead, uCodePage); pReadNew = ImmLocalAlloc(0, cbReadNew); - if (!pReadNew) + if (IS_NULL_UNEXPECTEDLY(pReadNew)) goto Quit; pRS = pReadNew; @@ -859,11 +864,11 @@ LONG WINAPI ImmGetCompositionStringA(HIMC hIMC, DWORD dwIndex, LPVOID lpBuf, DWO TRACE("(%p, %lu, %p, %lu)\n", hIMC, dwIndex, lpBuf, dwBufLen); - if (dwBufLen && !lpBuf) + if (dwBufLen && IS_NULL_UNEXPECTEDLY(lpBuf)) return 0; pClientImc = ImmLockClientImc(hIMC); - if (!pClientImc) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return 0; bAnsiClient = !(pClientImc->dwFlags & CLIENTIMC_WIDE); @@ -871,11 +876,11 @@ LONG WINAPI ImmGetCompositionStringA(HIMC hIMC, DWORD dwIndex, LPVOID lpBuf, DWO ImmUnlockClientImc(pClientImc); pIC = ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) return 0; pCS = ImmLockIMCC(pIC->hCompStr); - if (!pCS) + if (IS_NULL_UNEXPECTEDLY(pCS)) { ImmUnlockIMC(hIMC); return 0; @@ -902,11 +907,11 @@ LONG WINAPI ImmGetCompositionStringW(HIMC hIMC, DWORD dwIndex, LPVOID lpBuf, DWO TRACE("(%p, %lu, %p, %lu)\n", hIMC, dwIndex, lpBuf, dwBufLen); - if (dwBufLen && !lpBuf) + if (dwBufLen && IS_NULL_UNEXPECTEDLY(lpBuf)) return 0; pClientImc = ImmLockClientImc(hIMC); - if (!pClientImc) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return 0; bAnsiClient = !(pClientImc->dwFlags & CLIENTIMC_WIDE); @@ -914,11 +919,11 @@ LONG WINAPI ImmGetCompositionStringW(HIMC hIMC, DWORD dwIndex, LPVOID lpBuf, DWO ImmUnlockClientImc(pClientImc); pIC = ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) return 0; pCS = ImmLockIMCC(pIC->hCompStr); - if (!pCS) + if (IS_NULL_UNEXPECTEDLY(pCS)) { ImmUnlockIMC(hIMC); return 0; diff --git a/dll/win32/imm32/ctf.c b/dll/win32/imm32/ctf.c index 711e2d5cadf..8495725ed0f 100644 --- a/dll/win32/imm32/ctf.c +++ b/dll/win32/imm32/ctf.c @@ -20,7 +20,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(imm); // Win: LoadCtfIme HMODULE APIENTRY Imm32LoadCtfIme(VOID) { - FIXME("()\n"); return NULL; } @@ -29,6 +28,7 @@ HRESULT APIENTRY Imm32CtfImeDestroyInputContext(HIMC hIMC) { if (!Imm32LoadCtfIme()) return E_FAIL; + #if 1 FIXME("(%p)\n", hIMC); return E_NOTIMPL; @@ -126,7 +126,7 @@ BOOL WINAPI CtfImmIsGuidMapEnable(HIMC hIMC) return ret; pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return ret; ret = pImeDpi->CtfImeIsGuidMapEnable(hIMC); @@ -158,7 +158,7 @@ HRESULT WINAPI CtfImmGetGuidAtom(HIMC hIMC, DWORD dwUnknown, LPDWORD pdwGuidAtom return S_OK; pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return hr; hr = pImeDpi->CtfImeGetGuidAtom(hIMC, dwUnknown, pdwGuidAtom); diff --git a/dll/win32/imm32/guideline.c b/dll/win32/imm32/guideline.c index ea0658e0505..7a3e26b63a2 100644 --- a/dll/win32/imm32/guideline.c +++ b/dll/win32/imm32/guideline.c @@ -22,20 +22,20 @@ ImmGetGuideLineAW(HIMC hIMC, DWORD dwIndex, LPVOID lpBuf, DWORD dwBufLen, BOOL b UINT uCodePage; pClientImc = ImmLockClientImc(hIMC); - if (!pClientImc) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return 0; uCodePage = pClientImc->uCodePage; pIC = ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) { ImmUnlockClientImc(pClientImc); return 0; } pGuideLine = ImmLockIMCC(pIC->hGuideLine); - if (!pGuideLine) + if (IS_NULL_UNEXPECTEDLY(pGuideLine)) { ImmUnlockIMC(hIMC); ImmUnlockClientImc(pClientImc); diff --git a/dll/win32/imm32/ime.c b/dll/win32/imm32/ime.c index 45cde47c7f1..9bc3cd6830a 100644 --- a/dll/win32/imm32/ime.c +++ b/dll/win32/imm32/ime.c @@ -60,21 +60,28 @@ BOOL APIENTRY Imm32InquireIme(PIMEDPI pImeDpi) if (IS_IME_HKL(pImeDpi->hKL)) { if (!pImeDpi->ImeInquire(pImeInfo, szUIClass, dwSysInfoFlags)) + { + ERR("\n"); return FALSE; + } } else if (IS_CICERO_MODE() && !IS_16BIT_MODE()) { if (!pImeDpi->CtfImeInquireExW(pImeInfo, szUIClass, dwSysInfoFlags, pImeDpi->hKL)) + { + ERR("\n"); return FALSE; + } } else { + ERR("\n"); return FALSE; } szUIClass[_countof(szUIClass) - 1] = UNICODE_NULL; /* Avoid buffer overrun */ - if (pImeInfo->dwPrivateDataSize == 0) + if (pImeInfo->dwPrivateDataSize < sizeof(DWORD)) pImeInfo->dwPrivateDataSize = sizeof(DWORD); #define VALID_IME_PROP (IME_PROP_AT_CARET | \ @@ -117,17 +124,35 @@ BOOL APIENTRY Imm32InquireIme(PIMEDPI pImeDpi) #define VALID_SELECT_CAPS (SELECT_CAP_CONVERSION | SELECT_CAP_SENTENCE) if (pImeInfo->fdwProperty & ~VALID_IME_PROP) + { + ERR("\n"); return FALSE; + } if (pImeInfo->fdwConversionCaps & ~VALID_CMODE_CAPS) + { + ERR("\n"); return FALSE; + } if (pImeInfo->fdwSentenceCaps & ~VALID_SMODE_CAPS) + { + ERR("\n"); return FALSE; + } if (pImeInfo->fdwUICaps & ~VALID_UI_CAPS) + { + ERR("\n"); return FALSE; + } if (pImeInfo->fdwSCSCaps & ~VALID_SCS_CAPS) + { + ERR("\n"); return FALSE; + } if (pImeInfo->fdwSelectCaps & ~VALID_SELECT_CAPS) + { + ERR("\n"); return FALSE; + } #undef VALID_IME_PROP #undef VALID_CMODE_CAPS @@ -151,7 +176,13 @@ BOOL APIENTRY Imm32InquireIme(PIMEDPI pImeDpi) pImeDpi->szUIClass[_countof(pImeDpi->szUIClass) - 1] = UNICODE_NULL; } - return GetClassInfoW(pImeDpi->hInst, pImeDpi->szUIClass, &wcW); + if (!GetClassInfoW(pImeDpi->hInst, pImeDpi->szUIClass, &wcW)) + { + ERR("\n"); + return FALSE; + } + + return TRUE; } /* Define stub IME functions */ @@ -177,7 +208,7 @@ BOOL APIENTRY Imm32LoadIME(PIMEINFOEX pImeInfoEx, PIMEDPI pImeDpi) pImeDpi->hInst = hIME = LoadLibraryW(szPath); if (hIME == NULL) { - ERR("Imm32LoadIME: LoadLibraryW(%s) failed\n", debugstr_w(szPath)); + ERR("LoadLibraryW(%s) failed\n", debugstr_w(szPath)); return FALSE; } @@ -242,14 +273,23 @@ PIMEDPI APIENTRY Imm32LoadImeDpi(HKL hKL, BOOL bLock) UINT uCodePage; LCID lcid; - if (!ImmGetImeInfoEx(&ImeInfoEx, ImeInfoExKeyboardLayout, &hKL) || - ImeInfoEx.fLoadFlag == 1) + if (!IS_IME_HKL(hKL)) + return NULL; + + if (!ImmGetImeInfoEx(&ImeInfoEx, ImeInfoExKeyboardLayout, &hKL)) + { + ERR("\n"); + return NULL; + } + + if (ImeInfoEx.fLoadFlag == 1) { + ERR("\n"); return NULL; } pImeDpiNew = ImmLocalAlloc(HEAP_ZERO_MEMORY, sizeof(IMEDPI)); - if (pImeDpiNew == NULL) + if (IS_NULL_UNEXPECTEDLY(pImeDpiNew)) return NULL; pImeDpiNew->hKL = hKL; @@ -263,6 +303,7 @@ PIMEDPI APIENTRY Imm32LoadImeDpi(HKL hKL, BOOL bLock) if (!Imm32LoadIME(&ImeInfoEx, pImeDpiNew)) { + ERR("\n"); ImmLocalFree(pImeDpiNew); return FALSE; } @@ -302,7 +343,10 @@ PIMEDPI APIENTRY Imm32FindOrLoadImeDpi(HKL hKL) PIMEDPI pImeDpi; if (!IS_IME_HKL(hKL) && (!IS_CICERO_MODE() || IS_16BIT_MODE())) + { + ERR("\n"); return NULL; + } pImeDpi = ImmLockImeDpi(hKL); if (pImeDpi == NULL) @@ -315,7 +359,6 @@ ImeDpi_Escape(PIMEDPI pImeDpi, HIMC hIMC, UINT uSubFunc, LPVOID lpData, HKL hKL) { if (IS_IME_HKL(hKL)) return pImeDpi->ImeEscape(hIMC, uSubFunc, lpData); - if (IS_CICERO_MODE() && !IS_16BIT_MODE()) return pImeDpi->CtfImeEscapeEx(hIMC, uSubFunc, lpData, hKL); @@ -412,7 +455,7 @@ LRESULT WINAPI ImmPutImeMenuItemsIntoMappedFile(HIMC hIMC) pView = MapViewOfFile(hMapping, FILE_MAP_ALL_ACCESS, 0, 0, 0); if (!pView || pView->dwVersion != 1) { - ERR("hMapping %p, pView %p\n", hMapping, pView); + ERR("\n"); goto Quit; } @@ -423,16 +466,13 @@ LRESULT WINAPI ImmPutImeMenuItemsIntoMappedFile(HIMC hIMC) { cbItems = pView->dwItemCount * sizeof(IMEMENUITEMINFOW); pItems = ImmLocalAlloc(HEAP_ZERO_MEMORY, cbItems); - if (!pItems) - { - ERR("!pItems\n"); + if (IS_NULL_UNEXPECTEDLY(pItems)) goto Quit; - } } cItems = ImmGetImeMenuItemsW(hIMC, pView->dwFlags, pView->dwType, pParent, pItems, cbItems); pView->dwItemCount = cItems; - if (cItems == 0) + if (IS_ZERO_UNEXPECTEDLY(cItems)) goto Quit; if (pItems) @@ -490,7 +530,7 @@ Imm32GetImeMenuItemWInterProcess(HIMC hIMC, DWORD dwFlags, DWORD dwType, LPVOID hImeWnd = (HWND)NtUserQueryInputContext(hIMC, QIC_DEFAULTWINDOWIME); if (!hImeWnd || !IsWindow(hImeWnd)) { - ERR("hImeWnd %p\n", hImeWnd); + ERR("\n"); return 0; } @@ -503,11 +543,8 @@ Imm32GetImeMenuItemWInterProcess(HIMC hIMC, DWORD dwFlags, DWORD dwType, LPVOID hMapping = CreateFileMappingW(INVALID_HANDLE_VALUE, NULL, PAGE_READWRITE, 0, cbView, L"ImmMenuInfo"); pView = MapViewOfFile(hMapping, FILE_MAP_READ | FILE_MAP_WRITE, 0, 0, 0); - if (!pView) - { - ERR("hMapping %p, pView %p\n", hMapping, pView); + if (IS_NULL_UNEXPECTEDLY(pView)) goto Quit; - } ZeroMemory(pView, cbView); pView->dwVersion = 1; @@ -521,7 +558,10 @@ Imm32GetImeMenuItemWInterProcess(HIMC hIMC, DWORD dwFlags, DWORD dwType, LPVOID } if (!SendMessageW(hImeWnd, WM_IME_SYSTEM, IMS_GETIMEMENU, (LPARAM)hIMC)) + { + ERR("\n"); goto Quit; + } ret = pView->dwItemCount; @@ -575,11 +615,11 @@ ImmGetImeMenuItemsAW(HIMC hIMC, DWORD dwFlags, DWORD dwType, LPVOID lpImeParentM BOOL bImcIsAnsi; HKL hKL; - if (!hIMC) + if (IS_NULL_UNEXPECTEDLY(hIMC)) return 0; dwProcessId = (DWORD)NtUserQueryInputContext(hIMC, QIC_INPUTPROCESSID); - if (dwProcessId == 0) + if (IS_ZERO_UNEXPECTEDLY(dwProcessId)) return 0; if (dwProcessId != GetCurrentProcessId()) @@ -591,11 +631,11 @@ ImmGetImeMenuItemsAW(HIMC hIMC, DWORD dwFlags, DWORD dwType, LPVOID lpImeParentM } pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return 0; dwThreadId = (DWORD)NtUserQueryInputContext(hIMC, QIC_INPUTTHREADID); - if (dwThreadId == 0) + if (IS_ZERO_UNEXPECTEDLY(dwThreadId)) { ImmUnlockIMC(hIMC); return 0; @@ -603,7 +643,7 @@ ImmGetImeMenuItemsAW(HIMC hIMC, DWORD dwFlags, DWORD dwType, LPVOID lpImeParentM hKL = GetKeyboardLayout(dwThreadId); pImeDpi = ImmLockImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) { ImmUnlockIMC(hIMC); return 0; @@ -622,7 +662,7 @@ ImmGetImeMenuItemsAW(HIMC hIMC, DWORD dwFlags, DWORD dwType, LPVOID lpImeParentM { cbTotal = ((dwSize / sizeof(IMEMENUITEMINFOA)) * sizeof(IMEMENUITEMINFOW)); pNewItems = ImmLocalAlloc(0, cbTotal); - if (!pNewItems) + if (IS_NULL_UNEXPECTEDLY(pNewItems)) goto Quit; } } @@ -635,7 +675,7 @@ ImmGetImeMenuItemsAW(HIMC hIMC, DWORD dwFlags, DWORD dwType, LPVOID lpImeParentM { cbTotal = ((dwSize / sizeof(IMEMENUITEMINFOW)) * sizeof(IMEMENUITEMINFOA)); pNewItems = ImmLocalAlloc(0, cbTotal); - if (!pNewItems) + if (IS_NULL_UNEXPECTEDLY(pNewItems)) goto Quit; } } @@ -647,7 +687,7 @@ ImmGetImeMenuItemsAW(HIMC hIMC, DWORD dwFlags, DWORD dwType, LPVOID lpImeParentM } ret = pImeDpi->ImeGetImeMenuItems(hIMC, dwFlags, dwType, pNewParent, pNewItems, dwSize); - if (!ret || !lpImeMenu) + if (IS_ZERO_UNEXPECTEDLY(ret) || !lpImeMenu) goto Quit; if (bImcIsAnsi != bTargetIsAnsi) @@ -663,6 +703,7 @@ ImmGetImeMenuItemsAW(HIMC hIMC, DWORD dwFlags, DWORD dwType, LPVOID lpImeParentM { if (!Imm32ImeMenuWideToAnsi(pItemW, pItemA, pImeDpi->uCodePage)) { + ERR("\n"); ret = 0; break; } @@ -679,6 +720,7 @@ ImmGetImeMenuItemsAW(HIMC hIMC, DWORD dwFlags, DWORD dwType, LPVOID lpImeParentM { if (!Imm32ImeMenuAnsiToWide(pItemA, pItemW, pImeDpi->uCodePage, TRUE)) { + ERR("\n"); ret = 0; break; } @@ -691,6 +733,7 @@ Quit: ImmLocalFree(pNewItems); ImmUnlockImeDpi(pImeDpi); ImmUnlockIMC(hIMC); + TRACE("ret: 0x%X\n", ret); return ret; } @@ -705,11 +748,11 @@ HKL WINAPI ImmInstallIMEA(LPCSTR lpszIMEFileName, LPCSTR lpszLayoutText) TRACE("(%s, %s)\n", debugstr_a(lpszIMEFileName), debugstr_a(lpszLayoutText)); pszFileNameW = Imm32WideFromAnsi(CP_ACP, lpszIMEFileName); - if (!pszFileNameW) + if (IS_NULL_UNEXPECTEDLY(pszFileNameW)) goto Quit; pszLayoutTextW = Imm32WideFromAnsi(CP_ACP, lpszLayoutText); - if (!pszLayoutTextW) + if (IS_NULL_UNEXPECTEDLY(pszLayoutTextW)) goto Quit; hKL = ImmInstallIMEW(pszFileNameW, pszLayoutTextW); @@ -737,16 +780,18 @@ HKL WINAPI ImmInstallIMEW(LPCWSTR lpszIMEFileName, LPCWSTR lpszLayoutText) GetFullPathNameW(lpszIMEFileName, _countof(szImeFileName), szImeFileName, &pchFilePart); CharUpperW(szImeFileName); - if (!pchFilePart) + if (IS_NULL_UNEXPECTEDLY(pchFilePart)) return NULL; /* Load the IME version info */ InfoEx.hkl = hNewKL = NULL; StringCchCopyW(InfoEx.wszImeFile, _countof(InfoEx.wszImeFile), pchFilePart); - if (Imm32LoadImeVerInfo(&InfoEx) && InfoEx.hkl) - wLangID = LOWORD(InfoEx.hkl); - else + if (!Imm32LoadImeVerInfo(&InfoEx) || !InfoEx.hkl) + { + ERR("\n"); return NULL; + } + wLangID = LOWORD(InfoEx.hkl); /* Get the IME layouts from registry */ cLayouts = Imm32GetImeLayout(NULL, 0); @@ -755,6 +800,7 @@ HKL WINAPI ImmInstallIMEW(LPCWSTR lpszIMEFileName, LPCWSTR lpszLayoutText) pLayouts = ImmLocalAlloc(0, cLayouts * sizeof(REG_IME)); if (!pLayouts || !Imm32GetImeLayout(pLayouts, cLayouts)) { + ERR("\n"); ImmLocalFree(pLayouts); return NULL; } @@ -764,7 +810,10 @@ HKL WINAPI ImmInstallIMEW(LPCWSTR lpszIMEFileName, LPCWSTR lpszLayoutText) if (lstrcmpiW(pLayouts[iLayout].szFileName, pchFilePart) == 0) { if (wLangID != LOWORD(pLayouts[iLayout].hKL)) + { + ERR("\n"); goto Quit; /* The language is different */ + } hNewKL = pLayouts[iLayout].hKL; /* Found */ break; @@ -776,6 +825,7 @@ HKL WINAPI ImmInstallIMEW(LPCWSTR lpszIMEFileName, LPCWSTR lpszLayoutText) if (ImmGetImeInfoEx(&InfoEx, ImeInfoExImeFileName, pchFilePart) && !UnloadKeyboardLayout(InfoEx.hkl)) { + ERR("\n"); hNewKL = NULL; goto Quit; } @@ -787,6 +837,7 @@ HKL WINAPI ImmInstallIMEW(LPCWSTR lpszIMEFileName, LPCWSTR lpszLayoutText) if (lstrcmpiW(szImeFileName, szImeDestPath) != 0 && !Imm32CopyImeFile(szImeFileName, szImeDestPath)) { + ERR("\n"); hNewKL = NULL; goto Quit; } @@ -800,11 +851,12 @@ HKL WINAPI ImmInstallIMEW(LPCWSTR lpszIMEFileName, LPCWSTR lpszLayoutText) if (Imm32WriteImeLayout(hNewKL, pchFilePart, lpszLayoutText)) { /* Load the keyboard layout */ - Imm32UIntToStr((DWORD)(DWORD_PTR)hNewKL, 16, szImeKey, _countof(szImeKey)); + StringCchPrintfW(szImeKey, _countof(szImeKey), L"%08X", (DWORD)(DWORD_PTR)hNewKL); hNewKL = LoadKeyboardLayoutW(szImeKey, KLF_REPLACELANG); } else { + ERR("\n"); hNewKL = NULL; } } @@ -830,7 +882,10 @@ BOOL WINAPI ImmIsIME(HKL hKL) HWND WINAPI ImmGetDefaultIMEWnd(HWND hWnd) { if (!IS_IMM_MODE()) + { + WARN("\n"); return NULL; + } if (hWnd == NULL) return (HWND)NtUserGetThreadState(THREADSTATE_DEFAULTIMEWINDOW); @@ -849,12 +904,12 @@ BOOL WINAPI ImmNotifyIME(HIMC hIMC, DWORD dwAction, DWORD dwIndex, DWORD dwValue TRACE("(%p, %lu, %lu, %lu)\n", hIMC, dwAction, dwIndex, dwValue); - if (hIMC && Imm32IsCrossThreadAccess(hIMC)) + if (hIMC && IS_CROSS_THREAD_HIMC(hIMC)) return FALSE; hKL = GetKeyboardLayout(0); pImeDpi = ImmLockImeDpi(hKL); - if (pImeDpi == NULL) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return FALSE; ret = pImeDpi->NotifyIME(hIMC, dwAction, dwIndex, dwValue); @@ -888,7 +943,10 @@ ImmGetImeInfoEx(PIMEINFOEX pImeInfoEx, IMEINFOEXCLASS SearchType, PVOID pvSearch if (!IS_IME_HKL(hKL)) { if (CtfImmIsTextFrameServiceDisabled() || !IS_CICERO_MODE() || IS_16BIT_MODE()) + { + TRACE("\n"); return FALSE; + } } SearchType = ImeInfoExKeyboardLayout; @@ -896,7 +954,10 @@ ImmGetImeInfoEx(PIMEINFOEX pImeInfoEx, IMEINFOEXCLASS SearchType, PVOID pvSearch else { if (!IS_IME_HKL(hKL)) + { + TRACE("\n"); return FALSE; + } } } else if (SearchType == ImeInfoExImeFileName) @@ -997,7 +1058,10 @@ BOOL WINAPI ImmLoadIME(HKL hKL) PIMEDPI pImeDpi; if (!IS_IME_HKL(hKL) && (!IS_CICERO_MODE() || IS_16BIT_MODE())) + { + TRACE("\n"); return FALSE; + } pImeDpi = Imm32FindImeDpi(hKL); if (pImeDpi == NULL) @@ -1023,9 +1087,15 @@ UINT WINAPI ImmGetDescriptionA(HKL hKL, LPSTR lpszDescription, UINT uBufLen) TRACE("(%p,%p,%d)\n", hKL, lpszDescription, uBufLen); - if (!ImmGetImeInfoEx(&info, ImeInfoExKeyboardLayout, &hKL) || !IS_IME_HKL(hKL)) + if (!IS_IME_HKL(hKL)) return 0; + if (!ImmGetImeInfoEx(&info, ImeInfoExKeyboardLayout, &hKL)) + { + ERR("\n"); + return 0; + } + StringCchLengthW(info.wszImeDescription, _countof(info.wszImeDescription), &cch); cch = WideCharToMultiByte(CP_ACP, 0, info.wszImeDescription, (INT)cch, lpszDescription, uBufLen, NULL, NULL); @@ -1044,9 +1114,15 @@ UINT WINAPI ImmGetDescriptionW(HKL hKL, LPWSTR lpszDescription, UINT uBufLen) TRACE("(%p, %p, %d)\n", hKL, lpszDescription, uBufLen); - if (!ImmGetImeInfoEx(&info, ImeInfoExKeyboardLayout, &hKL) || !IS_IME_HKL(hKL)) + if (!IS_IME_HKL(hKL)) return 0; + if (!ImmGetImeInfoEx(&info, ImeInfoExKeyboardLayout, &hKL)) + { + ERR("\n"); + return 0; + } + if (uBufLen != 0) StringCchCopyW(lpszDescription, uBufLen, info.wszImeDescription); @@ -1065,8 +1141,16 @@ UINT WINAPI ImmGetIMEFileNameA( HKL hKL, LPSTR lpszFileName, UINT uBufLen) TRACE("(%p, %p, %u)\n", hKL, lpszFileName, uBufLen); - if (!ImmGetImeInfoEx(&info, ImeInfoExKeyboardLayout, &hKL) || !IS_IME_HKL(hKL)) + if (!IS_IME_HKL(hKL)) + { + if (uBufLen > 0) + lpszFileName[0] = 0; + return 0; + } + + if (!ImmGetImeInfoEx(&info, ImeInfoExKeyboardLayout, &hKL)) { + ERR("\n"); if (uBufLen > 0) lpszFileName[0] = 0; return 0; @@ -1096,13 +1180,21 @@ UINT WINAPI ImmGetIMEFileNameW(HKL hKL, LPWSTR lpszFileName, UINT uBufLen) TRACE("(%p, %p, %u)\n", hKL, lpszFileName, uBufLen); - if (!ImmGetImeInfoEx(&info, ImeInfoExKeyboardLayout, &hKL) || !IS_IME_HKL(hKL)) + if (!IS_IME_HKL(hKL)) { if (uBufLen > 0) lpszFileName[0] = 0; return 0; } + if (!ImmGetImeInfoEx(&info, ImeInfoExKeyboardLayout, &hKL)) + { + ERR("\n"); + if (uBufLen > 0) + lpszFileName[0] = 0; + return 0; + } + StringCchLengthW(info.wszImeFile, _countof(info.wszImeFile), &cch); if (uBufLen == 0) return (UINT)cch; @@ -1128,8 +1220,14 @@ DWORD WINAPI ImmGetProperty(HKL hKL, DWORD fdwIndex) TRACE("(%p, %lu)\n", hKL, fdwIndex); + if (!IS_IME_HKL(hKL)) + return FALSE; + if (!ImmGetImeInfoEx(&ImeInfoEx, ImeInfoExKeyboardLayout, &hKL)) + { + ERR("\n"); return FALSE; + } if (fdwIndex == IGP_GETIMEVERSION) return ImeInfoEx.dwImeWinVersion; @@ -1137,7 +1235,7 @@ DWORD WINAPI ImmGetProperty(HKL hKL, DWORD fdwIndex) if (ImeInfoEx.fLoadFlag != 2) { pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (pImeDpi == NULL) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return FALSE; pImeInfo = &pImeDpi->ImeInfo; @@ -1177,10 +1275,10 @@ LRESULT WINAPI ImmEscapeA(HKL hKL, HIMC hIMC, UINT uSubFunc, LPVOID lpData) TRACE("(%p, %p, %u, %p)\n", hKL, hIMC, uSubFunc, lpData); pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return 0; - if (!ImeDpi_IsUnicode(pImeDpi) || !lpData) + if (!ImeDpi_IsUnicode(pImeDpi) || !lpData) /* No conversion needed */ { ret = ImeDpi_Escape(pImeDpi, hIMC, uSubFunc, lpData, hKL); ImmUnlockImeDpi(pImeDpi); @@ -1247,6 +1345,7 @@ LRESULT WINAPI ImmEscapeA(HKL hKL, HIMC hIMC, UINT uSubFunc, LPVOID lpData) } ImmUnlockImeDpi(pImeDpi); + TRACE("ret: %p\n", ret); return ret; } @@ -1265,10 +1364,10 @@ LRESULT WINAPI ImmEscapeW(HKL hKL, HIMC hIMC, UINT uSubFunc, LPVOID lpData) TRACE("(%p, %p, %u, %p)\n", hKL, hIMC, uSubFunc, lpData); pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return 0; - if (ImeDpi_IsUnicode(pImeDpi) || !lpData) + if (ImeDpi_IsUnicode(pImeDpi) || !lpData) /* No conversion needed */ { ret = ImeDpi_Escape(pImeDpi, hIMC, uSubFunc, lpData, hKL); ImmUnlockImeDpi(pImeDpi); @@ -1324,6 +1423,7 @@ LRESULT WINAPI ImmEscapeW(HKL hKL, HIMC hIMC, UINT uSubFunc, LPVOID lpData) } ImmUnlockImeDpi(pImeDpi); + TRACE("ret: %p\n", ret); return ret; } @@ -1337,16 +1437,16 @@ BOOL WINAPI ImmGetOpenStatus(HIMC hIMC) TRACE("(%p)\n", hIMC); - if (!hIMC) + if (IS_NULL_UNEXPECTEDLY(hIMC)) return FALSE; pIC = ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; ret = pIC->fOpen; - ImmUnlockIMC(hIMC); + TRACE("ret: %d\n", ret); return ret; } @@ -1362,11 +1462,11 @@ BOOL WINAPI ImmSetOpenStatus(HIMC hIMC, BOOL fOpen) TRACE("(%p, %d)\n", hIMC, fOpen); - if (Imm32IsCrossThreadAccess(hIMC)) + if (IS_CROSS_THREAD_HIMC(hIMC)) return FALSE; pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; if (pIC->fOpen != fOpen) @@ -1385,6 +1485,10 @@ BOOL WINAPI ImmSetOpenStatus(HIMC hIMC, BOOL fOpen) IMC_SETOPENSTATUS, IMN_SETOPENSTATUS, 0); NtUserNotifyIMEStatus(hWnd, fOpen, dwConversion); } + else + { + TRACE("No change.\n"); + } return TRUE; } @@ -1400,7 +1504,7 @@ BOOL WINAPI ImmGetStatusWindowPos(HIMC hIMC, LPPOINT lpptPos) TRACE("(%p, %p)\n", hIMC, lpptPos); pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; ret = !!(pIC->fdwInit & INIT_STATUSWNDPOS); @@ -1421,11 +1525,11 @@ BOOL WINAPI ImmSetStatusWindowPos(HIMC hIMC, LPPOINT lpptPos) TRACE("(%p, {%ld, %ld})\n", hIMC, lpptPos->x, lpptPos->y); - if (Imm32IsCrossThreadAccess(hIMC)) + if (IS_CROSS_THREAD_HIMC(hIMC)) return FALSE; pIC = ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; hWnd = pIC->hWnd; @@ -1450,7 +1554,7 @@ BOOL WINAPI ImmGetCompositionWindow(HIMC hIMC, LPCOMPOSITIONFORM lpCompForm) TRACE("(%p, %p)\n", hIMC, lpCompForm); pIC = ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; if (pIC->fdwInit & INIT_COMPFORM) @@ -1471,11 +1575,11 @@ BOOL WINAPI ImmSetCompositionWindow(HIMC hIMC, LPCOMPOSITIONFORM lpCompForm) LPINPUTCONTEXTDX pIC; HWND hWnd; - if (Imm32IsCrossThreadAccess(hIMC)) + if (IS_CROSS_THREAD_HIMC(hIMC)) return FALSE; pIC = (LPINPUTCONTEXTDX)ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; pIC->cfCompForm = *lpCompForm; @@ -1507,14 +1611,14 @@ BOOL WINAPI ImmGetCompositionFontA(HIMC hIMC, LPLOGFONTA lplf) TRACE("(%p, %p)\n", hIMC, lplf); pClientImc = ImmLockClientImc(hIMC); - if (pClientImc == NULL) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return FALSE; bWide = (pClientImc->dwFlags & CLIENTIMC_WIDE); ImmUnlockClientImc(pClientImc); pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; if (pIC->fdwInit & INIT_LOGFONT) @@ -1544,14 +1648,14 @@ BOOL WINAPI ImmGetCompositionFontW(HIMC hIMC, LPLOGFONTW lplf) TRACE("(%p, %p)\n", hIMC, lplf); pClientImc = ImmLockClientImc(hIMC); - if (pClientImc == NULL) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return FALSE; bWide = (pClientImc->dwFlags & CLIENTIMC_WIDE); ImmUnlockClientImc(pClientImc); pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; if (pIC->fdwInit & INIT_LOGFONT) @@ -1582,11 +1686,11 @@ BOOL WINAPI ImmSetCompositionFontA(HIMC hIMC, LPLOGFONTA lplf) TRACE("(%p, %p)\n", hIMC, lplf); - if (Imm32IsCrossThreadAccess(hIMC)) + if (IS_CROSS_THREAD_HIMC(hIMC)) return FALSE; pClientImc = ImmLockClientImc(hIMC); - if (pClientImc == NULL) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return FALSE; bWide = (pClientImc->dwFlags & CLIENTIMC_WIDE); @@ -1599,7 +1703,7 @@ BOOL WINAPI ImmSetCompositionFontA(HIMC hIMC, LPLOGFONTA lplf) } pIC = (LPINPUTCONTEXTDX)ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; if (GetWin32ClientInfo()->dwExpWinVer < _WIN32_WINNT_NT4) /* old version (3.x)? */ @@ -1638,11 +1742,11 @@ BOOL WINAPI ImmSetCompositionFontW(HIMC hIMC, LPLOGFONTW lplf) TRACE("(%p, %p)\n", hIMC, lplf); - if (Imm32IsCrossThreadAccess(hIMC)) + if (IS_CROSS_THREAD_HIMC(hIMC)) return FALSE; pClientImc = ImmLockClientImc(hIMC); - if (pClientImc == NULL) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return FALSE; bWide = (pClientImc->dwFlags & CLIENTIMC_WIDE); @@ -1655,7 +1759,7 @@ BOOL WINAPI ImmSetCompositionFontW(HIMC hIMC, LPLOGFONTW lplf) } pIC = (LPINPUTCONTEXTDX)ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; if (GetWin32ClientInfo()->dwExpWinVer < _WIN32_WINNT_NT4) /* old version (3.x)? */ @@ -1697,10 +1801,10 @@ ImmGetConversionListA(HKL hKL, HIMC hIMC, LPCSTR pSrc, LPCANDIDATELIST lpDst, lpDst, dwBufLen, uFlag); pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (pImeDpi == NULL) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return 0; - if (!ImeDpi_IsUnicode(pImeDpi)) + if (!ImeDpi_IsUnicode(pImeDpi)) /* No conversion needed */ { ret = pImeDpi->ImeConversionList(hIMC, pSrc, lpDst, dwBufLen, uFlag); ImmUnlockImeDpi(pImeDpi); @@ -1710,20 +1814,20 @@ ImmGetConversionListA(HKL hKL, HIMC hIMC, LPCSTR pSrc, LPCANDIDATELIST lpDst, if (pSrc) { pszSrcW = Imm32WideFromAnsi(pImeDpi->uCodePage, pSrc); - if (pszSrcW == NULL) + if (IS_NULL_UNEXPECTEDLY(pszSrcW)) goto Quit; } cb = pImeDpi->ImeConversionList(hIMC, pszSrcW, NULL, 0, uFlag); - if (cb == 0) + if (IS_ZERO_UNEXPECTEDLY(cb)) goto Quit; pCL = ImmLocalAlloc(0, cb); - if (pCL == NULL) + if (IS_NULL_UNEXPECTEDLY(pCL)) goto Quit; cb = pImeDpi->ImeConversionList(hIMC, pszSrcW, pCL, cb, uFlag); - if (cb == 0) + if (IS_ZERO_UNEXPECTEDLY(cb)) goto Quit; ret = CandidateListWideToAnsi(pCL, lpDst, dwBufLen, pImeDpi->uCodePage); @@ -1732,6 +1836,7 @@ Quit: ImmLocalFree(pszSrcW); ImmLocalFree(pCL); ImmUnlockImeDpi(pImeDpi); + TRACE("ret: 0x%X\n", ret); return ret; } @@ -1752,10 +1857,10 @@ ImmGetConversionListW(HKL hKL, HIMC hIMC, LPCWSTR pSrc, LPCANDIDATELIST lpDst, lpDst, dwBufLen, uFlag); pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return 0; - if (ImeDpi_IsUnicode(pImeDpi)) + if (ImeDpi_IsUnicode(pImeDpi)) /* No conversion needed */ { ret = pImeDpi->ImeConversionList(hIMC, pSrc, lpDst, dwBufLen, uFlag); ImmUnlockImeDpi(pImeDpi); @@ -1765,20 +1870,20 @@ ImmGetConversionListW(HKL hKL, HIMC hIMC, LPCWSTR pSrc, LPCANDIDATELIST lpDst, if (pSrc) { pszSrcA = Imm32AnsiFromWide(pImeDpi->uCodePage, pSrc); - if (pszSrcA == NULL) + if (IS_NULL_UNEXPECTEDLY(pszSrcA)) goto Quit; } cb = pImeDpi->ImeConversionList(hIMC, pszSrcA, NULL, 0, uFlag); - if (cb == 0) + if (IS_ZERO_UNEXPECTEDLY(cb)) goto Quit; pCL = ImmLocalAlloc(0, cb); - if (!pCL) + if (IS_NULL_UNEXPECTEDLY(pCL)) goto Quit; cb = pImeDpi->ImeConversionList(hIMC, pszSrcA, pCL, cb, uFlag); - if (!cb) + if (IS_ZERO_UNEXPECTEDLY(cb)) goto Quit; ret = CandidateListAnsiToWide(pCL, lpDst, dwBufLen, pImeDpi->uCodePage); @@ -1787,6 +1892,7 @@ Quit: ImmLocalFree(pszSrcA); ImmLocalFree(pCL); ImmUnlockImeDpi(pImeDpi); + TRACE("ret: 0x%X\n", ret); return ret; } @@ -1800,13 +1906,20 @@ BOOL WINAPI ImmGetConversionStatus(HIMC hIMC, LPDWORD lpfdwConversion, LPDWORD l TRACE("(%p %p %p)\n", hIMC, lpfdwConversion, lpfdwSentence); pIC = ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; if (lpfdwConversion) + { *lpfdwConversion = pIC->fdwConversion; + TRACE("0x%X\n", *lpfdwConversion); + } + if (lpfdwSentence) + { *lpfdwSentence = pIC->fdwSentence; + TRACE("0x%X\n", *lpfdwSentence); + } ImmUnlockIMC(hIMC); return TRUE; @@ -1829,11 +1942,11 @@ BOOL WINAPI ImmSetConversionStatus(HIMC hIMC, DWORD fdwConversion, DWORD fdwSent if (!IS_IME_HKL(hKL) && IS_CICERO_MODE() && !IS_16BIT_MODE()) fUseCicero = TRUE; - if (Imm32IsCrossThreadAccess(hIMC)) + if (IS_CROSS_THREAD_HIMC(hIMC)) return FALSE; pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; if (pIC->fdwConversion != fdwConversion) @@ -1883,31 +1996,31 @@ BOOL WINAPI ImmConfigureIMEA(HKL hKL, HWND hWnd, DWORD dwMode, LPVOID lpData) TRACE("(%p, %p, 0x%lX, %p)\n", hKL, hWnd, dwMode, lpData); - if (!ValidateHwnd(hWnd) || Imm32IsCrossProcessAccess(hWnd)) + if (IS_NULL_UNEXPECTEDLY(ValidateHwnd(hWnd)) || IS_CROSS_PROCESS_HWND(hWnd)) return FALSE; pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return FALSE; RtlZeroMemory(&RegWordW, sizeof(RegWordW)); if (!ImeDpi_IsUnicode(pImeDpi) || !lpData || dwMode != IME_CONFIG_REGISTERWORD) - goto DoIt; + goto DoIt; /* No conversion needed */ pRegWordA = lpData; if (pRegWordA->lpReading) { RegWordW.lpReading = Imm32WideFromAnsi(pImeDpi->uCodePage, pRegWordA->lpReading); - if (!RegWordW.lpReading) + if (IS_NULL_UNEXPECTEDLY(RegWordW.lpReading)) goto Quit; } if (pRegWordA->lpWord) { RegWordW.lpWord = Imm32WideFromAnsi(pImeDpi->uCodePage, pRegWordA->lpWord); - if (!RegWordW.lpWord) + if (IS_NULL_UNEXPECTEDLY(RegWordW.lpWord)) goto Quit; } @@ -1922,6 +2035,7 @@ Quit: ImmLocalFree(RegWordW.lpReading); ImmLocalFree(RegWordW.lpWord); ImmUnlockImeDpi(pImeDpi); + TRACE("ret: %d\n", ret); return ret; } @@ -1937,31 +2051,31 @@ BOOL WINAPI ImmConfigureIMEW(HKL hKL, HWND hWnd, DWORD dwMode, LPVOID lpData) TRACE("(%p, %p, 0x%lX, %p)\n", hKL, hWnd, dwMode, lpData); - if (!ValidateHwnd(hWnd) || Imm32IsCrossProcessAccess(hWnd)) + if (IS_NULL_UNEXPECTEDLY(ValidateHwnd(hWnd)) || IS_CROSS_PROCESS_HWND(hWnd)) return FALSE; pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return FALSE; RtlZeroMemory(&RegWordA, sizeof(RegWordA)); if (ImeDpi_IsUnicode(pImeDpi) || !lpData || dwMode != IME_CONFIG_REGISTERWORD) - goto DoIt; + goto DoIt; /* No conversion needed */ pRegWordW = lpData; if (pRegWordW->lpReading) { RegWordA.lpReading = Imm32AnsiFromWide(pImeDpi->uCodePage, pRegWordW->lpReading); - if (!RegWordA.lpReading) + if (IS_NULL_UNEXPECTEDLY(RegWordA.lpReading)) goto Quit; } if (pRegWordW->lpWord) { RegWordA.lpWord = Imm32AnsiFromWide(pImeDpi->uCodePage, pRegWordW->lpWord); - if (!RegWordA.lpWord) + if (IS_NULL_UNEXPECTEDLY(RegWordA.lpWord)) goto Quit; } @@ -1976,6 +2090,7 @@ Quit: ImmLocalFree(RegWordA.lpReading); ImmLocalFree(RegWordA.lpWord); ImmUnlockImeDpi(pImeDpi); + TRACE("ret: %d\n", ret); return ret; } @@ -2024,16 +2139,17 @@ BOOL WINAPI ImmWINNLSEnableIME(HWND hWnd, BOOL enable) } hIMC = (HIMC)NtUserGetThreadState(THREADSTATE_DEFAULTINPUTCONTEXT); - if (!hIMC) + if (IS_NULL_UNEXPECTEDLY(hIMC)) return FALSE; pClientImc = ImmLockClientImc(hIMC); - if (!pClientImc) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return FALSE; ret = !(pClientImc->dwFlags & CLIENTIMC_DISABLEIME); if (!!enable == ret) { + TRACE("Same\n"); ImmUnlockClientImc(pClientImc); return ret; } diff --git a/dll/win32/imm32/imm.c b/dll/win32/imm32/imm.c index d6e0cb1431f..117384111be 100644 --- a/dll/win32/imm32/imm.c +++ b/dll/win32/imm32/imm.c @@ -31,7 +31,10 @@ static BOOL APIENTRY ImmInitializeGlobals(HMODULE hMod) status = RtlInitializeCriticalSection(&gcsImeDpi); if (NT_ERROR(status)) + { + ERR("\n"); return FALSE; + } gfImmInitialized = TRUE; return TRUE; @@ -68,20 +71,14 @@ BOOL WINAPI ImmLoadLayout(HKL hKL, PIMEINFOEX pImeInfoEx) REGKEY_KEYBOARD_LAYOUTS, HandleToUlong(hKL)); error = RegOpenKeyExW(HKEY_LOCAL_MACHINE, szLayout, 0, KEY_READ, &hLayoutKey); - if (error) - { - ERR("RegOpenKeyExW: 0x%08lX\n", error); + if (IS_ERROR_UNEXPECTEDLY(error)) return FALSE; - } } else { error = RegOpenKeyExW(HKEY_LOCAL_MACHINE, REGKEY_IMM, 0, KEY_READ, &hLayoutKey); - if (error) - { - ERR("RegOpenKeyExW: 0x%08lX\n", error); + if (IS_ERROR_UNEXPECTEDLY(error)) return FALSE; - } } cbData = sizeof(pImeInfoEx->wszImeFile); @@ -93,9 +90,9 @@ BOOL WINAPI ImmLoadLayout(HKL hKL, PIMEINFOEX pImeInfoEx) pImeInfoEx->fLoadFlag = 0; - if (error != ERROR_SUCCESS || dwType != REG_SZ) + if (IS_ERROR_UNEXPECTEDLY(error) || dwType != REG_SZ) { - ERR("RegQueryValueExW: 0x%lX, 0x%lX\n", error, dwType); + ERR("\n"); return FALSE; } @@ -129,7 +126,7 @@ BOOL WINAPI ImmFreeLayout(DWORD dwUnknown) if (cKLs) { pList = ImmLocalAlloc(0, cKLs * sizeof(HKL)); - if (pList == NULL) + if (IS_NULL_UNEXPECTEDLY(pList)) return FALSE; cKLs = GetKeyboardLayoutList(cKLs, pList); @@ -147,7 +144,10 @@ BOOL WINAPI ImmFreeLayout(DWORD dwUnknown) StringCchPrintfW(szKBD, _countof(szKBD), L"%08X", LangID); if (!LoadKeyboardLayoutW(szKBD, KLF_ACTIVATE)) + { + WARN("\n"); LoadKeyboardLayoutW(L"00000409", KLF_ACTIVATE | 0x200); + } } else if (dwUnknown == 2) { @@ -187,7 +187,7 @@ VOID APIENTRY Imm32SelectInputContext(HKL hNewKL, HKL hOldKL, HIMC hIMC) PIME_STATE pNewState = NULL, pOldState = NULL; pClientImc = ImmLockClientImc(hIMC); - if (!pClientImc) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return; pNewImeDpi = ImmLockImeDpi(hNewKL); @@ -496,13 +496,16 @@ HIMC WINAPI ImmAssociateContext(HWND hWnd, HIMC hIMC) TRACE("(%p, %p)\n", hWnd, hIMC); if (!IS_IMM_MODE()) + { + WARN("\n"); return NULL; + } pWnd = ValidateHwnd(hWnd); - if (!pWnd) + if (IS_NULL_UNEXPECTEDLY(pWnd)) return NULL; - if (hIMC && Imm32IsCrossThreadAccess(hIMC)) + if (hIMC && IS_CROSS_THREAD_HIMC(hIMC)) return NULL; hOldIMC = pWnd->hImc; @@ -542,9 +545,12 @@ BOOL WINAPI ImmAssociateContextEx(HWND hWnd, HIMC hIMC, DWORD dwFlags) TRACE("(%p, %p, 0x%lX)\n", hWnd, hIMC, dwFlags); if (!IS_IMM_MODE()) + { + WARN("\n"); return FALSE; + } - if (hIMC && !(dwFlags & IACE_DEFAULT) && Imm32IsCrossThreadAccess(hIMC)) + if (hIMC && !(dwFlags & IACE_DEFAULT) && IS_CROSS_THREAD_HIMC(hIMC)) return FALSE; hwndFocus = (HWND)NtUserQueryWindow(hWnd, QUERY_WINDOW_FOCUS); @@ -587,14 +593,17 @@ HIMC WINAPI ImmCreateContext(void) TRACE("()\n"); if (!IS_IMM_MODE()) + { + WARN("\n"); return NULL; + } pClientImc = ImmLocalAlloc(HEAP_ZERO_MEMORY, sizeof(CLIENTIMC)); - if (pClientImc == NULL) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return NULL; hIMC = NtUserCreateInputContext((ULONG_PTR)pClientImc); - if (hIMC == NULL) + if (IS_NULL_UNEXPECTEDLY(hIMC)) { ImmLocalFree(pClientImc); return NULL; @@ -637,23 +646,35 @@ BOOL APIENTRY Imm32DestroyInputContext(HIMC hIMC, HKL hKL, BOOL bKeep) PCLIENTIMC pClientImc; PIMC pIMC; - if (!hIMC || !IS_IMM_MODE()) + if (hIMC == NULL) + return FALSE; + + if (!IS_IMM_MODE()) + { + WARN("\n"); return FALSE; + } pIMC = ValidateHandle(hIMC, TYPE_INPUTCONTEXT); - if (!pIMC || pIMC->head.pti != Imm32CurrentPti()) + if (IS_NULL_UNEXPECTEDLY(pIMC)) + return FALSE; + + if (pIMC->head.pti != Imm32CurrentPti()) { - ERR("invalid pIMC: %p\n", pIMC); + ERR("\n"); return FALSE; } pClientImc = (PCLIENTIMC)pIMC->dwClientImcData; - if (!pClientImc) + if (pClientImc == NULL) + { + TRACE("pClientImc == NULL\n"); goto Finish; + } if ((pClientImc->dwFlags & CLIENTIMC_UNKNOWN2) && !bKeep) { - ERR("CLIENTIMC_UNKNOWN2\n"); + ERR("\n"); return FALSE; } @@ -662,14 +683,13 @@ BOOL APIENTRY Imm32DestroyInputContext(HIMC hIMC, HKL hKL, BOOL bKeep) InterlockedIncrement(&pClientImc->cLockObj); - if (!pClientImc->hInputContext) + if (IS_NULL_UNEXPECTEDLY(pClientImc->hInputContext)) goto Quit; pIC = (LPINPUTCONTEXTDX)ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) { ImmUnlockClientImc(pClientImc); - ERR("!pIC\n"); return FALSE; } @@ -725,24 +745,29 @@ Imm32CreateInputContext(HIMC hIMC, LPINPUTCONTEXT pIC, PCLIENTIMC pClientImc, HK pIC->hCandInfo = ImmCreateIMCC(sizeof(CANDIDATEINFO)); pIC->hGuideLine = ImmCreateIMCC(sizeof(GUIDELINE)); pIC->hMsgBuf = ImmCreateIMCC(sizeof(UINT)); - if (!pIC->hCompStr || !pIC->hCandInfo || !pIC->hGuideLine || !pIC->hMsgBuf) + if (IS_NULL_UNEXPECTEDLY(pIC->hCompStr) || + IS_NULL_UNEXPECTEDLY(pIC->hCandInfo) || + IS_NULL_UNEXPECTEDLY(pIC->hGuideLine) || + IS_NULL_UNEXPECTEDLY(pIC->hMsgBuf)) + { goto Fail; + } /* Initialize IC components */ pCS = ImmLockIMCC(pIC->hCompStr); - if (!pCS) + if (IS_NULL_UNEXPECTEDLY(pCS)) goto Fail; pCS->dwSize = sizeof(COMPOSITIONSTRING); ImmUnlockIMCC(pIC->hCompStr); pCI = ImmLockIMCC(pIC->hCandInfo); - if (!pCI) + if (IS_NULL_UNEXPECTEDLY(pCI)) goto Fail; pCI->dwSize = sizeof(CANDIDATEINFO); ImmUnlockIMCC(pIC->hCandInfo); pGL = ImmLockIMCC(pIC->hGuideLine); - if (!pGL) + if (IS_NULL_UNEXPECTEDLY(pGL)) goto Fail; pGL->dwSize = sizeof(GUIDELINE); ImmUnlockIMCC(pIC->hGuideLine); @@ -772,7 +797,7 @@ Imm32CreateInputContext(HIMC hIMC, LPINPUTCONTEXT pIC, PCLIENTIMC pClientImc, HK /* Create private data */ pIC->hPrivate = ImmCreateIMCC(cbPrivate); - if (!pIC->hPrivate) + if (IS_NULL_UNEXPECTEDLY(pIC->hPrivate)) goto Fail; CtfImmTIMCreateInputContext(hIMC); @@ -819,7 +844,7 @@ LPINPUTCONTEXT APIENTRY Imm32InternalLockIMC(HIMC hIMC, BOOL fSelect) PIMEDPI pImeDpi = NULL; pClientImc = ImmLockClientImc(hIMC); - if (!pClientImc) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return NULL; RtlEnterCriticalSection(&pClientImc->cs); @@ -827,15 +852,11 @@ LPINPUTCONTEXT APIENTRY Imm32InternalLockIMC(HIMC hIMC, BOOL fSelect) if (pClientImc->hInputContext) { pIC = LocalLock(pClientImc->hInputContext); - if (pIC) - { - CtfImmTIMCreateInputContext(hIMC); - goto Success; - } - else - { + if (IS_NULL_UNEXPECTEDLY(pIC)) goto Failure; - } + + CtfImmTIMCreateInputContext(hIMC); + goto Success; } dwThreadId = (DWORD)NtUserQueryInputContext(hIMC, QIC_INPUTTHREADID); @@ -853,11 +874,14 @@ LPINPUTCONTEXT APIENTRY Imm32InternalLockIMC(HIMC hIMC, BOOL fSelect) } if (!NtUserQueryInputContext(hIMC, QIC_DEFAULTWINDOWIME)) + { + ERR("\n"); goto Failure; + } hIC = LocalAlloc(LHND, sizeof(INPUTCONTEXTDX)); pIC = LocalLock(hIC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) { LocalFree(hIC); goto Failure; @@ -867,6 +891,7 @@ LPINPUTCONTEXT APIENTRY Imm32InternalLockIMC(HIMC hIMC, BOOL fSelect) hNewKL = GetKeyboardLayout(dwThreadId); if (!Imm32CreateInputContext(hIMC, pIC, pClientImc, hNewKL, fSelect)) { + ERR("\n"); LocalUnlock(hIC); pClientImc->hInputContext = LocalFree(hIC); goto Failure; @@ -894,9 +919,12 @@ BOOL WINAPI ImmDestroyContext(HIMC hIMC) TRACE("(%p)\n", hIMC); if (!IS_IMM_MODE()) + { + WARN("\n"); return FALSE; + } - if (Imm32IsCrossThreadAccess(hIMC)) + if (IS_CROSS_THREAD_HIMC(hIMC)) return FALSE; hKL = GetKeyboardLayout(0); @@ -913,11 +941,11 @@ PCLIENTIMC WINAPI ImmLockClientImc(HIMC hImc) TRACE("(%p)\n", hImc); - if (!hImc) + if (IS_NULL_UNEXPECTEDLY(hImc)) return NULL; pIMC = ValidateHandle(hImc, TYPE_INPUTCONTEXT); - if (!pIMC || !Imm32CheckImcProcess(pIMC)) + if (IS_NULL_UNEXPECTEDLY(pIMC) || !Imm32CheckImcProcess(pIMC)) return NULL; pClientImc = (PCLIENTIMC)pIMC->dwClientImcData; @@ -929,7 +957,7 @@ PCLIENTIMC WINAPI ImmLockClientImc(HIMC hImc) } pClientImc = ImmLocalAlloc(HEAP_ZERO_MEMORY, sizeof(CLIENTIMC)); - if (!pClientImc) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return NULL; RtlInitializeCriticalSection(&pClientImc->cs); @@ -937,6 +965,7 @@ PCLIENTIMC WINAPI ImmLockClientImc(HIMC hImc) if (!NtUserUpdateInputContext(hImc, UIC_CLIENTIMCDATA, (DWORD_PTR)pClientImc)) { + ERR("\n"); ImmLocalFree(pClientImc); return NULL; } @@ -978,7 +1007,10 @@ static HIMC APIENTRY ImmGetSaveContext(HWND hWnd, DWORD dwContextFlags) PWND pWnd; if (!IS_IMM_MODE()) + { + TRACE("Not IMM mode.\n"); return NULL; + } if (!hWnd) { @@ -987,7 +1019,7 @@ static HIMC APIENTRY ImmGetSaveContext(HWND hWnd, DWORD dwContextFlags) } pWnd = ValidateHwnd(hWnd); - if (!pWnd || Imm32IsCrossProcessAccess(hWnd)) + if (IS_NULL_UNEXPECTEDLY(pWnd) || IS_CROSS_PROCESS_HWND(hWnd)) return NULL; hIMC = pWnd->hImc; @@ -996,10 +1028,12 @@ static HIMC APIENTRY ImmGetSaveContext(HWND hWnd, DWORD dwContextFlags) Quit: pClientImc = ImmLockClientImc(hIMC); - if (pClientImc == NULL) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return NULL; + if ((dwContextFlags & 2) && (pClientImc->dwFlags & CLIENTIMC_DISABLEIME)) hIMC = NULL; + ImmUnlockClientImc(pClientImc); return hIMC; } @@ -1010,7 +1044,7 @@ Quit: HIMC WINAPI ImmGetContext(HWND hWnd) { TRACE("(%p)\n", hWnd); - if (hWnd == NULL) + if (IS_NULL_UNEXPECTEDLY(hWnd)) return NULL; return ImmGetSaveContext(hWnd, 2); } @@ -1034,7 +1068,7 @@ BOOL WINAPI ImmUnlockIMC(HIMC hIMC) PCLIENTIMC pClientImc; pClientImc = ImmLockClientImc(hIMC); - if (pClientImc == NULL) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return FALSE; if (pClientImc->hInputContext) @@ -1108,7 +1142,7 @@ BOOL WINAPI ImmEnumInputContext(DWORD dwThreadId, IMCENUMPROC lpfn, LPARAM lPara TRACE("(%lu, %p, %p)\n", dwThreadId, lpfn, lParam); dwCount = Imm32BuildHimcList(dwThreadId, &phList); - if (!dwCount) + if (IS_ZERO_UNEXPECTEDLY(dwCount)) return FALSE; for (dwIndex = 0; dwIndex < dwCount; ++dwIndex) @@ -1140,7 +1174,10 @@ BOOL WINAPI ImmSetActiveContext(HWND hWnd, HIMC hIMC, BOOL fActive) TRACE("(%p, %p, %d)\n", hWnd, hIMC, fActive); if (!IS_IMM_MODE()) + { + WARN("\n"); return FALSE; + } pClientImc = ImmLockClientImc(hIMC); @@ -1151,11 +1188,11 @@ BOOL WINAPI ImmSetActiveContext(HWND hWnd, HIMC hIMC, BOOL fActive) } else if (hIMC) { - if (!pClientImc) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return FALSE; pIC = (LPINPUTCONTEXTDX)ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) { ImmUnlockClientImc(pClientImc); return FALSE; @@ -1241,10 +1278,31 @@ BOOL WINAPI ImmSetActiveContextConsoleIME(HWND hwnd, BOOL fFlag) TRACE("(%p, %d)\n", hwnd, fFlag); hIMC = ImmGetContext(hwnd); - if (hIMC) - return ImmSetActiveContext(hwnd, hIMC, fFlag); - return FALSE; + if (IS_NULL_UNEXPECTEDLY(hIMC)) + return FALSE; + return ImmSetActiveContext(hwnd, hIMC, fFlag); +} + +#ifndef NDEBUG +VOID APIENTRY Imm32UnitTest(VOID) +{ + if (0) + { + DWORD dwValue; + WCHAR szText[64]; + + Imm32StrToUInt(L"123", &dwValue, 10); + ASSERT(dwValue == 123); + Imm32StrToUInt(L"100", &dwValue, 16); + ASSERT(dwValue == 0x100); + + Imm32UIntToStr(123, 10, szText, _countof(szText)); + ASSERT(lstrcmpW(szText, L"123") == 0); + Imm32UIntToStr(0x100, 16, szText, _countof(szText)); + ASSERT(lstrcmpW(szText, L"100") == 0); + } } +#endif BOOL WINAPI User32InitializeImmEntryTable(DWORD); @@ -1273,6 +1331,9 @@ ImmDllInitialize( ERR("User32InitializeImmEntryTable failed\n"); return FALSE; } +#ifndef NDEBUG + Imm32UnitTest(); +#endif break; case DLL_THREAD_ATTACH: diff --git a/dll/win32/imm32/keymsg.c b/dll/win32/imm32/keymsg.c index 11198aa545a..8e1cac90154 100644 --- a/dll/win32/imm32/keymsg.c +++ b/dll/win32/imm32/keymsg.c @@ -69,7 +69,7 @@ BOOL APIENTRY Imm32CImeNonImeToggle(HIMC hIMC, HKL hKL, HWND hWnd, LANGID LangID LPINPUTCONTEXT pIC; BOOL fOpen; - if (hWnd == NULL) + if (IS_NULL_UNEXPECTEDLY(hWnd)) return FALSE; if (LOWORD(hKL) != LangID || !ImmIsIME(hKL)) @@ -79,7 +79,7 @@ BOOL APIENTRY Imm32CImeNonImeToggle(HIMC hIMC, HKL hKL, HWND hWnd, LANGID LangID } pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return TRUE; fOpen = pIC->fOpen; @@ -105,7 +105,7 @@ BOOL APIENTRY Imm32CShapeToggle(HIMC hIMC, HKL hKL, HWND hWnd) return FALSE; pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return TRUE; fOpen = pIC->fOpen; @@ -136,7 +136,7 @@ BOOL APIENTRY Imm32CSymbolToggle(HIMC hIMC, HKL hKL, HWND hWnd) return FALSE; pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return TRUE; fOpen = pIC->fOpen; @@ -191,7 +191,7 @@ BOOL APIENTRY Imm32KShapeToggle(HIMC hIMC) DWORD dwConversion, dwSentence; pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; dwConversion = (pIC->fdwConversion ^ IME_CMODE_FULLSHAPE); @@ -214,7 +214,7 @@ BOOL APIENTRY Imm32KHanjaConvert(HIMC hIMC) DWORD dwConversion, dwSentence; pIC = ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; dwConversion = (pIC->fdwConversion ^ IME_CMODE_HANJACONVERT); @@ -233,7 +233,7 @@ BOOL APIENTRY Imm32KEnglish(HIMC hIMC) BOOL fOpen; pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; dwConversion = (pIC->fdwConversion ^ IME_CMODE_NATIVE); @@ -253,7 +253,7 @@ BOOL APIENTRY Imm32ProcessHotKey(HWND hWnd, HIMC hIMC, HKL hKL, DWORD dwHotKeyID PIMEDPI pImeDpi; BOOL ret; - if (hIMC && Imm32IsCrossThreadAccess(hIMC)) + if (hIMC && IS_CROSS_THREAD_HIMC(hIMC)) return FALSE; switch (dwHotKeyID) @@ -289,6 +289,7 @@ BOOL APIENTRY Imm32ProcessHotKey(HWND hWnd, HIMC hIMC, HKL hKL, DWORD dwHotKeyID return Imm32CSymbolToggle(hIMC, hKL, hWnd); default: + WARN("0x%X\n", dwHotKeyID); break; } @@ -296,7 +297,7 @@ BOOL APIENTRY Imm32ProcessHotKey(HWND hWnd, HIMC hIMC, HKL hKL, DWORD dwHotKeyID return FALSE; pImeDpi = ImmLockImeDpi(hKL); - if (pImeDpi == NULL) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return FALSE; ret = (BOOL)pImeDpi->ImeEscape(hIMC, IME_ESC_PRIVATE_HOTKEY, &dwHotKeyID); @@ -318,7 +319,7 @@ ImmIsUIMessageAW(HWND hWndIME, UINT msg, WPARAM wParam, LPARAM lParam, BOOL bAns return FALSE; } - if (!hWndIME) + if (IS_NULL_UNEXPECTEDLY(hWndIME)) return TRUE; if (bAnsi) @@ -369,7 +370,7 @@ LRESULT APIENTRY CtfImmSetLangBand(HWND hWnd, BOOL fSet) if (hWnd && gpsi) pWnd = ValidateHwndNoErr(hWnd); - if (!pWnd) + if (IS_NULL_UNEXPECTEDLY(pWnd)) return 0; if (pWnd->state2 & WNDS2_WMCREATEMSGPROCESSED) @@ -380,7 +381,7 @@ LRESULT APIENTRY CtfImmSetLangBand(HWND hWnd, BOOL fSet) } pSetBand = ImmLocalAlloc(0, sizeof(IMM_DELAY_SET_LANG_BAND)); - if (!pSetBand) + if (IS_NULL_UNEXPECTEDLY(pSetBand)) return 0; pSetBand->hWnd = hWnd; @@ -399,12 +400,17 @@ static BOOL CALLBACK Imm32SendNotificationProc(HIMC hIMC, LPARAM lParam) LPINPUTCONTEXTDX pIC; pIC = (LPINPUTCONTEXTDX)ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) return TRUE; hWnd = pIC->hWnd; - if (!IsWindow(hWnd)) + if (hWnd == NULL || !IsWindow(hWnd)) + { + ERR("\n"); goto Quit; + } + + TRACE("dwChange: 0x%08X\n", pIC->dwChange); if (pIC->dwChange & INPUTCONTEXTDX_CHANGE_OPEN) SendMessageW(hWnd, WM_IME_NOTIFY, IMN_SETOPENSTATUS, 0); @@ -426,17 +432,11 @@ BOOL APIENTRY Imm32SendNotification(BOOL bProcess) return ImmEnumInputContext((bProcess ? -1 : 0), Imm32SendNotificationProc, 0); } -VOID APIENTRY Imm32RequestError(DWORD dwError) -{ - FIXME("()\n"); - SetLastError(dwError); -} - LRESULT APIENTRY Imm32ProcessRequest(HIMC hIMC, PWND pWnd, DWORD dwCommand, LPVOID pData, BOOL bAnsiAPI) { HWND hWnd; - DWORD ret = 0, dwCharPos, cchCompStr; + DWORD ret = 0, dwCharPos, cchCompStr, dwSize; LPVOID pCS, pTempData = pData; LPRECONVERTSTRING pRS; LPIMECHARPOSITION pICP; @@ -456,10 +456,17 @@ Imm32ProcessRequest(HIMC hIMC, PWND pWnd, DWORD dwCommand, LPVOID pData, BOOL bA }; if (dwCommand == 0 || dwCommand > IMR_DOCUMENTFEED) + { + ERR("\n"); return 0; /* Out of range */ + } - if (pData && IsBadWritePtr(pData, acbData[bAnsiAPI * 7 + dwCommand - 1])) + dwSize = acbData[bAnsiAPI * 7 + dwCommand - 1]; + if (pData && IsBadWritePtr(pData, dwSize)) + { + ERR("\n"); return 0; /* Invalid pointer */ + } /* Sanity check */ switch (dwCommand) @@ -468,7 +475,7 @@ Imm32ProcessRequest(HIMC hIMC, PWND pWnd, DWORD dwCommand, LPVOID pData, BOOL bA pRS = pData; if (pRS && (pRS->dwVersion != 0 || pRS->dwSize < sizeof(RECONVERTSTRING))) { - Imm32RequestError(ERROR_INVALID_PARAMETER); + ERR("\n"); return 0; } break; @@ -477,7 +484,7 @@ Imm32ProcessRequest(HIMC hIMC, PWND pWnd, DWORD dwCommand, LPVOID pData, BOOL bA pRS = pData; if (!pRS || pRS->dwVersion != 0) { - Imm32RequestError(ERROR_INVALID_PARAMETER); + ERR("\n"); return 0; } break; @@ -485,7 +492,7 @@ Imm32ProcessRequest(HIMC hIMC, PWND pWnd, DWORD dwCommand, LPVOID pData, BOOL bA default: if (!pData) { - Imm32RequestError(ERROR_INVALID_PARAMETER); + ERR("\n"); return 0; } break; @@ -503,18 +510,20 @@ Imm32ProcessRequest(HIMC hIMC, PWND pWnd, DWORD dwCommand, LPVOID pData, BOOL bA { case IMR_COMPOSITIONFONT: if (bAnsiAPI == bAnsiWnd) - goto DoIt; + goto DoIt; /* No conversion needed */ + if (bAnsiWnd) pTempData = ImmLocalAlloc(0, sizeof(LOGFONTA)); else pTempData = ImmLocalAlloc(0, sizeof(LOGFONTW)); - if (!pTempData) + + if (IS_NULL_UNEXPECTEDLY(pTempData)) return 0; break; case IMR_RECONVERTSTRING: case IMR_CONFIRMRECONVERTSTRING: case IMR_DOCUMENTFEED: if (bAnsiAPI == bAnsiWnd || !pData) - goto DoIt; + goto DoIt; /* No conversion needed */ if (bAnsiWnd) ret = Imm32ReconvertAnsiFromWide(NULL, pData, uCodePage); @@ -522,7 +531,7 @@ Imm32ProcessRequest(HIMC hIMC, PWND pWnd, DWORD dwCommand, LPVOID pData, BOOL bA ret = Imm32ReconvertWideFromAnsi(NULL, pData, uCodePage); pTempData = ImmLocalAlloc(0, ret + sizeof(WCHAR)); - if (!pTempData) + if (IS_NULL_UNEXPECTEDLY(pTempData)) return 0; pRS = pTempData; @@ -540,7 +549,7 @@ Imm32ProcessRequest(HIMC hIMC, PWND pWnd, DWORD dwCommand, LPVOID pData, BOOL bA case IMR_QUERYCHARPOSITION: if (bAnsiAPI == bAnsiWnd) - goto DoIt; + goto DoIt; /* No conversion needed */ pICP = pData; dwCharPos = pICP->dwCharPos; @@ -548,11 +557,11 @@ Imm32ProcessRequest(HIMC hIMC, PWND pWnd, DWORD dwCommand, LPVOID pData, BOOL bA if (bAnsiAPI) { cchCompStr = ImmGetCompositionStringA(hIMC, GCS_COMPSTR, NULL, 0); - if (!cchCompStr) + if (IS_ZERO_UNEXPECTEDLY(cchCompStr)) return 0; pCS = ImmLocalAlloc(0, (cchCompStr + 1) * sizeof(CHAR)); - if (!pCS) + if (IS_NULL_UNEXPECTEDLY(pCS)) return 0; ImmGetCompositionStringA(hIMC, GCS_COMPSTR, pCS, cchCompStr); @@ -561,11 +570,11 @@ Imm32ProcessRequest(HIMC hIMC, PWND pWnd, DWORD dwCommand, LPVOID pData, BOOL bA else { cchCompStr = ImmGetCompositionStringW(hIMC, GCS_COMPSTR, NULL, 0); - if (!cchCompStr) + if (IS_ZERO_UNEXPECTEDLY(cchCompStr)) return 0; pCS = ImmLocalAlloc(0, (cchCompStr + 1) * sizeof(WCHAR)); - if (!pCS) + if (IS_NULL_UNEXPECTEDLY(pCS)) return 0; ImmGetCompositionStringW(hIMC, GCS_COMPSTR, pCS, cchCompStr); @@ -576,6 +585,7 @@ Imm32ProcessRequest(HIMC hIMC, PWND pWnd, DWORD dwCommand, LPVOID pData, BOOL bA break; default: + WARN("\n"); break; } @@ -588,7 +598,7 @@ DoIt: ret = SendMessageW(hWnd, WM_IME_REQUEST, dwCommand, (LPARAM)pTempData); if (bAnsiAPI == bAnsiWnd) - goto Quit; + goto Quit; /* No conversion needed */ /* Get back to caller */ switch (dwCommand) @@ -624,6 +634,7 @@ DoIt: break; default: + WARN("\n"); break; } @@ -641,11 +652,11 @@ LRESULT APIENTRY ImmRequestMessageAW(HIMC hIMC, WPARAM wParam, LPARAM lParam, BO HWND hWnd; PWND pWnd = NULL; - if (!hIMC || Imm32IsCrossThreadAccess(hIMC)) + if (IS_NULL_UNEXPECTEDLY(hIMC) || IS_CROSS_THREAD_HIMC(hIMC)) return FALSE; pIC = ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; hWnd = pIC->hWnd; @@ -733,7 +744,7 @@ UINT WINAPI ImmGetVirtualKey(HWND hWnd) hIMC = ImmGetContext(hWnd); pIC = (LPINPUTCONTEXTDX)ImmLockIMC(hIMC); - if (!pIC) + if (IS_NULL_UNEXPECTEDLY(pIC)) return ret; if (pIC->bNeedsTrans) @@ -749,7 +760,7 @@ DWORD WINAPI ImmGetAppCompatFlags(HIMC hIMC) DWORD dwFlags; pClientIMC = ImmLockClientImc(hIMC); - if (pClientIMC == NULL) + if (IS_NULL_UNEXPECTEDLY(pClientIMC)) return 0; dwFlags = pClientIMC->dwCompatFlags; @@ -833,8 +844,12 @@ ImmProcessKey(HWND hWnd, HKL hKL, UINT vKey, LPARAM lParam, DWORD dwHotKeyID) { /* The key has been processed by IME's ImeProcessKey */ LANGID wLangID = LANGIDFROMLCID(GetSystemDefaultLCID()); - if (PRIMARYLANGID(wLangID) != LANG_KOREAN || /* Korean doesn't wanna this code */ - (vKey != VK_PROCESSKEY && !(ret & IPHK_HOTKEY))) /* Is the key to be processed? */ + if (PRIMARYLANGID(wLangID) == LANG_KOREAN && + (vKey == VK_PROCESSKEY || (ret & IPHK_HOTKEY))) + { + /* Korean don't want VK_PROCESSKEY and IME hot-keys */ + } + else { /* Add WM_KEYDOWN:VK_PROCESSKEY message */ ImmTranslateMessage(hWnd, WM_KEYDOWN, VK_PROCESSKEY, lParam); @@ -844,7 +859,7 @@ ImmProcessKey(HWND hWnd, HKL hKL, UINT vKey, LPARAM lParam, DWORD dwHotKeyID) } ImmReleaseContext(hWnd, hIMC); - return ret; + return ret; /* Returns IPHK_... flags */ } /*********************************************************************** @@ -869,6 +884,7 @@ LRESULT WINAPI ImmSystemHandler(HIMC hIMC, WPARAM wParam, LPARAM lParam) return CtfImmSetLangBand((HWND)lParam, (wParam == IMS_SETLANGBAND)); default: + WARN("\n"); return 0; } } @@ -888,18 +904,18 @@ BOOL WINAPI ImmGenerateMessage(HIMC hIMC) TRACE("(%p)\n", hIMC); - if (Imm32IsCrossThreadAccess(hIMC)) + if (IS_CROSS_THREAD_HIMC(hIMC)) return FALSE; pClientImc = ImmLockClientImc(hIMC); - if (pClientImc == NULL) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return FALSE; bAnsi = !(pClientImc->dwFlags & CLIENTIMC_WIDE); ImmUnlockClientImc(pClientImc); pIC = ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return FALSE; dwCount = pIC->dwNumMsgBuf; @@ -908,12 +924,12 @@ BOOL WINAPI ImmGenerateMessage(HIMC hIMC) hMsgBuf = pIC->hMsgBuf; pMsgs = ImmLockIMCC(hMsgBuf); - if (pMsgs == NULL) + if (IS_NULL_UNEXPECTEDLY(pMsgs)) goto Quit; cbTrans = dwCount * sizeof(TRANSMSG); pTrans = ImmLocalAlloc(0, cbTrans); - if (pTrans == NULL) + if (IS_NULL_UNEXPECTEDLY(pTrans)) goto Quit; RtlCopyMemory(pTrans, pMsgs, cbTrans); @@ -962,7 +978,7 @@ ImmPostMessages(HWND hwnd, HIMC hIMC, DWORD dwCount, LPTRANSMSG lpTransMsg) BOOL bAnsi; pClientImc = ImmLockClientImc(hIMC); - if (pClientImc == NULL) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return; bAnsi = !(pClientImc->dwFlags & CLIENTIMC_WIDE); @@ -1043,7 +1059,7 @@ BOOL WINAPI ImmTranslateMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lKeyD hIMC = ImmGetContext(hwnd); pIC = (LPINPUTCONTEXTDX)ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) { ImmReleaseContext(hwnd, hIMC); return FALSE; @@ -1071,11 +1087,14 @@ BOOL WINAPI ImmTranslateMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lKeyD dwThreadId = GetWindowThreadProcessId(hwnd, NULL); hKL = GetKeyboardLayout(dwThreadId); pImeDpi = ImmLockImeDpi(hKL); - if (pImeDpi == NULL) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) goto Quit; if (!GetKeyboardState(abKeyState)) /* get keyboard ON/OFF status */ + { + WARN("\n"); goto Quit; + } /* convert a virtual key if IME_PROP_KBD_CHAR_FIRST */ vk = pIC->nVKey; @@ -1110,7 +1129,7 @@ BOOL WINAPI ImmTranslateMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lKeyD /* allocate a list */ cbList = offsetof(TRANSMSGLIST, TransMsg) + MSG_COUNT * sizeof(TRANSMSG); pList = ImmLocalAlloc(0, cbList); - if (!pList) + if (IS_NULL_UNEXPECTEDLY(pList)) goto Quit; /* use IME conversion engine and convert the list */ @@ -1128,7 +1147,7 @@ BOOL WINAPI ImmTranslateMessage(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lKeyD else { pTransMsg = ImmLockIMCC(pIC->hMsgBuf); - if (pTransMsg == NULL) + if (IS_NULL_UNEXPECTEDLY(pTransMsg)) goto Quit; ImmPostMessages(hwnd, hIMC, kret, pTransMsg); ImmUnlockIMCC(pIC->hMsgBuf); @@ -1139,6 +1158,7 @@ Quit: ImmUnlockImeDpi(pImeDpi); ImmUnlockIMC(hIMC); ImmReleaseContext(hwnd, hIMC); + TRACE("ret: %d\n", ret); return ret; #undef MSG_COUNT } @@ -1173,7 +1193,7 @@ ImmSendMessageToActiveDefImeWndW(UINT uMsg, WPARAM wParam, LPARAM lParam) return 0; hwndIME = (HWND)NtUserQueryWindow((HWND)wParam, QUERY_WINDOW_DEFAULT_IME); - if (!hwndIME) + if (IS_NULL_UNEXPECTEDLY(hwndIME)) return 0; return SendMessageW(hwndIME, uMsg, wParam, lParam); diff --git a/dll/win32/imm32/precomp.h b/dll/win32/imm32/precomp.h index 8e996ab4ab0..629eaba4e4b 100644 --- a/dll/win32/imm32/precomp.h +++ b/dll/win32/imm32/precomp.h @@ -7,7 +7,7 @@ * Copyright 2017 James Tabor <james.tabor(a)reactos.org> * Copyright 2018 Amine Khaldi <amine.khaldi(a)reactos.org> * Copyright 2020 Oleg Dubinskiy <oleg.dubinskij2013(a)yandex.ua> - * Copyright 2020-2021 Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> + * Copyright 2020-2022 Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> */ #pragma once @@ -96,25 +96,48 @@ PIMEDPI APIENTRY Imm32FindOrLoadImeDpi(HKL hKL); LPINPUTCONTEXT APIENTRY Imm32InternalLockIMC(HIMC hIMC, BOOL fSelect); BOOL APIENTRY Imm32ReleaseIME(HKL hKL); BOOL APIENTRY Imm32IsSystemJapaneseOrKorean(VOID); - -static inline BOOL Imm32IsCrossThreadAccess(HIMC hIMC) -{ - DWORD dwImeThreadId = (DWORD)NtUserQueryInputContext(hIMC, QIC_INPUTTHREADID); - DWORD dwThreadId = GetCurrentThreadId(); - return (dwImeThreadId != dwThreadId); -} - -// Win: TestWindowProcess -static inline BOOL Imm32IsCrossProcessAccess(HWND hWnd) -{ - return (NtUserQueryWindow(hWnd, QUERY_WINDOW_UNIQUE_PROCESS_ID) != - (DWORD_PTR)NtCurrentTeb()->ClientId.UniqueProcess); -} - +BOOL APIENTRY Imm32IsCrossThreadAccess(HIMC hIMC); +BOOL APIENTRY Imm32IsCrossProcessAccess(HWND hWnd); BOOL WINAPI Imm32IsImcAnsi(HIMC hIMC); -#define ImeDpi_IsUnicode(pImeDpi) ((pImeDpi)->ImeInfo.fdwProperty & IME_PROP_UNICODE) -#define IS_16BIT_MODE() (GetWin32ClientInfo()->dwTIFlags & TIF_16BIT) +/* + * Unexpected Condition Checkers + * --- Examine the condition, and then generate trace log if necessary. + */ +#ifdef NDEBUG /* on Release */ +#define IS_NULL_UNEXPECTEDLY(p) (!(p)) +#define IS_ZERO_UNEXPECTEDLY(p) (!(p)) +#define IS_TRUE_UNEXPECTEDLY(x) (x) +#define IS_FALSE_UNEXPECTEDLY(x) (!(x)) +#define IS_ERROR_UNEXPECTEDLY(x) (!(x)) +#else /* on Debug */ +#define IS_NULL_UNEXPECTEDLY(p) \ + (!(p) ? (ros_dbg_log(__WINE_DBCL_ERR, __wine_dbch___default, \ + __FILE__, __FUNCTION__, __LINE__, "%s was NULL\n", #p), TRUE) \ + : FALSE) +#define IS_ZERO_UNEXPECTEDLY(p) \ + (!(p) ? (ros_dbg_log(__WINE_DBCL_ERR, __wine_dbch___default, \ + __FILE__, __FUNCTION__, __LINE__, "%s was zero\n", #p), TRUE) \ + : FALSE) +#define IS_TRUE_UNEXPECTEDLY(x) \ + ((x) ? (ros_dbg_log(__WINE_DBCL_ERR, __wine_dbch___default, \ + __FILE__, __FUNCTION__, __LINE__, "%s was non-zero\n", #x), TRUE) \ + : FALSE) +#define IS_FALSE_UNEXPECTEDLY(x) \ + ((!(x)) ? (ros_dbg_log(__WINE_DBCL_ERR, __wine_dbch___default, \ + __FILE__, __FUNCTION__, __LINE__, "%s was FALSE\n", #x), TRUE) \ + : FALSE) +#define IS_ERROR_UNEXPECTEDLY(x) \ + ((x) != ERROR_SUCCESS ? (ros_dbg_log(__WINE_DBCL_ERR, __wine_dbch___default, \ + __FILE__, __FUNCTION__, __LINE__, \ + "%s was 0x%X\n", #x, (x)), TRUE) \ + : FALSE) +#endif + +#define IS_CROSS_THREAD_HIMC(hIMC) IS_TRUE_UNEXPECTEDLY(Imm32IsCrossThreadAccess(hIMC)) +#define IS_CROSS_PROCESS_HWND(hWnd) IS_TRUE_UNEXPECTEDLY(Imm32IsCrossProcessAccess(hWnd)) +#define ImeDpi_IsUnicode(pImeDpi) ((pImeDpi)->ImeInfo.fdwProperty & IME_PROP_UNICODE) +#define IS_16BIT_MODE() (GetWin32ClientInfo()->dwTIFlags & TIF_16BIT) DWORD APIENTRY CandidateListWideToAnsi(const CANDIDATELIST *pWideCL, LPCANDIDATELIST pAnsiCL, DWORD dwBufLen, @@ -156,14 +179,7 @@ UINT APIENTRY Imm32GetImeLayout(PREG_IME pLayouts, UINT cLayouts); BOOL APIENTRY Imm32WriteImeLayout(HKL hKL, LPCWSTR pchFilePart, LPCWSTR pszLayoutText); HKL APIENTRY Imm32AssignNewLayout(UINT cKLs, const REG_IME *pLayouts, WORD wLangID); BOOL APIENTRY Imm32CopyImeFile(LPWSTR pszOldFile, LPCWSTR pszNewFile); - -/* Win: PtiCurrent */ -static inline PTHREADINFO FASTCALL Imm32CurrentPti(VOID) -{ - if (NtCurrentTeb()->Win32ThreadInfo == NULL) - NtUserGetThreadState(THREADSTATE_GETTHREADINFO); - return NtCurrentTeb()->Win32ThreadInfo; -} +PTHREADINFO FASTCALL Imm32CurrentPti(VOID); HBITMAP Imm32LoadBitmapFromBytes(const BYTE *pb); BOOL Imm32StoreBitmapToBytes(HBITMAP hbm, LPBYTE pbData, DWORD cbDataMax); diff --git a/dll/win32/imm32/regword.c b/dll/win32/imm32/regword.c index dfa4114a9ed..cdda6cb3234 100644 --- a/dll/win32/imm32/regword.c +++ b/dll/win32/imm32/regword.c @@ -38,14 +38,14 @@ Imm32EnumWordProcA2W(LPCSTR pszReadingA, DWORD dwStyle, LPCSTR pszRegisterA, LPV if (pszReadingA) { pszReadingW = Imm32WideFromAnsi(lpEnumData->uCodePage, pszReadingA); - if (pszReadingW == NULL) + if (IS_NULL_UNEXPECTEDLY(pszReadingW)) goto Quit; } if (pszRegisterA) { pszRegisterW = Imm32WideFromAnsi(lpEnumData->uCodePage, pszRegisterA); - if (pszRegisterW == NULL) + if (IS_NULL_UNEXPECTEDLY(pszRegisterW)) goto Quit; } @@ -68,14 +68,14 @@ Imm32EnumWordProcW2A(LPCWSTR pszReadingW, DWORD dwStyle, LPCWSTR pszRegisterW, L if (pszReadingW) { pszReadingA = Imm32AnsiFromWide(lpEnumData->uCodePage, pszReadingW); - if (pszReadingW == NULL) + if (IS_NULL_UNEXPECTEDLY(pszReadingW)) goto Quit; } if (pszRegisterW) { pszRegisterA = Imm32AnsiFromWide(lpEnumData->uCodePage, pszRegisterW); - if (pszRegisterA == NULL) + if (IS_NULL_UNEXPECTEDLY(pszRegisterA)) goto Quit; } @@ -105,10 +105,10 @@ ImmEnumRegisterWordA(HKL hKL, REGISTERWORDENUMPROCA lpfnEnumProc, dwStyle, debugstr_a(lpszRegister), lpData); pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return 0; - if (!ImeDpi_IsUnicode(pImeDpi)) + if (!ImeDpi_IsUnicode(pImeDpi)) /* No conversion needed */ { ret = pImeDpi->ImeEnumRegisterWord(lpfnEnumProc, lpszReading, dwStyle, lpszRegister, lpData); @@ -119,14 +119,14 @@ ImmEnumRegisterWordA(HKL hKL, REGISTERWORDENUMPROCA lpfnEnumProc, if (lpszReading) { pszReadingW = Imm32WideFromAnsi(pImeDpi->uCodePage, lpszReading); - if (pszReadingW == NULL) + if (IS_NULL_UNEXPECTEDLY(pszReadingW)) goto Quit; } if (lpszRegister) { pszRegisterW = Imm32WideFromAnsi(pImeDpi->uCodePage, lpszRegister); - if (pszRegisterW == NULL) + if (IS_NULL_UNEXPECTEDLY(pszRegisterW)) goto Quit; } @@ -162,10 +162,10 @@ ImmEnumRegisterWordW(HKL hKL, REGISTERWORDENUMPROCW lpfnEnumProc, dwStyle, debugstr_w(lpszRegister), lpData); pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return 0; - if (ImeDpi_IsUnicode(pImeDpi)) + if (ImeDpi_IsUnicode(pImeDpi)) /* No conversion needed */ { ret = pImeDpi->ImeEnumRegisterWord(lpfnEnumProc, lpszReading, dwStyle, lpszRegister, lpData); @@ -176,14 +176,14 @@ ImmEnumRegisterWordW(HKL hKL, REGISTERWORDENUMPROCW lpfnEnumProc, if (lpszReading) { pszReadingA = Imm32AnsiFromWide(pImeDpi->uCodePage, lpszReading); - if (pszReadingA == NULL) + if (IS_NULL_UNEXPECTEDLY(pszReadingA)) goto Quit; } if (lpszRegister) { pszRegisterA = Imm32AnsiFromWide(pImeDpi->uCodePage, lpszRegister); - if (pszRegisterA == NULL) + if (IS_NULL_UNEXPECTEDLY(pszRegisterA)) goto Quit; } @@ -199,6 +199,7 @@ Quit: ImmLocalFree(pszReadingA); ImmLocalFree(pszRegisterA); ImmUnlockImeDpi(pImeDpi); + TRACE("ret: %u\n", ret); return ret; } @@ -217,10 +218,10 @@ UINT WINAPI ImmGetRegisterWordStyleA(HKL hKL, UINT nItem, LPSTYLEBUFA lpStyleBuf TRACE("(%p, %u, %p)\n", hKL, nItem, lpStyleBuf); pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return 0; - if (!ImeDpi_IsUnicode(pImeDpi)) + if (!ImeDpi_IsUnicode(pImeDpi)) /* No conversion needed */ { ret = pImeDpi->ImeGetRegisterWordStyle(nItem, lpStyleBuf); goto Quit; @@ -229,7 +230,7 @@ UINT WINAPI ImmGetRegisterWordStyleA(HKL hKL, UINT nItem, LPSTYLEBUFA lpStyleBuf if (nItem > 0) { pNewStylesW = ImmLocalAlloc(0, nItem * sizeof(STYLEBUFW)); - if (!pNewStylesW) + if (IS_NULL_UNEXPECTEDLY(pNewStylesW)) goto Quit; } @@ -257,6 +258,7 @@ UINT WINAPI ImmGetRegisterWordStyleA(HKL hKL, UINT nItem, LPSTYLEBUFA lpStyleBuf Quit: ImmLocalFree(pNewStylesW); ImmUnlockImeDpi(pImeDpi); + TRACE("ret: %u\n", ret); return ret; } @@ -275,10 +277,10 @@ UINT WINAPI ImmGetRegisterWordStyleW(HKL hKL, UINT nItem, LPSTYLEBUFW lpStyleBuf TRACE("(%p, %u, %p)\n", hKL, nItem, lpStyleBuf); pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return 0; - if (ImeDpi_IsUnicode(pImeDpi)) + if (ImeDpi_IsUnicode(pImeDpi)) /* No conversion needed */ { ret = pImeDpi->ImeGetRegisterWordStyle(nItem, lpStyleBuf); goto Quit; @@ -287,7 +289,7 @@ UINT WINAPI ImmGetRegisterWordStyleW(HKL hKL, UINT nItem, LPSTYLEBUFW lpStyleBuf if (nItem > 0) { pNewStylesA = ImmLocalAlloc(0, nItem * sizeof(STYLEBUFA)); - if (!pNewStylesA) + if (IS_NULL_UNEXPECTEDLY(pNewStylesA)) goto Quit; } @@ -314,6 +316,7 @@ UINT WINAPI ImmGetRegisterWordStyleW(HKL hKL, UINT nItem, LPSTYLEBUFW lpStyleBuf Quit: ImmLocalFree(pNewStylesA); ImmUnlockImeDpi(pImeDpi); + TRACE("ret: %u\n", ret); return ret; } @@ -331,10 +334,10 @@ ImmRegisterWordA(HKL hKL, LPCSTR lpszReading, DWORD dwStyle, LPCSTR lpszRegister debugstr_a(lpszRegister)); pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return FALSE; - if (!ImeDpi_IsUnicode(pImeDpi)) + if (!ImeDpi_IsUnicode(pImeDpi)) /* No conversion needed */ { ret = pImeDpi->ImeRegisterWord(lpszReading, dwStyle, lpszRegister); ImmUnlockImeDpi(pImeDpi); @@ -344,14 +347,14 @@ ImmRegisterWordA(HKL hKL, LPCSTR lpszReading, DWORD dwStyle, LPCSTR lpszRegister if (lpszReading) { pszReadingW = Imm32WideFromAnsi(pImeDpi->uCodePage, lpszReading); - if (pszReadingW == NULL) + if (IS_NULL_UNEXPECTEDLY(pszReadingW)) goto Quit; } if (lpszRegister) { pszRegisterW = Imm32WideFromAnsi(pImeDpi->uCodePage, lpszRegister); - if (pszRegisterW == NULL) + if (IS_NULL_UNEXPECTEDLY(pszRegisterW)) goto Quit; } @@ -361,6 +364,7 @@ Quit: ImmLocalFree(pszReadingW); ImmLocalFree(pszRegisterW); ImmUnlockImeDpi(pImeDpi); + TRACE("ret: %d\n", ret); return ret; } @@ -378,10 +382,10 @@ ImmRegisterWordW(HKL hKL, LPCWSTR lpszReading, DWORD dwStyle, LPCWSTR lpszRegist debugstr_w(lpszRegister)); pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return FALSE; - if (ImeDpi_IsUnicode(pImeDpi)) + if (ImeDpi_IsUnicode(pImeDpi)) /* No conversion needed */ { ret = pImeDpi->ImeRegisterWord(lpszReading, dwStyle, lpszRegister); ImmUnlockImeDpi(pImeDpi); @@ -391,14 +395,14 @@ ImmRegisterWordW(HKL hKL, LPCWSTR lpszReading, DWORD dwStyle, LPCWSTR lpszRegist if (lpszReading) { pszReadingA = Imm32AnsiFromWide(pImeDpi->uCodePage, lpszReading); - if (!pszReadingA) + if (IS_NULL_UNEXPECTEDLY(pszReadingA)) goto Quit; } if (lpszRegister) { pszRegisterA = Imm32AnsiFromWide(pImeDpi->uCodePage, lpszRegister); - if (!pszRegisterA) + if (IS_NULL_UNEXPECTEDLY(pszRegisterA)) goto Quit; } @@ -408,6 +412,7 @@ Quit: ImmLocalFree(pszReadingA); ImmLocalFree(pszRegisterA); ImmUnlockImeDpi(pImeDpi); + TRACE("ret: %d\n", ret); return ret; } @@ -425,10 +430,10 @@ ImmUnregisterWordA(HKL hKL, LPCSTR lpszReading, DWORD dwStyle, LPCSTR lpszUnregi debugstr_a(lpszUnregister)); pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (pImeDpi == NULL) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return FALSE; - if (!ImeDpi_IsUnicode(pImeDpi)) + if (!ImeDpi_IsUnicode(pImeDpi)) /* No conversion needed */ { ret = pImeDpi->ImeUnregisterWord(lpszReading, dwStyle, lpszUnregister); ImmUnlockImeDpi(pImeDpi); @@ -438,14 +443,14 @@ ImmUnregisterWordA(HKL hKL, LPCSTR lpszReading, DWORD dwStyle, LPCSTR lpszUnregi if (lpszReading) { pszReadingW = Imm32WideFromAnsi(pImeDpi->uCodePage, lpszReading); - if (pszReadingW == NULL) + if (IS_NULL_UNEXPECTEDLY(pszReadingW)) goto Quit; } if (lpszUnregister) { pszUnregisterW = Imm32WideFromAnsi(pImeDpi->uCodePage, lpszUnregister); - if (pszUnregisterW == NULL) + if (IS_NULL_UNEXPECTEDLY(pszUnregisterW)) goto Quit; } @@ -455,6 +460,7 @@ Quit: ImmLocalFree(pszReadingW); ImmLocalFree(pszUnregisterW); ImmUnlockImeDpi(pImeDpi); + TRACE("ret: %d\n", ret); return ret; } @@ -472,10 +478,10 @@ ImmUnregisterWordW(HKL hKL, LPCWSTR lpszReading, DWORD dwStyle, LPCWSTR lpszUnre debugstr_w(lpszUnregister)); pImeDpi = Imm32FindOrLoadImeDpi(hKL); - if (!pImeDpi) + if (IS_NULL_UNEXPECTEDLY(pImeDpi)) return FALSE; - if (ImeDpi_IsUnicode(pImeDpi)) + if (ImeDpi_IsUnicode(pImeDpi)) /* No conversion needed */ { ret = pImeDpi->ImeUnregisterWord(lpszReading, dwStyle, lpszUnregister); ImmUnlockImeDpi(pImeDpi); @@ -485,14 +491,14 @@ ImmUnregisterWordW(HKL hKL, LPCWSTR lpszReading, DWORD dwStyle, LPCWSTR lpszUnre if (lpszReading) { pszReadingA = Imm32AnsiFromWide(pImeDpi->uCodePage, lpszReading); - if (!pszReadingA) + if (IS_NULL_UNEXPECTEDLY(pszReadingA)) goto Quit; } if (lpszUnregister) { pszUnregisterA = Imm32AnsiFromWide(pImeDpi->uCodePage, lpszUnregister); - if (!pszUnregisterA) + if (IS_NULL_UNEXPECTEDLY(pszUnregisterA)) goto Quit; } @@ -502,5 +508,6 @@ Quit: ImmLocalFree(pszReadingA); ImmLocalFree(pszUnregisterA); ImmUnlockImeDpi(pImeDpi); + TRACE("ret: %d\n", ret); return ret; } diff --git a/dll/win32/imm32/utils.c b/dll/win32/imm32/utils.c index e9906ced61e..b98697358e9 100644 --- a/dll/win32/imm32/utils.c +++ b/dll/win32/imm32/utils.c @@ -15,6 +15,29 @@ WINE_DEFAULT_DEBUG_CHANNEL(imm); HANDLE ghImmHeap = NULL; // Win: pImmHeap +/* Win: PtiCurrent */ +PTHREADINFO FASTCALL Imm32CurrentPti(VOID) +{ + if (NtCurrentTeb()->Win32ThreadInfo == NULL) + NtUserGetThreadState(THREADSTATE_GETTHREADINFO); + return NtCurrentTeb()->Win32ThreadInfo; +} + +BOOL APIENTRY Imm32IsCrossThreadAccess(HIMC hIMC) +{ + DWORD_PTR dwImeThreadId = NtUserQueryInputContext(hIMC, QIC_INPUTTHREADID); + DWORD_PTR dwCurrentThreadId = GetCurrentThreadId(); + return dwImeThreadId != dwCurrentThreadId; +} + +// Win: TestWindowProcess +BOOL APIENTRY Imm32IsCrossProcessAccess(HWND hWnd) +{ + DWORD_PTR WndPID = NtUserQueryWindow(hWnd, QUERY_WINDOW_UNIQUE_PROCESS_ID); + DWORD_PTR CurrentPID = (DWORD_PTR)NtCurrentTeb()->ClientId.UniqueProcess; + return WndPID != CurrentPID; +} + // Win: StrToUInt HRESULT APIENTRY Imm32StrToUInt(LPCWSTR pszText, LPDWORD pdwValue, ULONG nBase) @@ -48,7 +71,12 @@ BOOL APIENTRY Imm32IsSystemJapaneseOrKorean(VOID) LCID lcid = GetSystemDefaultLCID(); LANGID LangID = LANGIDFROMLCID(lcid); WORD wPrimary = PRIMARYLANGID(LangID); - return (wPrimary == LANG_JAPANESE || wPrimary == LANG_KOREAN); + if (wPrimary != LANG_JAPANESE || wPrimary != LANG_KOREAN) + { + WARN("\n"); + return FALSE; + } + return TRUE; } typedef struct tagBITMAPCOREINFO256 @@ -75,7 +103,10 @@ HBITMAP Imm32LoadBitmapFromBytes(const BYTE *pb) pbmci = (const BITMAPCOREINFO256 *)pb; hbm = CreateDIBSection(NULL, (LPBITMAPINFO)pbmci, DIB_RGB_COLORS, &pvBits, NULL, 0); if (!hbm || !GetObject(hbm, sizeof(BITMAP), &bm)) + { + ERR("\n"); return NULL; + } switch (pbmci->bmciHeader.bcBitCount) { @@ -86,6 +117,7 @@ HBITMAP Imm32LoadBitmapFromBytes(const BYTE *pb) cColors = 0; break; default: + ERR("\n"); DeleteObject(hbm); return NULL; } @@ -99,6 +131,7 @@ HBITMAP Imm32LoadBitmapFromBytes(const BYTE *pb) ib += bm.bmWidthBytes * bm.bmHeight; if (ib > cbBytes) { + ERR("\n"); DeleteObject(hbm); return NULL; } @@ -119,7 +152,10 @@ BOOL Imm32StoreBitmapToBytes(HBITMAP hbm, LPBYTE pbData, DWORD cbDataMax) *(LPDWORD)pb = 0; if (!GetObject(hbm, sizeof(BITMAP), &bm)) + { + ERR("\n"); return FALSE; + } ZeroMemory(&bmci, sizeof(bmci)); bmci.bmciHeader.bcSize = sizeof(BITMAPCOREHEADER); @@ -137,6 +173,7 @@ BOOL Imm32StoreBitmapToBytes(HBITMAP hbm, LPBYTE pbData, DWORD cbDataMax) cColors = 0; break; default: + ERR("\n"); return FALSE; } @@ -145,7 +182,10 @@ BOOL Imm32StoreBitmapToBytes(HBITMAP hbm, LPBYTE pbData, DWORD cbDataMax) cbBytes += cColors * sizeof(RGBTRIPLE); cbBytes += bm.bmWidthBytes * bm.bmHeight; if (cbBytes > cbDataMax) + { + ERR("\n"); return FALSE; + } hDC = CreateCompatibleDC(NULL); @@ -177,7 +217,7 @@ BOOL WINAPI Imm32IsImcAnsi(HIMC hIMC) { BOOL ret; PCLIENTIMC pClientImc = ImmLockClientImc(hIMC); - if (!pClientImc) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return -1; ret = !(pClientImc->dwFlags & CLIENTIMC_WIDE); ImmUnlockClientImc(pClientImc); @@ -188,7 +228,7 @@ LPWSTR APIENTRY Imm32WideFromAnsi(UINT uCodePage, LPCSTR pszA) { INT cch = lstrlenA(pszA); LPWSTR pszW = ImmLocalAlloc(0, (cch + 1) * sizeof(WCHAR)); - if (pszW == NULL) + if (IS_NULL_UNEXPECTEDLY(pszW)) return NULL; cch = MultiByteToWideChar(uCodePage, MB_PRECOMPOSED, pszA, cch, pszW, cch + 1); pszW[cch] = 0; @@ -200,7 +240,7 @@ LPSTR APIENTRY Imm32AnsiFromWide(UINT uCodePage, LPCWSTR pszW) INT cchW = lstrlenW(pszW); INT cchA = (cchW + 1) * sizeof(WCHAR); LPSTR pszA = ImmLocalAlloc(0, cchA); - if (!pszA) + if (IS_NULL_UNEXPECTEDLY(pszA)) return NULL; cchA = WideCharToMultiByte(uCodePage, 0, pszW, cchW, pszA, cchA, NULL, NULL); pszA[cchA] = 0; @@ -246,7 +286,10 @@ LONG APIENTRY IchAnsiFromWide(LONG cchWide, LPCWSTR pchWide, UINT uCodePage) BOOL Imm32GetSystemLibraryPath(LPWSTR pszPath, DWORD cchPath, LPCWSTR pszFileName) { if (!pszFileName[0] || !GetSystemDirectoryW(pszPath, cchPath)) + { + ERR("\n"); return FALSE; + } StringCchCatW(pszPath, cchPath, L"\\"); StringCchCatW(pszPath, cchPath, pszFileName); return TRUE; @@ -301,7 +344,10 @@ LPVOID FASTCALL ValidateHandleNoErr(HANDLE hObject, UINT uType) LPVOID ptr; if (!NtUserValidateHandleSecure(hObject)) + { + WARN("\n"); return NULL; + } ht = gSharedInfo.aheList; /* handle table */ ASSERT(ht); @@ -345,13 +391,24 @@ LPVOID FASTCALL ValidateHandle(HANDLE hObject, UINT uType) BOOL APIENTRY Imm32CheckImcProcess(PIMC pIMC) { HIMC hIMC; - DWORD dwProcessID; + DWORD_PTR dwPID1, dwPID2; + + if (IS_NULL_UNEXPECTEDLY(pIMC)) + return FALSE; + if (pIMC->head.pti == Imm32CurrentPti()) return TRUE; hIMC = pIMC->head.h; - dwProcessID = (DWORD)NtUserQueryInputContext(hIMC, QIC_INPUTPROCESSID); - return dwProcessID == (DWORD_PTR)NtCurrentTeb()->ClientId.UniqueProcess; + dwPID1 = (DWORD)NtUserQueryInputContext(hIMC, QIC_INPUTPROCESSID); + dwPID2 = (DWORD_PTR)NtCurrentTeb()->ClientId.UniqueProcess; + if (dwPID1 != dwPID2) + { + WARN("PID 0x%X != 0x%X\n", dwPID1, dwPID2); + return FALSE; + } + + return TRUE; } // Win: ImmLocalAlloc @@ -360,7 +417,7 @@ LPVOID APIENTRY ImmLocalAlloc(DWORD dwFlags, DWORD dwBytes) if (!ghImmHeap) { ghImmHeap = RtlGetProcessHeap(); - if (ghImmHeap == NULL) + if (IS_NULL_UNEXPECTEDLY(ghImmHeap)) return NULL; } return HeapAlloc(ghImmHeap, dwFlags, dwBytes); @@ -375,7 +432,7 @@ Imm32MakeIMENotify(HIMC hIMC, HWND hwnd, DWORD dwAction, DWORD_PTR dwIndex, DWOR HKL hKL; PIMEDPI pImeDpi; - if (dwAction) + if (dwAction != 0) { dwThreadId = (DWORD)NtUserQueryInputContext(hIMC, QIC_INPUTTHREADID); if (dwThreadId) @@ -409,7 +466,7 @@ DWORD APIENTRY Imm32BuildHimcList(DWORD dwThreadId, HIMC **pphList) HIMC *phNewList; phNewList = ImmLocalAlloc(0, dwCount * sizeof(HIMC)); - if (phNewList == NULL) + if (IS_NULL_UNEXPECTEDLY(phNewList)) return 0; Status = NtUserBuildHimcList(dwThreadId, dwCount, phNewList, &dwCount); @@ -420,7 +477,7 @@ DWORD APIENTRY Imm32BuildHimcList(DWORD dwThreadId, HIMC **pphList) return 0; phNewList = ImmLocalAlloc(0, dwCount * sizeof(HIMC)); - if (phNewList == NULL) + if (IS_NULL_UNEXPECTEDLY(phNewList)) return 0; Status = NtUserBuildHimcList(dwThreadId, dwCount, phNewList, &dwCount); @@ -428,6 +485,7 @@ DWORD APIENTRY Imm32BuildHimcList(DWORD dwThreadId, HIMC **pphList) if (NT_ERROR(Status) || !dwCount) { + ERR("\n"); ImmLocalFree(phNewList); return 0; } @@ -538,12 +596,11 @@ BOOL APIENTRY Imm32LoadImeStateSentence(LPINPUTCONTEXTDX pIC, PIME_STATE pState, HKL hKL) { PIME_SUBSTATE pSubState = Imm32FetchImeSubState(pState, hKL); - if (pSubState) - { - pIC->fdwSentence |= pSubState->dwValue; - return TRUE; - } - return FALSE; + if (IS_NULL_UNEXPECTEDLY(pSubState)) + return FALSE; + + pIC->fdwSentence |= pSubState->dwValue; + return TRUE; } // Win: SavePrivateMode @@ -551,12 +608,11 @@ BOOL APIENTRY Imm32SaveImeStateSentence(LPINPUTCONTEXTDX pIC, PIME_STATE pState, HKL hKL) { PIME_SUBSTATE pSubState = Imm32FetchImeSubState(pState, hKL); - if (pSubState) - { - pSubState->dwValue = (pIC->fdwSentence & 0xffff0000); - return TRUE; - } - return FALSE; + if (IS_NULL_UNEXPECTEDLY(pSubState)) + return FALSE; + + pSubState->dwValue = (pIC->fdwSentence & 0xffff0000); + return TRUE; } /* @@ -576,7 +632,10 @@ Imm32ReconvertWideFromAnsi(LPRECONVERTSTRING pDest, const RECONVERTSTRING *pSrc, LPWSTR pchDest; if (pSrc->dwVersion != 0) + { + ERR("\n"); return 0; + } cchDest = MultiByteToWideChar(uCodePage, MB_PRECOMPOSED, pchSrc, pSrc->dwStrLen, NULL, 0); @@ -585,7 +644,10 @@ Imm32ReconvertWideFromAnsi(LPRECONVERTSTRING pDest, const RECONVERTSTRING *pSrc, return cbDest; if (pDest->dwSize < cbDest) + { + ERR("\n"); return 0; + } /* dwSize */ pDest->dwSize = cbDest; @@ -621,6 +683,7 @@ Imm32ReconvertWideFromAnsi(LPRECONVERTSTRING pDest, const RECONVERTSTRING *pSrc, pchDest, cchDest); pchDest[cchDest] = 0; + TRACE("cbDest: 0x%X\n", cbDest); return cbDest; } @@ -641,7 +704,10 @@ Imm32ReconvertAnsiFromWide(LPRECONVERTSTRING pDest, const RECONVERTSTRING *pSrc, return cbDest; if (pDest->dwSize < cbDest) + { + ERR("\n"); return 0; + } /* dwSize */ pDest->dwSize = cbDest; @@ -679,6 +745,7 @@ Imm32ReconvertAnsiFromWide(LPRECONVERTSTRING pDest, const RECONVERTSTRING *pSrc, pchDest, cchDest, NULL, NULL); pchDest[cchDest] = 0; + TRACE("cchDest: 0x%X\n", cchDest); return cbDest; } @@ -696,11 +763,17 @@ static BOOL APIENTRY Imm32LoadImeFixedInfo(PIMEINFOEX pInfoEx, LPCVOID pVerInfo) UINT cbFixed = 0; VS_FIXEDFILEINFO *pFixed; if (!s_fnVerQueryValueW(pVerInfo, L"\\", (LPVOID*)&pFixed, &cbFixed) || !cbFixed) + { + ERR("\n"); return FALSE; + } /* NOTE: The IME module must contain a version info of input method driver. */ if (pFixed->dwFileType != VFT_DRV || pFixed->dwFileSubtype != VFT2_DRV_INPUTMETHOD) + { + ERR("\n"); return FALSE; + } pInfoEx->dwProdVersion = pFixed->dwProductVersionMS; pInfoEx->dwImeWinVersion = 0x40000; @@ -737,10 +810,13 @@ BOOL APIENTRY Imm32LoadImeLangAndDesc(PIMEINFOEX pInfoEx, LPCVOID pVerInfo) /* Getting the version info. See VerQueryValue */ ret = s_fnVerQueryValueW(pVerInfo, L"\\VarFileInfo\\Translation", (LPVOID*)&pw, &cbData); if (!ret || !cbData) + { + ERR("\n"); return FALSE; + } if (pInfoEx->hkl == NULL) - pInfoEx->hkl = (HKL)(DWORD_PTR)*pw; /* This is an invalid HKL */ + pInfoEx->hkl = (HKL)(DWORD_PTR)*pw; /* Try the current language and the Unicode codepage (0x04B0) */ LangID = LANGIDFROMLCID(GetThreadLocale()); @@ -777,7 +853,7 @@ BOOL APIENTRY Imm32LoadImeVerInfo(PIMEINFOEX pImeInfoEx) if (!hinstVersion) { hinstVersion = LoadLibraryW(szPath); - if (!hinstVersion) + if (IS_NULL_UNEXPECTEDLY(hinstVersion)) return FALSE; bLoaded = TRUE; @@ -796,11 +872,11 @@ BOOL APIENTRY Imm32LoadImeVerInfo(PIMEINFOEX pImeInfoEx) Imm32GetSystemLibraryPath(szPath, _countof(szPath), pImeInfoEx->wszImeFile); cbVerInfo = s_fnGetFileVersionInfoSizeW(szPath, &dwHandle); - if (!cbVerInfo) + if (IS_ZERO_UNEXPECTEDLY(cbVerInfo)) goto Quit; pVerInfo = ImmLocalAlloc(0, cbVerInfo); - if (!pVerInfo) + if (IS_NULL_UNEXPECTEDLY(pVerInfo)) goto Quit; /* Load the version info of the IME module */ @@ -815,6 +891,7 @@ BOOL APIENTRY Imm32LoadImeVerInfo(PIMEINFOEX pImeInfoEx) Quit: if (bLoaded) FreeLibrary(hinstVersion); + TRACE("ret: %d\n", ret); return ret; } @@ -877,7 +954,7 @@ UINT APIENTRY Imm32GetImeLayout(PREG_IME pLayouts, UINT cLayouts) /* Open the registry keyboard layouts */ lError = RegOpenKeyW(HKEY_LOCAL_MACHINE, REGKEY_KEYBOARD_LAYOUTS, &hkeyLayouts); - if (lError != ERROR_SUCCESS) + if (IS_ERROR_UNEXPECTEDLY(lError)) return 0; for (iKey = nCount = 0; ; ++iKey) @@ -900,8 +977,11 @@ UINT APIENTRY Imm32GetImeLayout(PREG_IME pLayouts, UINT cLayouts) break; lError = RegOpenKeyW(hkeyLayouts, szImeKey, &hkeyIME); /* Open the IME key */ - if (lError != ERROR_SUCCESS) + if (IS_ERROR_UNEXPECTEDLY(lError)) + { + WARN("\n"); continue; + } /* Load the "Ime File" value */ szImeFileName[0] = 0; @@ -913,12 +993,18 @@ UINT APIENTRY Imm32GetImeLayout(PREG_IME pLayouts, UINT cLayouts) /* We don't allow the invalid "IME File" values for security reason */ if (!szImeFileName[0] || wcscspn(szImeFileName, L":\\/") != wcslen(szImeFileName)) + { + WARN("\n"); continue; + } Imm32StrToUInt(szImeKey, &Value, 16); hKL = (HKL)(DWORD_PTR)Value; if (!IS_IME_HKL(hKL)) + { + WARN("\n"); continue; + } /* Store the IME key and the IME filename */ pLayouts[nCount].hKL = hKL; @@ -946,27 +1032,27 @@ BOOL APIENTRY Imm32WriteImeLayout(HKL hKL, LPCWSTR pchFilePart, LPCWSTR pszLayou /* Open the registry keyboard layouts */ lError = RegOpenKeyW(HKEY_LOCAL_MACHINE, REGKEY_KEYBOARD_LAYOUTS, &hkeyLayouts); - if (lError != ERROR_SUCCESS) + if (IS_ERROR_UNEXPECTEDLY(lError)) return FALSE; /* Get the IME key from hKL */ - Imm32UIntToStr((DWORD)(DWORD_PTR)hKL, 16, szImeKey, _countof(szImeKey)); + StringCchPrintf(szImeKey, _countof(szImeKey), L"%08X", (DWORD)(DWORD_PTR)hKL); /* Create a registry IME key */ lError = RegCreateKeyW(hkeyLayouts, szImeKey, &hkeyIME); - if (lError != ERROR_SUCCESS) + if (IS_ERROR_UNEXPECTEDLY(lError)) goto Failure; /* Write "Ime File" */ cbData = (wcslen(pchFilePart) + 1) * sizeof(WCHAR); lError = RegSetValueExW(hkeyIME, L"Ime File", 0, REG_SZ, (LPBYTE)pchFilePart, cbData); - if (lError != ERROR_SUCCESS) + if (IS_ERROR_UNEXPECTEDLY(lError)) goto Failure; /* Write "Layout Text" */ cbData = (wcslen(pszLayoutText) + 1) * sizeof(WCHAR); lError = RegSetValueExW(hkeyIME, L"Layout Text", 0, REG_SZ, (LPBYTE)pszLayoutText, cbData); - if (lError != ERROR_SUCCESS) + if (IS_ERROR_UNEXPECTEDLY(lError)) goto Failure; /* Choose "Layout File" from hKL */ @@ -981,7 +1067,7 @@ BOOL APIENTRY Imm32WriteImeLayout(HKL hKL, LPCWSTR pchFilePart, LPCWSTR pszLayou /* Write "Layout File" */ cbData = (wcslen(pszLayoutFile) + 1) * sizeof(WCHAR); lError = RegSetValueExW(hkeyIME, L"Layout File", 0, REG_SZ, (LPBYTE)pszLayoutFile, cbData); - if (lError != ERROR_SUCCESS) + if (IS_ERROR_UNEXPECTEDLY(lError)) goto Failure; RegCloseKey(hkeyIME); @@ -1007,6 +1093,7 @@ BOOL APIENTRY Imm32WriteImeLayout(HKL hKL, LPCWSTR pchFilePart, LPCWSTR pszLayou if (iPreload >= MAX_PRELOAD) /* Not found */ { + WARN("\n"); RegCloseKey(hkeyPreload); return FALSE; } @@ -1048,7 +1135,7 @@ BOOL APIENTRY Imm32CopyImeFile(LPWSTR pszOldFile, LPCWSTR pszNewFile) if (!hinstLZ32) { hinstLZ32 = LoadLibraryW(szLZ32Path); - if (!hinstLZ32) + if (IS_NULL_UNEXPECTEDLY(hinstLZ32)) return FALSE; bLoaded = TRUE; } @@ -1163,7 +1250,7 @@ DWORD WINAPI ImmGetIMCLockCount(HIMC hIMC) PCLIENTIMC pClientImc; pClientImc = ImmLockClientImc(hIMC); - if (pClientImc == NULL) + if (IS_NULL_UNEXPECTEDLY(pClientImc)) return 0; ret = 0; diff --git a/dll/win32/imm32/win3.c b/dll/win32/imm32/win3.c index 68dc4861f8c..7904c751561 100644 --- a/dll/win32/imm32/win3.c +++ b/dll/win32/imm32/win3.c @@ -16,7 +16,7 @@ DWORD APIENTRY Imm32JTransCompA(LPINPUTCONTEXTDX pIC, LPCOMPOSITIONSTRING pCS, const TRANSMSG *pSrc, LPTRANSMSG pDest) { - // FIXME + FIXME("\n"); *pDest = *pSrc; return 1; } @@ -26,7 +26,7 @@ DWORD APIENTRY Imm32JTransCompW(LPINPUTCONTEXTDX pIC, LPCOMPOSITIONSTRING pCS, const TRANSMSG *pSrc, LPTRANSMSG pDest) { - // FIXME + FIXME("\n"); *pDest = *pSrc; return 1; } @@ -52,7 +52,7 @@ WINNLSTranslateMessageJ(DWORD dwCount, LPTRANSMSG pTrans, LPINPUTCONTEXTDX pIC, // clone the message list cbTempList = (dwCount + 1) * sizeof(TRANSMSG); pTempList = ImmLocalAlloc(HEAP_ZERO_MEMORY, cbTempList); - if (pTempList == NULL) + if (IS_NULL_UNEXPECTEDLY(pTempList)) return 0; RtlCopyMemory(pTempList, pTrans, dwCount * sizeof(TRANSMSG)); @@ -188,19 +188,22 @@ WINNLSTranslateMessage(DWORD dwCount, LPTRANSMSG pEntries, HIMC hIMC, BOOL bAnsi LPCOMPOSITIONSTRING pCS; pIC = (LPINPUTCONTEXTDX)ImmLockIMC(hIMC); - if (pIC == NULL) + if (IS_NULL_UNEXPECTEDLY(pIC)) return 0; pCS = ImmLockIMCC(pIC->hCompStr); - if (pCS) + if (IS_NULL_UNEXPECTEDLY(pCS)) { - if (wLang == LANG_JAPANESE) - ret = WINNLSTranslateMessageJ(dwCount, pEntries, pIC, pCS, bAnsi); - else if (wLang == LANG_KOREAN) - ret = WINNLSTranslateMessageK(dwCount, pEntries, pIC, pCS, bAnsi); - ImmUnlockIMCC(pIC->hCompStr); + ImmUnlockIMC(hIMC); + return 0; } + if (wLang == LANG_JAPANESE) + ret = WINNLSTranslateMessageJ(dwCount, pEntries, pIC, pCS, bAnsi); + else if (wLang == LANG_KOREAN) + ret = WINNLSTranslateMessageK(dwCount, pEntries, pIC, pCS, bAnsi); + + ImmUnlockIMCC(pIC->hCompStr); ImmUnlockIMC(hIMC); return ret; }

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