diff options
author | ConfuSomu | 2024-02-06 17:41:29 -0500 |
---|---|---|
committer | ConfuSomu | 2024-02-06 17:45:07 -0500 |
commit | 0e32ef940e53c4f10d84762f2de6836025f87610 (patch) | |
tree | 6843f5d3b592db9e17deb105d958e61fa2237a26 | |
parent | 948bfe5e1032f7e19046638c3b47001f45d5a3f1 (diff) | |
download | ActorViewer-0e32ef940e53c4f10d84762f2de6836025f87610.tar ActorViewer-0e32ef940e53c4f10d84762f2de6836025f87610.tar.gz ActorViewer-0e32ef940e53c4f10d84762f2de6836025f87610.zip |
Fix harmless missing return for non-void function
choose_icon's switch statement was missing a return statement for the
default case, which would be unreachable anyways (unless in the case of
a bug) as all enum values are already covered.
-rw-r--r-- | src/list_item.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/list_item.cpp b/src/list_item.cpp index 18379ea..d93b81f 100644 --- a/src/list_item.cpp +++ b/src/list_item.cpp @@ -18,6 +18,7 @@ QIcon& choose_icon(StatusType status_type) { case StatusType::REBLOG: static QIcon reblog_icon(ICON_PATH_PREFIX"/repeat.png"); return reblog_icon; case StatusType::UNKNOWN: + default: static QIcon unknown_icon(ICON_PATH_PREFIX"/question.png"); return unknown_icon; } } |