From 19aa4de116a87621ec12f30697dcbab679db395f Mon Sep 17 00:00:00 2001
From: Alexey Rochev <equeim@gmail.com>
Date: Mon, 11 Dec 2023 00:15:14 +0300
Subject: [PATCH] Oops

---
 cmake/CommonOptions.cmake | 3 ++-
 src/fileutils.cpp         | 1 +
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/cmake/CommonOptions.cmake b/cmake/CommonOptions.cmake
index 8a3d0428..9170b661 100644
--- a/cmake/CommonOptions.cmake
+++ b/cmake/CommonOptions.cmake
@@ -38,7 +38,6 @@ function(set_common_options_on_targets)
         -Wall
         -Wextra
         -Wpedantic
-        -Werror=non-virtual-dtor
         -Wcast-align
         -Woverloaded-virtual
         -Wconversion
@@ -46,6 +45,8 @@ function(set_common_options_on_targets)
         -Wdouble-promotion
         -Wformat=2
         -Werror=format
+        -Werror=non-virtual-dtor
+        -Werror=return-type
     )
     if (CMAKE_CXX_COMPILER_ID MATCHES "Clang")
         list(APPEND gcc_style_warnings -Wlogical-op-parentheses -Wno-gnu-zero-variadic-macro-arguments)
diff --git a/src/fileutils.cpp b/src/fileutils.cpp
index d93d445f..6123148e 100644
--- a/src/fileutils.cpp
+++ b/src/fileutils.cpp
@@ -287,6 +287,7 @@ namespace tremotesf {
                 return true;
             }
             logInfo("isSessionIdFileExists: did not find transmission-daemon session id file");
+            return false;
         }
     }
 }