Skip to content
Back to pull request #17915

Remove redundant code #20588

Sign in to view logs

Remove redundant code

Remove redundant code #20588

Triggered via push October 12, 2023 18:43
@sumitsayssumitsays
pushed c797f40
user/sumita/matmulbn
Status Success
Total duration 32s
Artifacts –

gradle-wrapper-validation.yml

on: push
Validation
22s
Validation
Fit to window
Zoom out
Zoom in

Annotations

49 warnings
[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#L13: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L13
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L23: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L23
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L26: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L26
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L32: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L32
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L37: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L37
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L47: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L47
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L49: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L49
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L55: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L55
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L62: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L62
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L106: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L106
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L109: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L109
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L118: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L118
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L132: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L132
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L140: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L140
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L144: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L144
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L151: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L151
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L176: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L176
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L191: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L191
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L196: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L196
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L198: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L198
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L200: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L200
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L202: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L202
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L204: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L204
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L220: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L220
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L223: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L223
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L236: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L236
At least two spaces is best between code and comments [whitespace/comments] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L241: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L241
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L266: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L266
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L275: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L275
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L294: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L294
Namespace should be terminated with "// namespace onnxruntime" [readability/namespace] [5]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L125: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L125
Add #include <utility> for pair<> [build/include_what_you_use] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L252: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L252
Add #include <string> for string [build/include_what_you_use] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L267: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L267
Add #include <vector> for vector<> [build/include_what_you_use] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L273: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L273
Add #include <limits> for numeric_limits<> [build/include_what_you_use] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.cc#L294: onnxruntime/core/optimizer/matmul_bn_fusion.cc#L294
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#L14: onnxruntime/core/optimizer/matmul_bn_fusion.h#L14
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L15: onnxruntime/core/optimizer/matmul_bn_fusion.h#L15
public: should be indented +1 space inside class MatmulBNFusion [whitespace/indent] [3]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L17: onnxruntime/core/optimizer/matmul_bn_fusion.h#L17
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L18: onnxruntime/core/optimizer/matmul_bn_fusion.h#L18
Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L18: onnxruntime/core/optimizer/matmul_bn_fusion.h#L18
Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L22: onnxruntime/core/optimizer/matmul_bn_fusion.h#L22
{ should almost always be at the end of the previous line [whitespace/braces] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L26: onnxruntime/core/optimizer/matmul_bn_fusion.h#L26
private: should be indented +1 space inside class MatmulBNFusion [whitespace/indent] [3]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L43: onnxruntime/core/optimizer/matmul_bn_fusion.h#L43
Namespace should be terminated with "// namespace onnxruntime" [readability/namespace] [5]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L21: onnxruntime/core/optimizer/matmul_bn_fusion.h#L21
Add #include <vector> for vector<> [build/include_what_you_use] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L40: onnxruntime/core/optimizer/matmul_bn_fusion.h#L40
Add #include <utility> for pair<> [build/include_what_you_use] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L40: onnxruntime/core/optimizer/matmul_bn_fusion.h#L40
Add #include <string> for string [build/include_what_you_use] [4]
[cpplint] onnxruntime/core/optimizer/matmul_bn_fusion.h#L43: onnxruntime/core/optimizer/matmul_bn_fusion.h#L43
Could not find a newline character at the end of the file. [whitespace/ending_newline] [5]