Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2024-12-28 01:55:19 +00:00
Code Issues Projects Releases Packages Wiki Activity

[REACTOS] Refactor CRegKey usage (#3779)

Browse source 
- [MSPAINT] ReadDWORD(): Improve 'dwValue' handling
Addendum to 3185db7 (r72158).

- [DRWTSN32] Settings_GetOutputPath(): Fix 'key' handling
Addendum to 1d10606.

- [RAPPS] Remove 2 superfluous 'CRegKey::Close()' calls
Follow-up to 42e111f.
This commit is contained in:
Serge Gautherie 2021-07-01 00:08:33 +02:00 committed by Stanislav Motylkov
parent a0bf7a05cc
commit 18d15bcc8b
No known key found for this signature in database
GPG key ID: AFE513258CBA9E92
3 changed files with 5 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
base/applications/drwtsn32/main.cpp
View file

@ -170,17 +170,13 @@ std::wstring Settings_GetOutputPath(void)
{
WCHAR Buffer[MAX_PATH] = L"";
ULONG BufferSize = _countof(Buffer);
BOOL UseDefaultPath = FALSE;
BOOL UseDefaultPath = TRUE;
CRegKey key;
if (key.Open(HKEY_CURRENT_USER, L"SOFTWARE\\ReactOS\\Crash Reporter", KEY_READ) != ERROR_SUCCESS)
if (key.Open(HKEY_CURRENT_USER, L"SOFTWARE\\ReactOS\\Crash Reporter", KEY_READ) == ERROR_SUCCESS &&
key.QueryStringValue(L"Dump Directory", Buffer, &BufferSize) == ERROR_SUCCESS)
{
UseDefaultPath = TRUE;
}
if (key.QueryStringValue(L"Dump Directory", Buffer, &BufferSize) != ERROR_SUCCESS)
{
UseDefaultPath = TRUE;
UseDefaultPath = FALSE;
}
if (UseDefaultPath)

5
base/applications/mspaint/registry.cpp
View file

@ -19,10 +19,7 @@ static DWORD ReadDWORD(CRegKey &key, LPCTSTR lpName, DWORD &dwValue, BOOL bCheck
{
DWORD dwPrev = dwValue;
if (key.QueryDWORDValue(lpName, dwValue) != ERROR_SUCCESS)
dwValue = dwPrev;
if (bCheckForDef && dwValue == 0)
if (key.QueryDWORDValue(lpName, dwValue) != ERROR_SUCCESS || (bCheckForDef && dwValue == 0))
dwValue = dwPrev;
return dwPrev;

3
base/applications/rapps/misc.cpp
View file

@ -233,8 +233,6 @@ VOID InitLogs()
{
hLog = RegisterEventSourceW(NULL, L"ReactOS Application Manager");
}
key.Close();
}
@ -302,7 +300,6 @@ BOOL GetInstalledVersion_WowUser(ATL::CStringW* szVersionResult,
bHasSucceded = TRUE;
szVersion.ReleaseBuffer();
}
key.Close();
return bHasSucceded;
}