Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: cert-err58 warning in GTK client #6861

Merged
merged 1 commit into from
Jun 1, 2024
Merged

Conversation

ckerr
Copy link
Member

@ckerr ckerr commented May 27, 2024

Fix these cert-err58-cpp warnings:

[95/150] Building CXX object gtk/CMakeFiles/transmission-gtk.dir/SystemTrayIcon.cc.o
/home/charles/src/transmission/gtk/SystemTrayIcon.cc:58:12: warning: initialization of 'TrayIconName' with static storage duration may throw an exception that cannot be caught [cert-err58-cpp]
auto const TrayIconName = Glib::ustring("transmission-tray-icon"s);
^
/usr/include/glibmm-2.4/glibmm/ustring.h:403:14: note: possibly throwing constructor declared here
GLIBMM_API ustring(std::string&& src);
^
/home/charles/src/transmission/gtk/SystemTrayIcon.cc:59:12: warning: initialization of 'AppIconName' with static storage duration may throw an exception that cannot be caught [cert-err58-cpp]
auto const AppIconName = Glib::ustring("transmission"s);
^
/usr/include/glibmm-2.4/glibmm/ustring.h:403:14: note: possibly throwing constructor declared here
GLIBMM_API ustring(std::string&& src);
^
/home/charles/src/transmission/gtk/SystemTrayIcon.cc:63:12: warning: initialization of 'AppName' with static storage duration may throw an exception that cannot be caught [cert-err58-cpp]
auto const AppName = Glib::ustring("transmission-gtk"s);
^
/usr/include/glibmm-2.4/glibmm/ustring.h:403:14: note: possibly throwing constructor declared here
GLIBMM_API ustring(std::string&& src);
^

@ckerr ckerr added scope:gtk type:fix A bug fix notes:none Should not be listed in release notes labels May 27, 2024
@mikedld mikedld merged commit 2b75869 into main Jun 1, 2024
24 checks passed
@mikedld mikedld deleted the fix/cert-err58-warning branch June 1, 2024 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notes:none Should not be listed in release notes scope:gtk type:fix A bug fix
Development

Successfully merging this pull request may close these issues.

None yet

2 participants