Merge branch 'master' of https://github.com/Blitz54/PathOfBuilding-SimpleGraphic into fontin-fonts

Blitz54 · Blitz54 · commit bd84bf1a0f84 · 2025-10-25T17:45:33.000-05:00
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -26,6 +26,13 @@ jobs:
         with:
           submodules: "recursive"
 
+      - name: Append short commit sha to CFG_VERSION_NUM in config.h
+        shell: powershell
+        run: |
+          $file = 'config.h'
+          $short=$(git rev-parse --short HEAD)
+          (Get-Content $file -Raw) -replace '(#define\s+CFG_VERSION_NUM\s+"[^"]*)(")', "`$1-$short`$2" | Set-Content $file
+
       - name: run-vcpkg
         uses: lukka/run-vcpkg@v11
         with:
@@ -45,7 +52,15 @@ jobs:
 
       - name: Build DLL
         run: "cmake --build build --config Release -t INSTALL"
-        
+
+      - name: Show DLL resources
+        run: |
+          $dll = "install-prefix\SimpleGraphic.dll"
+          Write-Host "DLL path: $dll"
+          if (Test-Path $dll) {
+            Write-Host "VersionInfo:"; (Get-Item $dll).VersionInfo | Format-List
+          } else { Write-Host "DLL not found" }
+
       - name: Archive DLL
         uses: actions/upload-artifact@v4
         with:
diff --git a/.gitignore b/.gitignore
@@ -1 +1,3 @@
-/build/
+/build/
+/build-SimpleGraphic
+/.idea
diff --git a/config.h b/config.h
@@ -4,7 +4,7 @@
 // Configuration
 //
 
-#define CFG_VERSION_NUM "2.2"
+#define CFG_VERSION_NUM "2.4"
 #define CFG_VERSION		"SimpleGraphic v" CFG_VERSION_NUM
 #define CFG_TITLE		"SimpleGraphic"
 #define CFG_LOGFILE		"SimpleGraphic/SimpleGraphic.log"
