diff --git a/onnx/common/graph_node_list.h b/onnx/common/graph_node_list.h
index bdd54391cce..ead527b9a45 100644
--- a/onnx/common/graph_node_list.h
+++ b/onnx/common/graph_node_list.h
@@ -54,6 +54,8 @@ struct generic_graph_node_list_iterator final {
     : cur(cur), d(d) {}
   generic_graph_node_list_iterator(const generic_graph_node_list_iterator & rhs)
     : cur(rhs.cur), d(rhs.d) {}
+  generic_graph_node_list_iterator & operator=(
+    const generic_graph_node_list_iterator&) = default;
   T * operator*() const { return cur; }
   T * operator->() const { return cur; }
   generic_graph_node_list_iterator & operator++() {
diff --git a/onnx/defs/parser.cc b/onnx/defs/parser.cc
index 0b1c9bb56d4..f38577c9755 100644
--- a/onnx/defs/parser.cc
+++ b/onnx/defs/parser.cc
@@ -116,7 +116,7 @@ Status OnnxParser::ParseInput(ValueInfoList& inputs, TensorList& initializers) {
           // default value for input
           TensorProto& tp = *initializers.Add();
           tp.set_name(vi.name());
-          CHECK_PARSER_STATUS (Parse(tp, vi.type()));
+          CHECK_PARSER_STATUS(Parse(tp, vi.type()));
         }
       } while (Matches(','));
       MATCH(')');
@@ -139,7 +139,7 @@ Status OnnxParser::ParseValueInfo(ValueInfoList& value_infos, TensorList& initia
           // initializer
           TensorProto& tp = *initializers.Add();
           tp.set_name(vi.name());
-          CHECK_PARSER_STATUS (Parse(tp, vi.type()));
+          CHECK_PARSER_STATUS(Parse(tp, vi.type()));
         } else {
           // valueinfo
           *value_infos.Add() = vi;
@@ -391,8 +391,8 @@ Status OnnxParser::Parse(ModelProto& model) {
               import->set_version(intval);
             } while (Matches(','));
             MATCH(']');
-            break;
           }
+          break;
         }
         case KeyWordMap::KeyWord::PRODUCER_NAME:
           PARSE_TOKEN(strval);