shithub: cstory

Download patch

ref: bd876e9309acfa451fa31f3576a9bca71a5e9759
parent: f127010848d7f03ad527cf0c24fba4d931c723c8
author: Clownacy <[email protected]>
date: Tue Jan 28 16:26:28 EST 2020

Change icon resource IDs

Visual Studio 2017 doesn't like them being 0 and 1: the compiled EXE
uses the small icon as the taskbar icon. Changing them to 101 and
102, like CSE2 did before the accurate-portable-split, fixes this.

--- a/assets/resources/CSE2.rc
+++ b/assets/resources/CSE2.rc
@@ -66,8 +66,8 @@
 
 // Icon with lowest ID value placed first to ensure application icon
 // remains consistent on all systems.
-0                       ICON                    "ICON/0.ico"
-1                       ICON                    "ICON/ICON_MINI.ico"
+101                       ICON                    "ICON/0.ico"
+102                       ICON                    "ICON/ICON_MINI.ico"
 #endif    // Japanese resources
 /////////////////////////////////////////////////////////////////////////////
 
--- a/src/Main.cpp
+++ b/src/Main.cpp
@@ -194,8 +194,8 @@
 	RECT unused_rect = {0, 0, WINDOW_WIDTH, WINDOW_HEIGHT};
 
 #ifdef _WIN32	// On Windows, we use native icons instead (so we can give the taskbar and window separate icons, like the original EXE does)
-	SDL_SetHint(SDL_HINT_WINDOWS_INTRESOURCE_ICON, "0");
-	SDL_SetHint(SDL_HINT_WINDOWS_INTRESOURCE_ICON_SMALL, "1");
+	SDL_SetHint(SDL_HINT_WINDOWS_INTRESOURCE_ICON, "101");
+	SDL_SetHint(SDL_HINT_WINDOWS_INTRESOURCE_ICON_SMALL, "102");
 #endif
 
 	SDL_InitSubSystem(SDL_INIT_VIDEO);