From 3ec9deebd8e3496feacab83e41846dc4fffc3404 Mon Sep 17 00:00:00 2001 From: Phil Miller Date: Mon, 25 Jun 2018 10:58:59 -0500 Subject: [PATCH] Return 1 to indicate error from main() The statement 'return false;' was most likely to end up indicating successful exit by returning a 0 value, when failure seems to be intended. This was flagged as a warning in the Debian Clang Archive Rebuild. --- dump_xsettings.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dump_xsettings.cc b/dump_xsettings.cc index f94a0c0..e9330b4 100644 --- a/dump_xsettings.cc +++ b/dump_xsettings.cc @@ -270,7 +270,7 @@ int main(int argc, char** argv) { xsettingsd::DataReader reader(buffer, data_size); if (!xsettingsd::DumpSettings(&reader)) - return false; + return 1; return 0; }