diff --git a/engine/render.h b/engine/render.h
@@ -105,6 +105,8 @@ class r_IRenderer {
 	virtual int		VirtualScreenWidth() = 0;
 	virtual int		VirtualScreenHeight() = 0;
 	virtual float	VirtualScreenScaleFactor() = 0;
+	virtual void	SetDpiScaleOverridePercent(int percent) = 0;
+	virtual int		DpiScaleOverridePercent() const = 0;
 	virtual int		VirtualMap(int properValue) = 0;
 	virtual int		VirtualUnmap(int mappedValue) = 0;
 	
diff --git a/engine/render/r_font.cpp b/engine/render/r_font.cpp
@@ -10,6 +10,7 @@
 #include <iostream>
 #include <fstream>
 #include <string>
+#include <cmath>
 
 // =======
 // Classes
@@ -144,12 +145,12 @@ int r_font_c::StringWidthInternal(f_fontHeight_s* fh, std::u32string_view str, i
 	int heightIdx = (int)(std::find(fontHeights, fontHeights + numFontHeight, fh) - fontHeights);
 	auto tofuFont = FindSmallerFontHeight(height, heightIdx, tofuSizeReduction);
 
-	auto measureCodepoint = [](f_fontHeight_s* fh, char32_t cp) {
-		auto& glyph = fh->Glyph((char)(unsigned char)cp);
+	auto measureCodepoint = [](f_fontHeight_s* glyphFh, char32_t cp) {
+		auto& glyph = glyphFh->Glyph((char)(unsigned char)cp);
 		return glyph.width + glyph.spLeft + glyph.spRight;
 	};
 
-	int width = 0;
+	float width = 0.0f;
 	for (size_t idx = 0; idx < str.size();) {
 		auto ch = str[idx];
 		int escLen = IsColorEscape(str.substr(idx));
@@ -158,23 +159,26 @@ int r_font_c::StringWidthInternal(f_fontHeight_s* fh, std::u32string_view str, i
 		}
 		else if (ch >= (unsigned)fh->numGlyph) {
 			auto tofu = BuildTofuString(ch);
-			for (auto ch : tofu) {
-				width += measureCodepoint(tofuFont.fh, ch);
+			for (auto cp : tofu) {
+				width += measureCodepoint(tofuFont.fh, cp);
+				width = std::ceil(width);
 			}
 			++idx;
 		}
 		else if (ch == U'\t') {
 			auto& glyph = fh->Glyph(' ');
 			int spWidth = glyph.width + glyph.spLeft + glyph.spRight;
-			width += (int)(spWidth * 4 * scale);
+			width += spWidth * 4 * scale;
+			width = std::ceil(width);
 			++idx;
 		}
 		else {
-			width += (int)(measureCodepoint(fh, ch) * scale);
+			width += measureCodepoint(fh, ch) * scale;
+			width = std::ceil(width);
 			++idx;
 		}
 	}
-	return width;
+	return static_cast<int>(width);
 }
 
 int r_font_c::StringWidth(int height, std::u32string_view str)
@@ -187,7 +191,7 @@ int r_font_c::StringWidth(int height, std::u32string_view str)
 		auto lineEnd = std::find(I, str.end(), U'\n');
 		if (I != lineEnd) {
 			std::u32string_view line(&*I, std::distance(I, lineEnd));
-			int lw = (int)StringWidthInternal(fh, line, height, scale);
+			int lw = StringWidthInternal(fh, line, height, scale);
 			max = (std::max)(max, lw);
 		}
 		if (lineEnd == str.end()) {
@@ -203,12 +207,12 @@ size_t r_font_c::StringCursorInternal(f_fontHeight_s* fh, std::u32string_view st
 	int heightIdx = (int)(std::find(fontHeights, fontHeights + numFontHeight, fh) - fontHeights);
 	auto tofuFont = FindSmallerFontHeight(height, heightIdx, tofuSizeReduction);
 
-	auto measureCodepoint = [](f_fontHeight_s* fh, char32_t cp) {
-		auto& glyph = fh->Glyph((char)(unsigned char)cp);
+	auto measureCodepoint = [](f_fontHeight_s* glyphFh, char32_t cp) {
+		auto& glyph = glyphFh->Glyph((char)(unsigned char)cp);
 		return glyph.width + glyph.spLeft + glyph.spRight;
 	};
 
-	int x = 0;
+	float x = 0.0f;
 	auto I = str.begin();
 	auto lineEnd = std::find(I, str.end(), U'\n');
 	while (I != lineEnd) {
@@ -219,32 +223,34 @@ size_t r_font_c::StringCursorInternal(f_fontHeight_s* fh, std::u32string_view st
 		}
 		else if (*I >= (unsigned)fh->numGlyph) {
 			auto tofu = BuildTofuString(*I);
-			int tofuWidth = 0;
-			for (auto ch : tofu) {
-				tofuWidth += measureCodepoint(tofuFont.fh, ch);
+			for (auto cp : tofu) {
+				x += measureCodepoint(tofuFont.fh, cp);
+				x = std::ceil(x);
+				if (curX <= x) {
+					return std::distance(str.begin(), I);
+				}
 			}
-			int halfWidth = (int)(tofuWidth / 2.0f);
-			x += halfWidth;
-			if (curX <= x) {
-				break;
-			}
-			x += tofuWidth - halfWidth;
 			++I;
 		}
 		else if (*I == U'\t') {
 			auto& glyph = fh->Glyph(' ');
-			int spWidth = glyph.width + glyph.spLeft + glyph.spRight;
-			int fullWidth = (int)(spWidth * 4 * scale);
-			int halfWidth = (int)(fullWidth / 2.0f);
+			float fullWidth = (glyph.width + glyph.spLeft + glyph.spRight) * 4.0f * scale;
+			float halfWidth = std::ceil(fullWidth / 2.0f);
 			x += halfWidth;
+			x = std::ceil(x);
 			if (curX <= x) {
 				break;
 			}
 			x += fullWidth - halfWidth;
+			x = std::ceil(x);
+			if (curX <= x) {
+				break;
+			}
 			++I;
 		}
 		else {
-			x += (int)(measureCodepoint(fh, *I) * scale);
+			x += measureCodepoint(fh, *I) * scale;
+			x = std::ceil(x);
 			if (curX <= x) {
 				break;
 			}
@@ -339,10 +345,10 @@ void r_font_c::DrawTextLine(scp_t pos, int align, int height, col4_t col, std::u
 
 	// Calculate the string position
 	float x = pos[X];
-	float y = pos[Y];
+	float y = std::floor(pos[Y]);
 	if (align != F_LEFT) {
 		// Calculate the real width of the string
-		float width = (float)StringWidthInternal(fh, str, height, scale);
+		float width = StringWidthInternal(fh, str, height, scale);
 		switch (align) {
 		case F_CENTRE:
 			x = floor((renderer->VirtualScreenWidth() - width) / 2.0f + pos[X]);
@@ -359,6 +365,9 @@ void r_font_c::DrawTextLine(scp_t pos, int align, int height, col4_t col, std::u
 		}
 	}
 
+	// Snap the starting x position to the pixel grid so the leading glyph isn't blurred.
+	x = std::round(x);
+
 	r_tex_c* curTex{};
 
 	auto drawCodepoint = [this, &curTex, &x, y](f_fontHeight_s* fh, int height, float scale, int yShift, char32_t cp) {
diff --git a/engine/render/r_main.cpp b/engine/render/r_main.cpp
@@ -1855,32 +1855,45 @@ int r_renderer_c::DrawStringCursorIndex(int height, int font, const char* str, i
 // ==============
 
 int r_renderer_c::VirtualScreenWidth() {
-	return VirtualMap(sys->video->vid.size[0]);
+	int const properWidth = apiDpiAware ? sys->video->vid.fbSize[0] : sys->video->vid.size[0];
+	return VirtualMap(properWidth);
 }
 
 int r_renderer_c::VirtualScreenHeight() {
-	return VirtualMap(sys->video->vid.size[1]);
+	int const properHeight = apiDpiAware ? sys->video->vid.fbSize[1] : sys->video->vid.size[1];
+	return VirtualMap(properHeight);
 }
 
 float r_renderer_c::VirtualScreenScaleFactor() {
 	if (apiDpiAware) {
+		if (dpiScaleOverridePercent > 0) {
+			return dpiScaleOverridePercent / 100.0f;
+		}
 		return sys->video->vid.dpiScale;
 	}
 	return 1.0f;
 }
 
+void r_renderer_c::SetDpiScaleOverridePercent(int percent) {
+	dpiScaleOverridePercent = percent;
+}
+
+int r_renderer_c::DpiScaleOverridePercent() const {
+	return dpiScaleOverridePercent;
+}
+
 int r_renderer_c::VirtualMap(int properValue) {
 	if (apiDpiAware) {
 		return properValue;
 	}
-	return (int)(properValue / sys->video->vid.dpiScale);
+	return static_cast<int>(properValue / sys->video->vid.dpiScale);
 }
 
 int r_renderer_c::VirtualUnmap(int mappedValue) {
 	if (apiDpiAware) {
 		return mappedValue;
 	}
-	return (int)(mappedValue * sys->video->vid.dpiScale);
+	return static_cast<int>(mappedValue * sys->video->vid.dpiScale);
 }
 
 // =====
diff --git a/engine/render/r_main.h b/engine/render/r_main.h
@@ -99,6 +99,8 @@ class r_renderer_c: public r_IRenderer, public conCmdHandler_c {
 	int		VirtualScreenWidth();
 	int		VirtualScreenHeight();
 	float	VirtualScreenScaleFactor();
+	void	SetDpiScaleOverridePercent(int percent);
+	int		DpiScaleOverridePercent() const;
 	int		VirtualMap(int properValue);
 	int		VirtualUnmap(int mappedValue);
 
@@ -126,7 +128,7 @@ class r_renderer_c: public r_IRenderer, public conCmdHandler_c {
 	PFNGLINSERTEVENTMARKEREXTPROC	glInsertEventMarkerEXT = nullptr;
 	PFNGLPUSHGROUPMARKEREXTPROC		glPushGroupMarkerEXT = nullptr;
 	PFNGLPOPGROUPMARKEREXTPROC		glPopGroupMarkerEXT = nullptr;
-	
+
 	conVar_c*	r_compress = nullptr;
 	conVar_c*	r_screenshotFormat = nullptr;
 	conVar_c*	r_layerDebug = nullptr;
@@ -173,6 +175,7 @@ class r_renderer_c: public r_IRenderer, public conCmdHandler_c {
 	};
 
 	bool apiDpiAware{};
+	int dpiScaleOverridePercent = 0;
 	RenderTarget rttMain[2];
 	int	presentRtt = 0;
 
diff --git a/ui_api.cpp b/ui_api.cpp
diff --git a/vcpkg.json b/vcpkg.json

-Original file line number
+Diff line change
@@ @@ -1 +1,3 @@ @@
 -/build/
 +/build/
 +/build-SimpleGraphic
 +/.idea
Original file line number	Diff line number	Diff line change
`@@ -1855,32 +1855,45 @@ int r_renderer_c::DrawStringCursorIndex(int height, int font, const char* str, i`
`1855`	`1855`	`// ==============`
`1856`	`1856`
`1857`	`1857`	`int r_renderer_c::VirtualScreenWidth() {`
`1858`		`- return VirtualMap(sys->video->vid.size[0]);`
	`1858`	`+ int const properWidth = apiDpiAware ? sys->video->vid.fbSize[0] : sys->video->vid.size[0];`
	`1859`	`+ return VirtualMap(properWidth);`
`1859`	`1860`	`}`
`1860`	`1861`
`1861`	`1862`	`int r_renderer_c::VirtualScreenHeight() {`
`1862`		`- return VirtualMap(sys->video->vid.size[1]);`
	`1863`	`+ int const properHeight = apiDpiAware ? sys->video->vid.fbSize[1] : sys->video->vid.size[1];`
	`1864`	`+ return VirtualMap(properHeight);`
`1863`	`1865`	`}`
`1864`	`1866`
`1865`	`1867`	`float r_renderer_c::VirtualScreenScaleFactor() {`
`1866`	`1868`	`if (apiDpiAware) {`
	`1869`	`+ if (dpiScaleOverridePercent > 0) {`
	`1870`	`+ return dpiScaleOverridePercent / 100.0f;`
	`1871`	`+ }`
`1867`	`1872`	`return sys->video->vid.dpiScale;`
`1868`	`1873`	`}`
`1869`	`1874`	`return 1.0f;`
`1870`	`1875`	`}`
`1871`	`1876`
	`1877`	`+void r_renderer_c::SetDpiScaleOverridePercent(int percent) {`
	`1878`	`+ dpiScaleOverridePercent = percent;`
	`1879`	`+}`
	`1880`	`+`
	`1881`	`+int r_renderer_c::DpiScaleOverridePercent() const {`
	`1882`	`+ return dpiScaleOverridePercent;`
	`1883`	`+}`
	`1884`	`+`
`1872`	`1885`	`int r_renderer_c::VirtualMap(int properValue) {`
`1873`	`1886`	`if (apiDpiAware) {`
`1874`	`1887`	`return properValue;`
`1875`	`1888`	`}`
`1876`		`- return (int)(properValue / sys->video->vid.dpiScale);`
	`1889`	`+ return static_cast<int>(properValue / sys->video->vid.dpiScale);`
`1877`	`1890`	`}`
`1878`	`1891`
`1879`	`1892`	`int r_renderer_c::VirtualUnmap(int mappedValue) {`
`1880`	`1893`	`if (apiDpiAware) {`
`1881`	`1894`	`return mappedValue;`
`1882`	`1895`	`}`
`1883`		`- return (int)(mappedValue * sys->video->vid.dpiScale);`
	`1896`	`+ return static_cast<int>(mappedValue * sys->video->vid.dpiScale);`
`1884`	`1897`	`}`
`1885`	`1898`
`1886`	`1899`	`// =====`