- use const ref as return type

diff --git a/client/snapper/GlobalOptions.h b/client/snapper/GlobalOptions.h
index 4b4c3e339e16a029a354f0210c2de4a4c23716d1..a1e1604626657d5b049b5f2bb8e918fb380b6711 100644 (file)
--- a/client/snapper/GlobalOptions.h
+++ b/client/snapper/GlobalOptions.h
@@ -56,10 +56,10 @@ namespace snapper
        Style table_style() const { return _table_style; }
        bool abbreviate() const { return _abbreviate; }
        OutputFormat output_format() const { return _output_format; }
-       string separator() const { return _separator; }
+       const string& separator() const { return _separator; }
        bool headers() const { return _headers; }
-       string config() const { return _config; }
-       string root() const { return _root; }
+       const string& config() const { return _config; }
+       const string& root() const { return _root; }
        Ambit ambit() const { return _ambit; }
 
        void set_ambit(Ambit ambit) { _ambit = ambit; }

diff --git a/client/snbk/GlobalOptions.h b/client/snbk/GlobalOptions.h
index b57a6235d0a7a0540c76b1078bf188f1f1072e23..6147c26818bf5433dd7db0ea54f8a61f08b1f5ca 100644 (file)
--- a/client/snbk/GlobalOptions.h
+++ b/client/snbk/GlobalOptions.h
@@ -50,7 +50,7 @@ namespace snapper
        bool quiet() const { return _quiet; }
        bool verbose() const { return _verbose; }
        bool debug() const { return _debug; }
-       boost::optional<BackupConfig::TargetMode> target_mode() { return _target_mode; }
+       const boost::optional<BackupConfig::TargetMode>& target_mode() { return _target_mode; }
        bool automatic() const { return _automatic; }
        bool utc() const { return _utc; }
        bool iso() const { return _iso; }
@@ -59,9 +59,9 @@ namespace snapper
        bool help() const { return _help; }
        Style table_style() const { return _table_style; }
        OutputFormat output_format() const { return _output_format; }
-       string separator() const { return _separator; }
+       const string& separator() const { return _separator; }
        bool headers() const { return _headers; }
-       boost::optional<string> backup_config() const { return _backup_config; }
+       const boost::optional<string>& backup_config() const { return _backup_config; }
 
     private: