Adding explicit keyword to Timeout constructor to prevent accidental …

…conversion from int types.
wjwwood · Jul 30, 2012 · 7e46129 · 7e46129
1 parent c829e6a
commit 7e46129
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/include/serial/serial.h b/include/serial/serial.h
@@ -126,9 +126,11 @@ struct Timeout {
    */
   uint32_t write_timeout_multiplier;
 
-  Timeout (uint32_t inter_byte_timeout_=0, uint32_t read_timeout_constant_=0,
-           uint32_t read_timeout_multiplier_=0, uint32_t write_timeout_constant_=0,
-           uint32_t write_timeout_multiplier_=0)
+  explicit Timeout (uint32_t inter_byte_timeout_=0,
+                    uint32_t read_timeout_constant_=0,
+                    uint32_t read_timeout_multiplier_=0,
+                    uint32_t write_timeout_constant_=0,
+                    uint32_t write_timeout_multiplier_=0)
   : inter_byte_timeout(inter_byte_timeout_),
     read_timeout_constant(read_timeout_constant_),
     read_timeout_multiplier(read_timeout_multiplier_),