RepoMirrors
/
osu
mirror of https://github.com/ppy/osu
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Remove duplicate try-catch and move `NotifyShellUpdate()` to less hidden place

This commit is contained in:
Susko3 2024-02-07 22:03:16 +01:00
parent f9d257b99e
commit 0563507295
1 changed files with 8 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
osu.Desktop/Windows/WindowsAssociationManager.cs
View File

@ -68,6 +68,7 @@ public static void InstallAssociations(LocalisationManager? localisation)
}
updateDescriptions(localisation);
NotifyShellUpdate();
}
catch (Exception e)
{
@ -77,24 +78,15 @@ public static void InstallAssociations(LocalisationManager? localisation)
private static void updateDescriptions(LocalisationManager? localisation)
{
try
{
using var classes = Registry.CurrentUser.OpenSubKey(SOFTWARE_CLASSES, true);
if (classes == null)
return;
using var classes = Registry.CurrentUser.OpenSubKey(SOFTWARE_CLASSES, true);
if (classes == null)
return;
foreach (var association in file_associations)
association.UpdateDescription(classes, PROGRAM_ID_PREFIX, getLocalisedString(association.Description));
foreach (var association in file_associations)
association.UpdateDescription(classes, PROGRAM_ID_PREFIX, getLocalisedString(association.Description));
foreach (var association in uri_associations)
association.UpdateDescription(classes, getLocalisedString(association.Description));
NotifyShellUpdate();
}
catch (Exception e)
{
Logger.Log($@"Failed to update file and URI associations: {e.Message}");
}
foreach (var association in uri_associations)
association.UpdateDescription(classes, getLocalisedString(association.Description));
string getLocalisedString(LocalisableString s)
{