Skip to content

Add override specifier #20927

Add override specifier

Add override specifier #20927

Triggered via push October 16, 2023 23:44
Status Success
Total duration 28s
Artifacts
Validation
16s
Validation
Fit to window
Zoom out
Zoom in

Annotations

43 warnings
[cpplint] onnxruntime/core/optimizer/initializer.cc#L294: onnxruntime/core/optimizer/initializer.cc#L294
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/initializer.cc#L307: onnxruntime/core/optimizer/initializer.cc#L307
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/initializer.cc#L313: onnxruntime/core/optimizer/initializer.cc#L313
An else should appear on the same line as the preceding } [whitespace/newline] [4]
[cpplint] onnxruntime/core/optimizer/initializer.cc#L314: onnxruntime/core/optimizer/initializer.cc#L314
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/initializer.cc#L330: onnxruntime/core/optimizer/initializer.cc#L330
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/optimizer/initializer.cc#L331: onnxruntime/core/optimizer/initializer.cc#L331
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L8: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L8
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L102: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L102
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L123: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L123
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L127: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L127
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L165: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L165
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L184: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L184
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L186: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L186
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L188: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L188
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L190: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L190
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L192: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L192
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L210: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L210
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L223: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L223
At least two spaces is best between code and comments [whitespace/comments] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L228: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L228
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L253: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L253
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L280: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L280
Namespace should be terminated with "// namespace onnxruntime" [readability/namespace] [5]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L109: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L109
Add #include <utility> for pair<> [build/include_what_you_use] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L239: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L239
Add #include <string> for string [build/include_what_you_use] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L254: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L254
Add #include <vector> for vector<> [build/include_what_you_use] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L260: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L260
Add #include <limits> for numeric_limits<> [build/include_what_you_use] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L280: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L280
Could not find a newline character at the end of the file. [whitespace/ending_newline] [5]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L7: onnxruntime/core/optimizer/matmul_bn_fusion.h#L7
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L24: onnxruntime/core/optimizer/matmul_bn_fusion.h#L24
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L26: onnxruntime/core/optimizer/matmul_bn_fusion.h#L26
Namespace should be terminated with "// namespace onnxruntime" [readability/namespace] [5]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L17: onnxruntime/core/optimizer/matmul_bn_fusion.h#L17
Add #include <string> for string [build/include_what_you_use] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L17: onnxruntime/core/optimizer/matmul_bn_fusion.h#L17
Add #include <vector> for vector<> [build/include_what_you_use] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L26: onnxruntime/core/optimizer/matmul_bn_fusion.h#L26
Could not find a newline character at the end of the file. [whitespace/ending_newline] [5]
[cpplint] onnxruntime/test/optimizer/graph_transform_test.cc#L1107: onnxruntime/test/optimizer/graph_transform_test.cc#L1107
Consider using ASSERT_EQ instead of ASSERT_TRUE(a == b) [readability/check] [2]
[cpplint] onnxruntime/test/optimizer/graph_transform_test.cc#L1108: onnxruntime/test/optimizer/graph_transform_test.cc#L1108
Consider using ASSERT_EQ instead of ASSERT_TRUE(a == b) [readability/check] [2]
[cpplint] onnxruntime/test/optimizer/graph_transform_test.cc#L1109: onnxruntime/test/optimizer/graph_transform_test.cc#L1109
Consider using ASSERT_EQ instead of ASSERT_TRUE(a == b) [readability/check] [2]
[cpplint] onnxruntime/test/optimizer/graph_transform_test.cc#L1133: onnxruntime/test/optimizer/graph_transform_test.cc#L1133
An else should appear on the same line as the preceding } [whitespace/newline] [4]
[cpplint] onnxruntime/test/optimizer/graph_transform_test.cc#L1133: onnxruntime/test/optimizer/graph_transform_test.cc#L1133
If an else has a brace on one side, it should have it on both [readability/braces] [5]
[cpplint] onnxruntime/test/optimizer/graph_transform_test.cc#L1146: onnxruntime/test/optimizer/graph_transform_test.cc#L1146
Consider using ASSERT_EQ instead of ASSERT_TRUE(a == b) [readability/check] [2]
[cpplint] onnxruntime/test/optimizer/graph_transform_test.cc#L1147: onnxruntime/test/optimizer/graph_transform_test.cc#L1147
Consider using ASSERT_EQ instead of ASSERT_TRUE(a == b) [readability/check] [2]
[cpplint] onnxruntime/test/optimizer/graph_transform_test.cc#L1148: onnxruntime/test/optimizer/graph_transform_test.cc#L1148
Consider using ASSERT_EQ instead of ASSERT_TRUE(a == b) [readability/check] [2]
[cpplint] onnxruntime/test/optimizer/graph_transform_test.cc#L1189: onnxruntime/test/optimizer/graph_transform_test.cc#L1189
Consider using ASSERT_EQ instead of ASSERT_TRUE(a == b) [readability/check] [2]
[cpplint] onnxruntime/test/optimizer/graph_transform_test.cc#L1190: onnxruntime/test/optimizer/graph_transform_test.cc#L1190
Consider using ASSERT_EQ instead of ASSERT_TRUE(a == b) [readability/check] [2]
[cpplint] onnxruntime/test/optimizer/graph_transform_test.cc#L1191: onnxruntime/test/optimizer/graph_transform_test.cc#L1191
Consider using ASSERT_EQ instead of ASSERT_TRUE(a == b) [readability/check] [2]