anotherim/anotherim-desktop
2
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

stop exceptions from crossing ABI boundary in a few places

Browse source
This commit is contained in:
mjk 2021-03-05 21:29:43 +00:00 committed by LAGonauta
parent f1b5633ce6
commit 1bd1376cea
2 changed files with 22 additions and 21 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
plugins/windows-notification/api/src/shortcutcreator.cpp
View file

@ -4,6 +4,7 @@
#include "shortcutcreator.h"
#include "win32.hpp"
#include "converter.hpp"
#include "ginvoke.hpp"
#ifdef UNICODE
#define _UNICODE
@ -124,8 +125,14 @@ int32_t ValidateShortcut(const std::wstring& shortcut_path, const std::wstring&
return hr;
}
int32_t TryCreateShortcutInternal(const std::wstring& aumid)
bool TryCreateShortcutInternal(const char *const aumid)
{
auto waumid = sview_to_wstr(aumid);
if (waumid.empty())
{
return false;
}
auto exePath = GetCurrentModulePath();
auto shortcutPath = GetShortcutPath();
@ -134,27 +141,20 @@ int32_t TryCreateShortcutInternal(const std::wstring& aumid)
auto path = shortcutPath.value() + LR"(\Microsoft\Windows\Start Menu\Programs\Dino.lnk)";
if (!std::filesystem::exists(path))
{
return InstallShortcut(exePath.value(), aumid, path);
return SUCCEEDED(InstallShortcut(exePath.value(), waumid, path));
}
else
{
return ValidateShortcut(path, aumid);
return SUCCEEDED(ValidateShortcut(path, waumid));
}
return S_OK;
}
return S_FALSE;
return false;
}
extern "C"
{
gboolean TryCreateShortcut(const gchar* aumid)
{
auto result = sview_to_wstr(aumid);
if (result.empty())
{
return FALSE;
}
return SUCCEEDED(TryCreateShortcutInternal(result));
return g_try_invoke(TryCreateShortcutInternal, aumid);
}
}

17
plugins/windows-notification/api/src/win32.cpp
View file

@ -3,6 +3,7 @@
#include "win32.hpp"
#include "converter.hpp"
#include "ginvoke.hpp"
std::optional<std::wstring> GetCurrentModulePath()
{
@ -28,9 +29,14 @@ std::optional<std::wstring> GetShortcutPath()
return std::nullopt;
}
bool SetAppModelIDInternal(const std::wstring& aumid)
bool SetAppModelIDInternal(const char *const aumid)
{
return SUCCEEDED(SetCurrentProcessExplicitAppUserModelID(aumid.c_str()));
auto waumid = sview_to_wstr(aumid);
if (waumid.empty())
{
return false;
}
return SUCCEEDED(SetCurrentProcessExplicitAppUserModelID(waumid.c_str()));
}
extern "C"
@ -51,12 +57,7 @@ extern "C"
gboolean SetAppModelID(const gchar* aumid)
{
auto result = sview_to_wstr(aumid);
if (result.empty())
{
return FALSE;
}
return SetAppModelIDInternal(result);
return g_try_invoke(SetAppModelIDInternal, aumid);
}
}