Increased revision.

Changed main "ghost" window to a message-only window.

.gitignore

@@ -326,7 +326,7 @@ var/
 # PyInstaller
 #  Usually these files are written by a python script from a template
 #  before PyInstaller builds the exe, so as to inject date/other infos into it.
-*.manifest
+#*.manifest
 *.spec
 
 # Installer logs

wCenterWindow/Logger.cpp

@@ -0,0 +1,76 @@
+// wCenterWindow, v2.3.3
+// Logger.cpp
+//
+#include "framework.h"
+
+#define TS_LEN 30
+#define PATH_LEN 1024
+
+std::wofstream logfile;
+extern WCHAR szTitle[];
+extern LPVOID szBuffer;
+namespace fs = std::filesystem;
+
+std::wstring PrintTitle()
+{
+	wchar_t szWinTitle[2048];
+	StringCchPrintf(szWinTitle, 2048, L"%s", szBuffer);
+	return szWinTitle;
+}
+
+std::wstring GetTimeStamp()
+{
+	SYSTEMTIME lt;
+	GetLocalTime(&lt);
+	wchar_t ts[TS_LEN];
+	StringCchPrintfW(ts, TS_LEN, L"%d-%02d-%02d %02d:%02d:%02d.%03d - ", lt.wYear, lt.wMonth, lt.wDay, lt.wHour, lt.wMinute, lt.wSecond, lt.wMilliseconds);
+	return ts;
+}
+
+void OpenLogFile()
+{
+	wchar_t lpszPath[PATH_LEN]{};
+	DWORD dwPathLength = GetModuleFileNameW(NULL, lpszPath, PATH_LEN);
+	DWORD dwError = GetLastError();
+	if (ERROR_INSUFFICIENT_BUFFER == dwError)
+	{
+		MessageBoxW(NULL, L"Path to logfile is too long! Working without logging.", (LPCWSTR)szTitle, MB_OK | MB_ICONWARNING);
+		return;
+	}
+	if (NULL == dwPathLength)
+	{
+		MessageBoxW(NULL, L"Can't get module filename! Working without logging.", (LPCWSTR)szTitle, MB_OK | MB_ICONWARNING);
+		return;
+	}
+
+	fs::path log_path = lpszPath;
+	log_path.replace_extension(L".log");
+	fs::path bak_path = log_path;
+	bak_path.replace_extension(L".bak");
+
+	if (fs::exists(log_path)) fs::rename(log_path, bak_path);
+#ifdef _DEBUG
+	log_path = L"d:\\test.log";
+#endif
+	logfile.open(log_path);
+	if (logfile.is_open())
+	{
+		diag_log(L"Start logging.");
+		diag_log(L"Logfile:", log_path);
+		diag_log(log_path, L"successfully opened.");
+	}
+	else
+	{
+		MessageBoxW(NULL, L"Can't open logfile! Working without logging.", (LPCWSTR)szTitle, MB_OK | MB_ICONWARNING);
+	}
+	return;
+}
+
+void CloseLogFile()
+{
+	if (logfile)
+	{
+		diag_log(L"End logging.");
+		logfile.close();
+	}
+}

wCenterWindow/Logger.h

@@ -0,0 +1,60 @@
+// wCenterWindow, v2.3.3
+// Logger.h
+//
+#pragma once
+#include "framework.h"
+
+extern std::wofstream logfile;
+std::wstring GetTimeStamp();
+std::wstring PrintTitle();
+
+template <typename T1>
+void diag_log(T1 arg1)
+{
+	logfile << GetTimeStamp() << arg1 << std::endl;
+}
+
+template <typename T1, typename T2>
+void diag_log(T1 arg1, T2 arg2)
+{
+	logfile << GetTimeStamp() << arg1 << ' ' << arg2 << std::endl;
+}
+
+template <typename T1, typename T2, typename T3>
+void diag_log(T1 arg1, T2 arg2, T3 arg3)
+{
+	logfile << GetTimeStamp() << arg1 << ' ' << arg2 << ' ' << arg3 << std::endl;
+}
+
+template <typename T1, typename T2, typename T3, typename T4>
+void diag_log(T1 arg1, T2 arg2, T3 arg3, T4 arg4)
+{
+	logfile << GetTimeStamp() << arg1 << ' ' << arg2 << ' ' << arg3 << ' ' << arg4 << std::endl;
+}
+
+template <typename T1, typename T2, typename T3, typename T4, typename T5>
+void diag_log(T1 arg1, T2 arg2, T3 arg3, T4 arg4, T5 arg5)
+{
+	logfile << GetTimeStamp() << arg1 << ' ' << arg2 << ' ' << arg3 << ' ' << arg4 << ' ' << arg5 << std::endl;
+}
+
+template <typename T1, typename T2, typename T3, typename T4, typename T5, typename T6>
+void diag_log(T1 arg1, T2 arg2, T3 arg3, T4 arg4, T5 arg5, T6 arg6)
+{
+	logfile << GetTimeStamp() << arg1 << ' ' << arg2 << ' ' << arg3 << ' ' << arg4 << ' ' << arg5 << ' ' << arg6 << std::endl;
+}
+
+template <typename T1, typename T2, typename T3, typename T4, typename T5, typename T6, typename T7>
+void diag_log(T1 arg1, T2 arg2, T3 arg3, T4 arg4, T5 arg5, T6 arg6, T7 arg7)
+{
+	logfile << GetTimeStamp() << arg1 << ' ' << arg2 << ' ' << arg3 << ' ' << arg4 << ' ' << arg5 << ' ' << arg6 << ' ' << arg7 << std::endl;
+}
+
+template <typename T1, typename T2, typename T3, typename T4, typename T5, typename T6, typename T7, typename T8>
+void diag_log(T1 arg1, T2 arg2, T3 arg3, T4 arg4, T5 arg5, T6 arg6, T7 arg7, T8 arg8)
+{
+	logfile << GetTimeStamp() << arg1 << ' ' << arg2 << ' ' << arg3 << ' ' << arg4 << ' ' << arg5 << ' ' << arg6 << ' ' << arg7 << ' ' << arg8 << std::endl;
+}
+
+void OpenLogFile();
+void CloseLogFile();

