Learning/mRemoteNG
1
0
Fork 0
mirror of https://github.com/mRemoteNG/mRemoteNG.git synced 2026-02-17 22:11:48 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fix for #1257

Browse Source 
CreateGraphics was the function being disposed in the using, the form handle was still floating around. Suggestion to optimize the dpi function
This commit is contained in:
Camilo Alvarez
2019-01-21 23:29:26 -05:00
parent 968471ec4a
commit 1f700f7842
1 changed files with 8 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
mRemoteV1/UI/GraphicsUtilities/GdiPlusGraphicsProvider.cs
View File

@@ -11,12 +11,16 @@ namespace mRemoteNG.UI.GraphicsUtilities
// Dpi of a 'normal' definition screen
private const int BaselineDpi = 96;
public SizeF GetResolutionScalingFactor()
{
using (var g = new Form().CreateGraphics())
{
return new SizeF(g.DpiX / BaselineDpi, g.DpiY / BaselineDpi);
}
//This method could be optimized, as it is called for every control / subcontrol
//and causes overhead for 100s in the options page
using (var f = new Form())
{
var g = f.CreateGraphics();
return new SizeF(g.DpiX / BaselineDpi, g.DpiY / BaselineDpi);
}
}
}
}