Make operator<< to return void

This should help preventing misuse.

No functional change.

diff --git a/src/ucioption.cpp b/src/ucioption.cpp
index dbab6fc106c833652069b5185b73339f2862db63..a229027970b73bfdbf2e844331b4ac16843b9596 100644 (file)
--- a/src/ucioption.cpp
+++ b/src/ucioption.cpp
@@ -139,13 +139,12 @@ Option::operator std::string() const {
 
 /// operator<<() inits options and assigns idx in the correct printing order
 
-Option& Option::operator<<(const Option& o) {
+void Option::operator<<(const Option& o) {
 
   static size_t index = 0;
 
   *this = o;
   idx = index++;
-  return *this;
 }
 
 

diff --git a/src/ucioption.h b/src/ucioption.h
index 75e4ac1301e6d85e2b7daf8bb451bfeeb670a20e..19ca14fdd5746d74e85cecca358b7f6f47537761 100644 (file)
--- a/src/ucioption.h
+++ b/src/ucioption.h
@@ -47,7 +47,7 @@ public:
   Option(int v, int min, int max, OnChange = NULL);
 
   Option& operator=(const std::string& v);
-  Option& operator<<(const Option& o);
+  void operator<<(const Option& o);
   operator int() const;
   operator std::string() const;