wCenterWindow/framework.h

@@ -5,13 +5,22 @@
 #pragma once
 
 #include "targetver.h"
-//#define WIN32_LEAN_AND_MEAN             // Exclude rarely-used stuff from Windows headers
+#define WIN32_LEAN_AND_MEAN             // Exclude rarely-used stuff from Windows headers
+
 // Windows Header Files
-#include <windows.h>
+#include <iostream>
+#include <fstream>
+#include <filesystem>
+#include <string>
 #include <strsafe.h>
+#include <windows.h>
+#include <shellapi.h>
 #include <CommCtrl.h>
+
 // C RunTime Header Files
-#include <stdlib.h>
+//#include <stdlib.h>
-#include <malloc.h>
+//#include <malloc.h>
-#include <memory.h>
+//#include <memory.h>
-#include <tchar.h>
+
+// Project Specific Header Files
+#include "Logger.h"

wCenterWindow/wCenterWindow.cpp

@@ -1,5 +1,8 @@
-// wCenterWindow, v2.2
+// wCenterWindow, v2.3.3
 //
+// TODO: More verbose logs - partially done.
+// TODO: More verbose error messages 
+// TODO: Window's title in logs (Impossible?)
 
 #include "framework.h"
 #include "wCenterWindow.h"
@@ -8,17 +11,19 @@
 #define KEY_C 0x43
 #define KEY_V 0x56
 
+#define BUF_LEN 4096
 #define MAX_LOADSTRING 50
 #define WM_WCW	0x8F00
 
 // Global variables:
 HINSTANCE			hInst;									// Instance
-TCHAR				szTitle[MAX_LOADSTRING];				// Window's title
+WCHAR				szTitle[MAX_LOADSTRING];				// Window's title
-TCHAR				szClass[MAX_LOADSTRING];				// Window's class
+WCHAR				szClass[MAX_LOADSTRING];				// Window's class
-TCHAR				szAbout[MAX_LOADSTRING * 12];			// Description text
+WCHAR				szAbout[MAX_LOADSTRING * 12];			// Description text
-TCHAR				szWinTitle[256];
+WCHAR				szWinTitle[256];
-TCHAR				szWinClass[256];
+WCHAR				szWinClass[256];
-TCHAR				szWinCore[] = TEXT("Windows.UI.Core.CoreWindow");
+WCHAR				szWinCore[] = TEXT("Windows.UI.Core.CoreWindow");
+HANDLE				hHeap = NULL;
 HHOOK				hMouseHook = NULL, hKbdHook = NULL;		// Hook's handles
 HICON				hIcon = NULL;
 HMENU				hMenu = NULL, hPopup = NULL;
@@ -27,9 +32,14 @@ BOOL				bKPressed = FALSE, bMPressed = FALSE, bShowIcon = TRUE, bWorkArea = TRUE
 BOOL				bLCTRL = FALSE, bLWIN = FALSE, bKEYV = FALSE;
 
 RECT				rcFW = { 0 };
-NOTIFYICONDATA		nid = { 0 };
+NOTIFYICONDATAW		nid = { 0 };
-LPKBDLLHOOKSTRUCT	pkhs;
+LPKBDLLHOOKSTRUCT	pkhs = { 0 };
-MENUITEMINFO		mii;
+MENUITEMINFO		mii = { 0 };
+
+LPVOID				szBuffer;
+
+// {2D7B7F30-4B5F-4380-9807-57D7A2E37F6C}
+static const GUID	guid = { 0x2d7b7f30, 0x4b5f, 0x4380, { 0x98, 0x7, 0x57, 0xd7, 0xa2, 0xe3, 0x7f, 0x6c } };
 
 // Forward declarations of functions included in this code module:
 VOID				HandlingTrayIcon();
@@ -43,31 +53,22 @@ INT_PTR CALLBACK    About(HWND, UINT, WPARAM, LPARAM);
 
 
 
