[locomotiv] Use break rather than return (#3437)

For Push node execution, let's use break rather than return inside
switch statement, as it feels more natural.

Signed-off-by: Cheongyo Bahk <ch.bahk@samsung.com>

diff --git a/contrib/locomotiv/src/Node/Push.cpp b/contrib/locomotiv/src/Node/Push.cpp
index 1d545f7..901124e 100644 (file)
--- a/contrib/locomotiv/src/Node/Push.cpp
+++ b/contrib/locomotiv/src/Node/Push.cpp
@@ -40,7 +40,7 @@ void NodeExecution::execute(loco::Push *push)
     auto push_data = make_data(*from_bufptr);
     erase_annot_data(push);
     annot_data(push, std::move(push_data));
-    return;
+    break;
   }
   case loco::DataType::FLOAT32:
   {
@@ -48,7 +48,7 @@ void NodeExecution::execute(loco::Push *push)
     auto push_data = make_data(*from_bufptr);
     erase_annot_data(push);
     annot_data(push, std::move(push_data));
-    return;
+    break;
   }
   default:
     throw std::runtime_error("NYI for this DataType");