Fix missing break statement in SetAnyValueS case.
authorJon Simantov <jsimantov@google.com>
Wed, 5 Aug 2015 22:45:02 +0000 (15:45 -0700)
committerWouter van Oortmerssen <wvo@google.com>
Wed, 5 Aug 2015 23:06:47 +0000 (16:06 -0700)
Causing SetAnyValueS to treat all scalars as integers.

Change-Id: Ib467b255e7f32a1478180a91e65def31676399eb

src/reflection.cpp

index b0a80e4..35ae16f 100644 (file)
@@ -141,7 +141,9 @@ void SetAnyValueF(reflection::BaseType type, uint8_t *data, double val) {
 void SetAnyValueS(reflection::BaseType type, uint8_t *data, const char *val) {
   switch (type) {
     case reflection::Float:
-    case reflection::Double: SetAnyValueF(type, data, strtod(val, nullptr));
+    case reflection::Double:
+      SetAnyValueF(type, data, strtod(val, nullptr));
+      break;
     // TODO: support strings.
     default: SetAnyValueI(type, data, StringToInt(val)); break;
   }