-ATOM MyRegisterClass(HINSTANCE hInstance)
-{
-	WNDCLASSEX wcex = { 0 };
-	wcex.cbSize = sizeof(WNDCLASSEX);
-	wcex.lpfnWndProc = WndProc;
-	wcex.hInstance = hInstance;
-	wcex.hIcon = LoadIcon(hInstance, MAKEINTRESOURCE(IDI_TRAYICON));
-	wcex.hCursor = LoadCursor(nullptr, IDC_ARROW);
-	wcex.lpszClassName = szClass;
-	wcex.hIconSm = wcex.hIcon;
-	hIcon = wcex.hIcon;
-	return RegisterClassEx(&wcex);
-}
-
 VOID MoveWindowToMonitorCenter(HWND hwnd, BOOL bWorkArea, BOOL bResize)
 {
-	RECT fgwrc;
+	//diag_log(L"Entering MoveWindowToMonitorCenter(): hwnd =", hwnd, L"Title:", (LPWSTR)szBuffer);
+	diag_log(L"Entering MoveWindowToMonitorCenter(): hwnd =", hwnd);
+
+	RECT fgwrc = { 0 };
 	GetWindowRect(hwnd, &fgwrc);
 	LONG nWidth = fgwrc.right - fgwrc.left;
 	LONG nHeight = fgwrc.bottom - fgwrc.top;
 
-	MONITORINFO mi;
+	diag_log(L"Moving window from x =", fgwrc.left, L"y =", fgwrc.top);
+
+	MONITORINFO mi = { 0 };
 	mi.cbSize = sizeof(MONITORINFO);
-	GetMonitorInfo(MonitorFromWindow(hwnd, MONITOR_DEFAULTTONEAREST), &mi);
+	GetMonitorInfoW(MonitorFromWindow(hwnd, MONITOR_DEFAULTTONEAREST), &mi);
-	RECT area;
+	RECT area = { 0 };
 	if (bWorkArea)
 	{
 		area.bottom = mi.rcWork.bottom;
@@ -92,31 +93,51 @@ VOID MoveWindowToMonitorCenter(HWND hwnd, BOOL bWorkArea, BOOL bResize)
 	int x = (aw - nWidth) / 2;
 	int y = (ah - nHeight) / 2;
 
-	SendMessage(hwnd, WM_ENTERSIZEMOVE, NULL, NULL);
+	SendMessageW(hwnd, WM_ENTERSIZEMOVE, NULL, NULL);
 	MoveWindow(hwnd, x, y, nWidth, nHeight, TRUE);
-	SendMessage(hwnd, WM_EXITSIZEMOVE, NULL, NULL);
+	SendMessageW(hwnd, WM_EXITSIZEMOVE, NULL, NULL);
+
+	diag_log(L"Moving window to x =", x, L"y =", y);
+	diag_log(L"Quiting MoveWindowToMonitorCenter()");
 }
 
 
+ATOM MyRegisterClass(HINSTANCE hInstance)
+{
+	WNDCLASSEX wcex = { 0 };
+	wcex.cbSize = sizeof(WNDCLASSEX);
+	wcex.lpfnWndProc = WndProc;
+	wcex.hInstance = hInstance;
+	wcex.hIcon = LoadIconW(hInstance, MAKEINTRESOURCE(IDI_TRAYICON));
+	wcex.hCursor = LoadCursorW(nullptr, IDC_ARROW);
+	wcex.lpszClassName = szClass;
+	wcex.hIconSm = wcex.hIcon;
+	hIcon = wcex.hIcon;
+	return RegisterClassExW(&wcex);
+}
 
-int APIENTRY _tWinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPTSTR lpCmdLine, int nCmdShow)
+
+int APIENTRY wWinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPWSTR lpCmdLine, int nCmdShow)
 {
 	UNREFERENCED_PARAMETER(hPrevInstance);
 	UNREFERENCED_PARAMETER(lpCmdLine);
 
+	OpenLogFile();
+	diag_log(L"Entering WinMain()");
+
 	hInst = hInstance;
 
-	LoadString(hInstance, IDS_APP_TITLE, szTitle, _countof(szTitle));
+	LoadStringW(hInstance, IDS_APP_TITLE, szTitle, _countof(szTitle));
-	LoadString(hInstance, IDS_CLASSNAME, szClass, _countof(szClass));
+	LoadStringW(hInstance, IDS_CLASSNAME, szClass, _countof(szClass));
 
-	if (FindWindow(szClass, NULL))
+	if (FindWindowW(szClass, NULL))
 	{
 		ShowError(IDS_RUNNING);
 		return FALSE;
 	}
 
 	MyRegisterClass(hInstance);
-	hWnd = CreateWindowEx(0, szClass, szTitle, 0, 0, 0, 0, 0, NULL, NULL, hInstance, NULL);
+	hWnd = CreateWindowExW(0, szClass, szTitle, 0, 0, 0, 0, 0, HWND_MESSAGE, NULL, hInstance, NULL);
 	if (!hWnd)
 	{
 		ShowError(IDS_ERR_WND);
@@ -125,18 +146,28 @@ int APIENTRY _tWinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPTSTR lpCm
 
 	int nArgs = 0;
 	LPWSTR* szArglist = CommandLineToArgvW(GetCommandLineW(), &nArgs);
-	(nArgs >= 2 && 0 == lstrcmpiW(szArglist[1], TEXT("/hide"))) ? bShowIcon = FALSE : bShowIcon = TRUE;
+
+	diag_log(L"Arguments:", nArgs - 1);
+	for (int i = 1; i < nArgs; i++)
+	{
+		diag_log(L"Argument", i, L":", szArglist[i]);
+	}
+
+	(nArgs >= 2 && 0 == lstrcmpiW(szArglist[1], L"/hide")) ? bShowIcon = FALSE : bShowIcon = TRUE;
 	LocalFree(szArglist);
 	HandlingTrayIcon();
 
-	hTaskBar = FindWindow(TEXT("Shell_TrayWnd"), NULL);
+	hHeap = GetProcessHeap();
-	hProgman = FindWindow(TEXT("Progman"), NULL);
+	szBuffer = HeapAlloc(hHeap, HEAP_ZERO_MEMORY, BUF_LEN);
+
+	hTaskBar = FindWindowW(L"Shell_TrayWnd", NULL);
+	hProgman = FindWindowW(L"Progman", NULL);
 	hDesktop = GetDesktopWindow();
 
 	MSG msg;
 	BOOL bRet;
 
-	while ((bRet = GetMessage(&msg, NULL, 0, 0)) != 0)
+	while ((bRet = GetMessageW(&msg, NULL, 0, 0)) != 0)
 	{
 		if (bRet == -1)
 		{
@@ -146,14 +177,18 @@ int APIENTRY _tWinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPTSTR lpCm
 		else
 		{
 			TranslateMessage(&msg);
-			DispatchMessage(&msg);
+			DispatchMessageW(&msg);
 		}
 	}
 
 	if (hMouseHook) UnhookWindowsHookEx(hMouseHook);
 	if (hKbdHook) UnhookWindowsHookEx(hKbdHook);
 	if (hMenu) DestroyMenu(hMenu);
-	Shell_NotifyIcon(NIM_DELETE, &nid);
+	Shell_NotifyIconW(NIM_DELETE, &nid);
+
+	diag_log(L"Quiting WinMain(), msg.wParam =", (int)msg.wParam);
+	CloseLogFile();
+	HeapFree(hHeap, NULL, szBuffer);
 
 	return (int)msg.wParam;
 }
@@ -164,52 +199,62 @@ LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
 	{
 		case WM_CREATE:
 		{
-			hMenu = LoadMenu(hInst, MAKEINTRESOURCE(IDR_MENU));
+			hMenu = LoadMenuW(hInst, MAKEINTRESOURCE(IDR_MENU));
 			if (!hMenu)
 			{
+				diag_log(L"Loading context menu failed!");
 				ShowError(IDS_ERR_MENU);
-				SendMessage(hWnd, WM_CLOSE, NULL, NULL);
+				SendMessageW(hWnd, WM_CLOSE, NULL, NULL);
 			}
+			diag_log(L"Context menu successfully loaded");
 
 			hPopup = GetSubMenu(hMenu, 0);
 			if (!hPopup)
 			{
+				diag_log(L"Creating popup menu failed!");
 				ShowError(IDS_ERR_POPUP);
-				SendMessage(hWnd, WM_CLOSE, NULL, NULL);
+				SendMessageW(hWnd, WM_CLOSE, NULL, NULL);
 			}
+			diag_log(L"Popup menu successfully created");
 
 			mii.cbSize = sizeof(MENUITEMINFO);
 			mii.fMask = MIIM_STATE;
 			bWorkArea ? mii.fState = MFS_CHECKED : mii.fState = MFS_UNCHECKED;
-			SetMenuItemInfo(hPopup, ID_POPUPMENU_AREA, FALSE, &mii);
+			SetMenuItemInfoW(hPopup, ID_POPUPMENU_AREA, FALSE, &mii);
 
-			nid.cbSize = sizeof(NOTIFYICONDATA);
+			nid.cbSize = sizeof(NOTIFYICONDATAW);
 			nid.hWnd = hWnd;
 			nid.uVersion = NOTIFYICON_VERSION;
 			nid.uCallbackMessage = WM_WCW;
 			nid.hIcon = hIcon;
 			nid.uID = IDI_TRAYICON;
 			nid.uFlags = NIF_ICON | NIF_MESSAGE | NIF_TIP;
-			nid.dwInfoFlags = NIIF_INFO;
+			nid.dwInfoFlags = NIIF_NONE;
-			StringCchCopy(nid.szTip, _countof(nid.szTip), szTitle);
+			nid.dwState = NIS_HIDDEN;
+			nid.dwStateMask = NIS_HIDDEN;
+			StringCchCopyW(nid.szTip, _countof(nid.szTip), szTitle);
 
-			hMouseHook = SetWindowsHookEx(WH_MOUSE_LL, MouseHookProc, hInst, NULL);
+			hMouseHook = SetWindowsHookExW(WH_MOUSE_LL, MouseHookProc, hInst, NULL);
 			if (!hMouseHook)
 			{
+				diag_log(L"Creating mouse hook failed!");
 				ShowError(IDS_ERR_HOOK);
-				SendMessage(hWnd, WM_CLOSE, NULL, NULL);
+				SendMessageW(hWnd, WM_CLOSE, NULL, NULL);
 			}
+			diag_log(L"Mouse hook was successfully set");
 
-			hKbdHook = SetWindowsHookEx(WH_KEYBOARD_LL, KeyboardHookProc, hInst, NULL);
+			hKbdHook = SetWindowsHookExW(WH_KEYBOARD_LL, KeyboardHookProc, hInst, NULL);
 			if (!hKbdHook)
 			{
+				diag_log(L"Creating keyboard hook failed!");
 				ShowError(IDS_ERR_HOOK);
-				SendMessage(hWnd, WM_CLOSE, NULL, NULL);
+				SendMessageW(hWnd, WM_CLOSE, NULL, NULL);
 			}
+			diag_log(L"Keyboard hook was successfully set");
 
-			LoadString(hInst, IDS_ABOUT, szAbout, _countof(szAbout));
+			LoadStringW(hInst, IDS_ABOUT, szAbout, _countof(szAbout));
+			break;
 		}
-		break;
 
 		case WM_WCW:
 		{
@@ -228,27 +273,29 @@ LRESULT CALLBACK WndProc(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam)
 				{
 					bWorkArea = !bWorkArea;
 					bWorkArea ? mii.fState = MFS_CHECKED : mii.fState = MFS_UNCHECKED;
-					SetMenuItemInfo(hPopup, ID_POPUPMENU_AREA, FALSE, &mii);
+					SetMenuItemInfoW(hPopup, ID_POPUPMENU_AREA, FALSE, &mii);
+					diag_log(L"Changed 'Use workarea' option to", bWorkArea);
 				}
 				if (idMenu == ID_POPUPMENU_ABOUT && !bKPressed)
 				{
 					bKPressed = TRUE;
-					DialogBox(hInst, MAKEINTRESOURCE(IDD_ABOUTBOX), hWnd, (DLGPROC)About);
+					diag_log(L"Opening 'About' dialog");
+					DialogBoxW(hInst, MAKEINTRESOURCE(IDD_ABOUTBOX), hWnd, (DLGPROC)About);
 					bKPressed = FALSE;
 				}
-				if (idMenu == ID_POPUPMENU_EXIT) SendMessage(hWnd, WM_CLOSE, NULL, NULL);
+				if (idMenu == ID_POPUPMENU_EXIT) SendMessageW(hWnd, WM_CLOSE, NULL, NULL);
 			}
+			break;
 		}
-		break;
 
 		case WM_DESTROY:
 		{
 			PostQuitMessage(0);
+			break;
 		}
-		break;
 
 		default:
-			return DefWindowProc(hWnd, message, wParam, lParam);
+			return DefWindowProcW(hWnd, message, wParam, lParam);
 	}
 	return 0;
 }
