From aa405244308f79b5478c82f42d934c7d63599930 Mon Sep 17 00:00:00 2001 From: Joachim Henze Date: Sun, 13 Mar 2022 00:20:56 +0100 Subject: [PATCH] [0.4.7][NOTEPAD] Fix a MSVC warning about Globals.encFile (#648) (#650) CORE-7538 Observable in MSVC2010SP1 x86 dbg-config "...\dialog.c(365) : error C4133: 'function' : incompatible types - from 'ENCODING *' to 'int *'" CORE-7538 the fix is a squashed port of: 0.4.10-dev-286-g 6830ecb118c80f0dd625b79fe37b66315e024f46 (#650) 0.4.10-dev-283-g 97df61edfa1a630ed48de98998a352b6b3eb5466 0.4.10-dev-282-g 534a309edcac565afa82f4356e942ae9aaed3b2e (#648) got superseded by 2nd commit --- base/applications/notepad/dialog.c | 2 +- base/applications/notepad/main.h | 4 ++-- base/applications/notepad/text.c | 9 +++++---- 3 files changed, 8 insertions(+), 7 deletions(-) diff --git a/base/applications/notepad/dialog.c b/base/applications/notepad/dialog.c index 04e6e7b8375..d92e11a94f5 100644 --- a/base/applications/notepad/dialog.c +++ b/base/applications/notepad/dialog.c @@ -496,7 +496,7 @@ DIALOG_FileSaveAs_Hook(HWND hDlg, UINT msg, WPARAM wParam, LPARAM lParam) { hCombo = GetDlgItem(hDlg, ID_ENCODING); if (hCombo) - Globals.encFile = (int) SendMessage(hCombo, CB_GETCURSEL, 0, 0); + Globals.encFile = (ENCODING) SendMessage(hCombo, CB_GETCURSEL, 0, 0); hCombo = GetDlgItem(hDlg, ID_EOLN); if (hCombo) diff --git a/base/applications/notepad/main.h b/base/applications/notepad/main.h index c64ccf6e2f0..a84162fbc05 100644 --- a/base/applications/notepad/main.h +++ b/base/applications/notepad/main.h @@ -91,8 +91,8 @@ extern NOTEPAD_GLOBALS Globals; VOID SetFileName(LPCTSTR szFileName); /* from text.c */ -BOOL ReadText(HANDLE hFile, LPWSTR *ppszText, DWORD *pdwTextLen, int *pencFile, int *piEoln); -BOOL WriteText(HANDLE hFile, LPCWSTR pszText, DWORD dwTextLen, int encFile, int iEoln); +BOOL ReadText(HANDLE hFile, LPWSTR *ppszText, DWORD *pdwTextLen, ENCODING *pencFile, int *piEoln); +BOOL WriteText(HANDLE hFile, LPCWSTR pszText, DWORD dwTextLen, ENCODING encFile, int iEoln); /* from settings.c */ void NOTEPAD_LoadSettingsFromRegistry(void); diff --git a/base/applications/notepad/text.c b/base/applications/notepad/text.c index be7a76446a6..2c6ee8cd070 100644 --- a/base/applications/notepad/text.c +++ b/base/applications/notepad/text.c @@ -48,7 +48,7 @@ static BOOL Append(LPWSTR *ppszText, DWORD *pdwTextLen, LPCWSTR pszAppendText, D } BOOL -ReadText(HANDLE hFile, LPWSTR *ppszText, DWORD *pdwTextLen, int *pencFile, int *piEoln) +ReadText(HANDLE hFile, LPWSTR *ppszText, DWORD *pdwTextLen, ENCODING *pencFile, int *piEoln) { DWORD dwSize; LPBYTE pBytes = NULL; @@ -58,7 +58,7 @@ ReadText(HANDLE hFile, LPWSTR *ppszText, DWORD *pdwTextLen, int *pencFile, int * DWORD dwCharCount; BOOL bSuccess = FALSE; BYTE b = 0; - int encFile = ENCODING_ANSI; + ENCODING encFile = ENCODING_ANSI; int iCodePage = 0; WCHAR szCrlf[2] = {'\r', '\n'}; DWORD adwEolnCount[3] = {0, 0, 0}; @@ -147,6 +147,7 @@ ReadText(HANDLE hFile, LPWSTR *ppszText, DWORD *pdwTextLen, int *pencFile, int * pszAllocText[dwCharCount] = '\0'; pszText = pszAllocText; break; + DEFAULT_UNREACHABLE; } dwPos = 0; @@ -221,7 +222,7 @@ done: return bSuccess; } -static BOOL WriteEncodedText(HANDLE hFile, LPCWSTR pszText, DWORD dwTextLen, int encFile) +static BOOL WriteEncodedText(HANDLE hFile, LPCWSTR pszText, DWORD dwTextLen, ENCODING encFile) { LPBYTE pBytes = NULL; LPBYTE pAllocBuffer = NULL; @@ -315,7 +316,7 @@ done: return bSuccess; } -BOOL WriteText(HANDLE hFile, LPCWSTR pszText, DWORD dwTextLen, int encFile, int iEoln) +BOOL WriteText(HANDLE hFile, LPCWSTR pszText, DWORD dwTextLen, ENCODING encFile, int iEoln) { WCHAR wcBom; LPCWSTR pszLF = L"\n";