@@ -258,6 +305,7 @@ LRESULT CALLBACK MouseHookProc(int nCode, WPARAM wParam, LPARAM lParam)
 	if (wParam == WM_MBUTTONUP) bMPressed = FALSE;
 	if (wParam == WM_MBUTTONDOWN && bLCTRL && bLWIN && !bMPressed)
 	{
+		diag_log(L"Pressed LCTRL + LWIN + MMB");
 		bMPressed = TRUE;
 		hFgWnd = GetForegroundWindow();
 		BOOL bApproved = CheckWindow(hFgWnd);
@@ -285,6 +333,7 @@ LRESULT CALLBACK KeyboardHookProc(int nCode, WPARAM wParam, LPARAM lParam)
 
 		if (bLCTRL && bLWIN && pkhs->vkCode == KEY_I && !bKPressed)	// 'I' key
 		{
+			diag_log(L"Pressed LCTRL + LWIN + I");
 			bKPressed = TRUE;
 			bShowIcon = !bShowIcon;
 			HandlingTrayIcon();
@@ -293,6 +342,7 @@ LRESULT CALLBACK KeyboardHookProc(int nCode, WPARAM wParam, LPARAM lParam)
 
 		if (bLCTRL && bLWIN && pkhs->vkCode == KEY_C && !bKPressed && !bKEYV)	// 'C' key
 		{
+			diag_log(L"Pressed LCTRL + LWIN + C");
 			bKPressed = TRUE;
 			hFgWnd = GetForegroundWindow();
 			BOOL bApproved = CheckWindow(hFgWnd);
@@ -303,10 +353,16 @@ LRESULT CALLBACK KeyboardHookProc(int nCode, WPARAM wParam, LPARAM lParam)
 
 		if (bLCTRL && bLWIN && pkhs->vkCode == KEY_V && !bKPressed && !bKEYV)	// 'V' key
 		{
+			diag_log(L"Pressed LCTRL + LWIN + V");
 			bKPressed = TRUE; bKEYV = TRUE;
 			hFgWnd = GetForegroundWindow();
 			BOOL bApproved = CheckWindow(hFgWnd);
-			if (bApproved) DialogBox(hInst, MAKEINTRESOURCE(IDD_MANUAL_EDITING), hWnd, (DLGPROC)DlgProc);
+			if (bApproved)
+			{
+				diag_log(L"Opening 'Manual editing' dialog");
+				DialogBoxW(hInst, MAKEINTRESOURCE(IDD_MANUAL_EDITING), hFgWnd, (DLGPROC)DlgProc);
+				SetForegroundWindow(hFgWnd);
+			}
 			else hFgWnd = NULL;
 			bKEYV = FALSE;
 			return TRUE;
@@ -322,9 +378,9 @@ BOOL CALLBACK DlgProc(HWND hDlg, UINT dlgmsg, WPARAM wParam, LPARAM lParam)
 	{
 		case WM_INITDIALOG:
 		{
-			SetWindowText(hDlg, szTitle);
+			SetWindowTextW(hDlg, szTitle);
-			GetWindowText(hFgWnd, szWinTitle, _countof(szWinTitle));
+			GetWindowTextW(hFgWnd, szWinTitle, _countof(szWinTitle));
-			GetClassName(hFgWnd, szWinClass, _countof(szWinClass));
+			GetClassNameW(hFgWnd, szWinClass, _countof(szWinClass));
 			GetWindowRect(hFgWnd, &rcFW);
 			x = rcFW.left;
 			y = rcFW.top;
@@ -334,8 +390,8 @@ BOOL CALLBACK DlgProc(HWND hDlg, UINT dlgmsg, WPARAM wParam, LPARAM lParam)
 			SetDlgItemInt(hDlg, IDC_EDIT_Y, y, TRUE);
 			SetDlgItemInt(hDlg, IDC_EDIT_WIDTH, w, FALSE);
 			SetDlgItemInt(hDlg, IDC_EDIT_HEIGHT, h, FALSE);
-			SetDlgItemText(hDlg, IDC_EDIT_TITLE, szWinTitle);
+			SetDlgItemTextW(hDlg, IDC_EDIT_TITLE, szWinTitle);
-			SetDlgItemText(hDlg, IDC_EDIT_CLASS, szWinClass);
+			SetDlgItemTextW(hDlg, IDC_EDIT_CLASS, szWinClass);
 			UpdateWindow(hDlg);
 			break;
 		}
@@ -348,9 +404,9 @@ BOOL CALLBACK DlgProc(HWND hDlg, UINT dlgmsg, WPARAM wParam, LPARAM lParam)
 					y = GetDlgItemInt(hDlg, IDC_EDIT_Y, NULL, TRUE);
 					w = GetDlgItemInt(hDlg, IDC_EDIT_WIDTH, NULL, FALSE);
 					h = GetDlgItemInt(hDlg, IDC_EDIT_HEIGHT, NULL, FALSE);
-					SendMessage(hFgWnd, WM_ENTERSIZEMOVE, NULL, NULL);
+					SendMessageW(hFgWnd, WM_ENTERSIZEMOVE, NULL, NULL);
 					MoveWindow(hFgWnd, x, y, w, h, TRUE);
-					SendMessage(hFgWnd, WM_EXITSIZEMOVE, NULL, NULL);
+					SendMessageW(hFgWnd, WM_EXITSIZEMOVE, NULL, NULL);
 					return TRUE;
 					break;
 				}
@@ -358,6 +414,7 @@ BOOL CALLBACK DlgProc(HWND hDlg, UINT dlgmsg, WPARAM wParam, LPARAM lParam)
 				case IDC_BUTTON_CLOSE:
 				{
 					EndDialog(hDlg, LOWORD(wParam));
+					diag_log(L"Closing 'Manual editing' dialog");
 					break;
 				}
 			}
@@ -367,47 +424,62 @@ BOOL CALLBACK DlgProc(HWND hDlg, UINT dlgmsg, WPARAM wParam, LPARAM lParam)
 
 BOOL CheckWindow(HWND hFW)
 {
-	GetClassName(hFW, szWinClass, _countof(szWinClass));
+	diag_log(L"Entering CheckWindow(), hwnd=", hFW);
+	GetClassNameW(hFW, szWinClass, _countof(szWinClass));
 	if (hFW)
 	{
-		if (_tcscmp(szWinClass, szWinCore) != 0)
+		if (wcscmp(szWinClass, szWinCore) != 0)
 		{
 			if (hFW != hDesktop && hFW != hTaskBar && hFW != hProgman)
 			{
-				if (!IsIconic(hFW) && !IsZoomed(hFW)) return TRUE;
+				if (!IsIconic(hFW) && !IsZoomed(hFW))
+				{
+					GetWindowTextW(hFW, (LPWSTR)szBuffer, BUF_LEN - sizeof(WCHAR));
+					diag_log(L"Window with hwnd=", hFW, L"is approved");
+					return TRUE;
+				}
 				else ShowError(IDS_ERR_MAXMIN);
 			}
 		}
 	}
+	diag_log(L"Window with hwnd=", hFW, L"is not approved!");
+	diag_log(L"Quiting CheckWindow()");
 	return FALSE;
 }
 
 VOID HandlingTrayIcon()
 {
+	diag_log(L"Entering HandlingTrayIcon(), bShowIcon =", bShowIcon);
 	if (bShowIcon)
 	{
-		if (!Shell_NotifyIcon(NIM_ADD, &nid))
+		BOOL bResult1 = Shell_NotifyIconW(NIM_ADD, &nid);
+		diag_log(L"Shell_NotifyIconW(NIM_ADD):", bResult1);
+		BOOL bResult2 = Shell_NotifyIconW(NIM_SETVERSION, &nid);
+		diag_log(L"Shell_NotifyIconW(NIM_SETVERSION):", bResult2);
+		if (!bResult1 || !bResult2)
 		{
+			diag_log(L"Error creating trayicon.");
 			ShowError(IDS_ERR_ICON);
 			bShowIcon = FALSE;
 		}
 	}
 	else
 	{
-		Shell_NotifyIcon(NIM_DELETE, &nid);
+		Shell_NotifyIconW(NIM_DELETE, &nid);
 	}
+	diag_log(L"Quiting HandlingTrayIcon()");
 }
 
 VOID ShowError(UINT uID)
 {
-	TCHAR szErrorText[MAX_LOADSTRING]; // Error's text
+	WCHAR szErrorText[MAX_LOADSTRING]; // Error's text
-	LoadString(hInst, uID, szErrorText, _countof(szErrorText));
+	LoadStringW(hInst, uID, szErrorText, _countof(szErrorText));
-	MessageBox(hWnd, szErrorText, szTitle, MB_OK | MB_ICONERROR | MB_TOPMOST);
+	MessageBoxW(hWnd, szErrorText, szTitle, MB_OK | MB_ICONERROR | MB_TOPMOST);
 }
 
 INT_PTR CALLBACK About(HWND hDlg, UINT message, WPARAM wParam, LPARAM lParam)
 {
-	INITCOMMONCONTROLSEX icex;
+	INITCOMMONCONTROLSEX icex = { 0 };
 	icex.dwSize = sizeof(INITCOMMONCONTROLSEX);
 	icex.dwICC = ICC_LINK_CLASS;
 	InitCommonControlsEx(&icex);
@@ -416,7 +488,7 @@ INT_PTR CALLBACK About(HWND hDlg, UINT message, WPARAM wParam, LPARAM lParam)
 	{
 		case WM_INITDIALOG:
 		{
-			SetDlgItemText(hDlg, IDC_ABOUTHELP, szAbout);
+			SetDlgItemTextW(hDlg, IDC_ABOUTHELP, szAbout);
 			return (INT_PTR)TRUE;
 			break;
 		}
@@ -424,18 +496,13 @@ INT_PTR CALLBACK About(HWND hDlg, UINT message, WPARAM wParam, LPARAM lParam)
 		case WM_NOTIFY:
 		{
 			LPNMHDR pNMHdr = (LPNMHDR)lParam;
-			switch (pNMHdr->code)
+			if ((NM_CLICK == pNMHdr->code || NM_RETURN == pNMHdr->code) && IDC_DONATIONLINK == pNMHdr->idFrom)
 			{
-				case NM_CLICK:
+				PNMLINK pNMLink = (PNMLINK)pNMHdr;
-				case NM_RETURN:
+				LITEM item = pNMLink->item;
-					if (pNMHdr->idFrom == IDC_DONATIONLINK)
+				ShellExecuteW(NULL, L"open", item.szUrl, NULL, NULL, SW_SHOW);
-					{
+				diag_log(L"Pressed donation link");
-						PNMLINK pNMLink = (PNMLINK)pNMHdr;
+				return (INT_PTR)TRUE;
-						LITEM item = pNMLink->item;
-						ShellExecute(NULL, L"open", item.szUrl, NULL, NULL, SW_SHOW);
-						return (INT_PTR)TRUE;
-					}
-					break;
 			}
 			break;
 		}
@@ -444,9 +511,10 @@ INT_PTR CALLBACK About(HWND hDlg, UINT message, WPARAM wParam, LPARAM lParam)
 			if (LOWORD(wParam) == IDOK || LOWORD(wParam) == IDCANCEL)
 			{
 				EndDialog(hDlg, LOWORD(wParam));
+				diag_log(L"Closing 'About' dialog");
 				return (INT_PTR)TRUE;
-				break;
 			}
+			break;
 	}
 	return (INT_PTR)FALSE;
 }

wCenterWindow/wCenterWindow.manifest

@@ -0,0 +1,48 @@
+<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
+<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0" xmlns:asmv3="urn:schemas-microsoft-com:asm.v3">
+
+  <description>Centers windows by hotkey (C++)</description>
+
+  <assemblyIdentity
+      version="2.3.3.0"
+      processorArchitecture="x86"
+      name="W0LF.wCenterWindow.C++"
+      type="win32">
+  </assemblyIdentity>
+
+  <dependency>
+    <dependentAssembly>
+      <assemblyIdentity
+        type="Win32" 
+        name="Microsoft.Windows.Common-Controls" 
+        version="6.0.0.0" 
+        publicKeyToken="6595b64144ccf1df" 
+        language="*" 
+        processorArchitecture="*"/>
+    </dependentAssembly>
+  </dependency>
+
+  <compatibility xmlns="urn:schemas-microsoft-com:compatibility.v1">
+    <application>
+      <!--ID Windows Vista -->
+      <supportedOS Id="{e2011457-1546-43c5-a5fe-008deee3d3f0}"/>
+      <!--ID Windows 7 -->
+      <supportedOS Id="{35138b9a-5d96-4fbd-8e2d-a2440225f93a}"/>
+      <!--ID Windows 8 -->
+      <supportedOS Id="{4a2f28e3-53b9-4441-ba9c-d69d4a4a6e38}"/>
+      <!--ID Windows 8.1 -->
+      <supportedOS Id="{1f676c76-80e1-4239-95bb-83d0f6d0da78}"/>
+      <!--ID Windows 10 -->
+      <supportedOS Id="{8e0f7a12-bfb3-4fe8-b9a5-48fd50a15a9a}"/>
+    </application>
+  </compatibility>
+
+  <trustInfo xmlns="urn:schemas-microsoft-com:asm.v3">
+    <security>
+      <requestedPrivileges>
+        <requestedExecutionLevel level="asInvoker" uiAccess="false"></requestedExecutionLevel>
+      </requestedPrivileges>
+    </security>
+  </trustInfo>
+
+</assembly>

wCenterWindow/wCenterWindow.vcxproj

@@ -24,31 +24,32 @@
     <ProjectGuid>{f1a1603a-f5d0-47b8-8e4b-cf17747bcfba}</ProjectGuid>
     <RootNamespace>wCenterWindow</RootNamespace>
     <WindowsTargetPlatformVersion>10.0</WindowsTargetPlatformVersion>
+    <XPDeprecationWarning>false</XPDeprecationWarning>
   </PropertyGroup>
   <Import Project="$(VCTargetsPath)\Microsoft.Cpp.Default.props" />
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'" Label="Configuration">
     <ConfigurationType>Application</ConfigurationType>
     <UseDebugLibraries>true</UseDebugLibraries>
-    <PlatformToolset>v142</PlatformToolset>
+    <PlatformToolset>v143</PlatformToolset>
     <CharacterSet>Unicode</CharacterSet>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|Win32'" Label="Configuration">
     <ConfigurationType>Application</ConfigurationType>
     <UseDebugLibraries>false</UseDebugLibraries>
-    <PlatformToolset>v142</PlatformToolset>
+    <PlatformToolset>v143</PlatformToolset>
     <WholeProgramOptimization>true</WholeProgramOptimization>
     <CharacterSet>Unicode</CharacterSet>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'" Label="Configuration">
     <ConfigurationType>Application</ConfigurationType>
     <UseDebugLibraries>true</UseDebugLibraries>
-    <PlatformToolset>v142</PlatformToolset>
+    <PlatformToolset>v143</PlatformToolset>
     <CharacterSet>Unicode</CharacterSet>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'" Label="Configuration">
     <ConfigurationType>Application</ConfigurationType>
     <UseDebugLibraries>false</UseDebugLibraries>
-    <PlatformToolset>v142</PlatformToolset>
+    <PlatformToolset>v143</PlatformToolset>
     <WholeProgramOptimization>true</WholeProgramOptimization>
     <CharacterSet>Unicode</CharacterSet>
   </PropertyGroup>
@@ -88,7 +89,8 @@
       <WarningLevel>Level3</WarningLevel>
       <SDLCheck>true</SDLCheck>
       <PreprocessorDefinitions>WIN32;_DEBUG;_WINDOWS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
-      <ConformanceMode>true</ConformanceMode>
+      <ConformanceMode>false</ConformanceMode>
+      <LanguageStandard>stdcpp17</LanguageStandard>
     </ClCompile>
     <Link>
       <SubSystem>Windows</SubSystem>
@@ -108,6 +110,7 @@
       <PreprocessorDefinitions>WIN32;NDEBUG;_WINDOWS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
       <ConformanceMode>true</ConformanceMode>
       <RuntimeLibrary>MultiThreaded</RuntimeLibrary>
+      <LanguageStandard>stdcpp17</LanguageStandard>
     </ClCompile>
     <Link>
       <SubSystem>Windows</SubSystem>
@@ -126,10 +129,12 @@
       <SDLCheck>true</SDLCheck>
       <PreprocessorDefinitions>_DEBUG;_WINDOWS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
       <ConformanceMode>true</ConformanceMode>
+      <LanguageStandard>stdcpp17</LanguageStandard>
     </ClCompile>
     <Link>
       <SubSystem>Windows</SubSystem>
       <GenerateDebugInformation>true</GenerateDebugInformation>
+      <AdditionalDependencies>comctl32.lib;%(AdditionalDependencies)</AdditionalDependencies>
     </Link>
   </ItemDefinitionGroup>
   <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'">
@@ -140,21 +145,26 @@
       <SDLCheck>true</SDLCheck>
       <PreprocessorDefinitions>NDEBUG;_WINDOWS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
       <ConformanceMode>true</ConformanceMode>
+      <LanguageStandard>stdcpp17</LanguageStandard>
+      <RuntimeLibrary>MultiThreaded</RuntimeLibrary>
     </ClCompile>
     <Link>
       <SubSystem>Windows</SubSystem>
       <EnableCOMDATFolding>true</EnableCOMDATFolding>
       <OptimizeReferences>true</OptimizeReferences>
       <GenerateDebugInformation>true</GenerateDebugInformation>
+      <AdditionalDependencies>comctl32.lib;%(AdditionalDependencies)</AdditionalDependencies>
     </Link>
   </ItemDefinitionGroup>
   <ItemGroup>
     <ClInclude Include="framework.h" />
+    <ClInclude Include="Logger.h" />
     <ClInclude Include="Resource.h" />
     <ClInclude Include="targetver.h" />
     <ClInclude Include="wCenterWindow.h" />
   </ItemGroup>
   <ItemGroup>
+    <ClCompile Include="Logger.cpp" />
     <ClCompile Include="wCenterWindow.cpp" />
   </ItemGroup>
   <ItemGroup>

wCenterWindow/wCenterWindow.vcxproj.filters

@@ -27,11 +27,17 @@
     <ClInclude Include="wCenterWindow.h">
       <Filter>Header Files</Filter>
     </ClInclude>
+    <ClInclude Include="Logger.h">
+      <Filter>Header Files</Filter>
+    </ClInclude>
   </ItemGroup>
   <ItemGroup>
     <ClCompile Include="wCenterWindow.cpp">
       <Filter>Source Files</Filter>
     </ClCompile>
+    <ClCompile Include="Logger.cpp">
+      <Filter>Source Files</Filter>
+    </ClCompile>
   </ItemGroup>
   <ItemGroup>
     <ResourceCompile Include="wCenterWindow.rc">

wCenterWindow/wCenterWindow.vcxproj.user

@@ -1,4 +1,7 @@
 <?xml version="1.0" encoding="utf-8"?>
 <Project ToolsVersion="Current" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
-  <PropertyGroup />
+  <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'">
+    <LocalDebuggerCommandArguments>/test1 /test2</LocalDebuggerCommandArguments>
+    <DebuggerFlavor>WindowsLocalDebugger</DebuggerFlavor>
+  </PropertyGroup>
 </Project>