Skip to content

revert format

revert format #58

Triggered via push November 21, 2023 09:47
Status Success
Total duration 21s
Artifacts
Validation
12s
Validation
Fit to window
Zoom out
Zoom in

Annotations

237 warnings
[cpplint] onnxruntime/core/mlas/inc/mlas_q4.h#L360: onnxruntime/core/mlas/inc/mlas_q4.h#L360
Closing ) should be moved to the previous line [whitespace/parens] [2]
[cpplint] onnxruntime/core/mlas/lib/jblas_defs.h#L47: onnxruntime/core/mlas/lib/jblas_defs.h#L47
"virtual" is redundant since function is already declared as "override" [readability/inheritance] [4]
[cpplint] onnxruntime/core/mlas/lib/q4_dq.cpp#L1062: onnxruntime/core/mlas/lib/q4_dq.cpp#L1062
Closing ) should be moved to the previous line [whitespace/parens] [2]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/jit_blas_parallel.h#L407: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/jit_blas_parallel.h#L407
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/jit_blas_parallel.h#L585: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/jit_blas_parallel.h#L585
"virtual" is redundant since function is already declared as "override" [readability/inheritance] [4]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/jit_blas_parallel.h#L609: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/jit_blas_parallel.h#L609
"virtual" is redundant since function is already declared as "override" [readability/inheritance] [4]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/jit_blas_wrapper.h#L65: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/jit_blas_wrapper.h#L65
Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_mnemonic.h#L2302: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_mnemonic.h#L2302
If an else has a brace on one side, it should have it on both [readability/braces] [5]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L22: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L22
Extra space before last semicolon. If this should be an empty statement, use {} instead. [whitespace/semicolon] [5]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L149: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L149
Constructors callable with one argument should be marked explicit. [runtime/explicit] [5]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L151: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L151
Single-parameter constructors should be marked explicit. [runtime/explicit] [5]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L185: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L185
Do not use variable-length arrays. Use an appropriately named ('k' followed by CamelCase) compile-time constant for the size. [runtime/arrays] [1]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L188: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L188
Do not use variable-length arrays. Use an appropriately named ('k' followed by CamelCase) compile-time constant for the size. [runtime/arrays] [1]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L189: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L189
Do not use variable-length arrays. Use an appropriately named ('k' followed by CamelCase) compile-time constant for the size. [runtime/arrays] [1]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L343: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L343
If/else bodies with multiple statements require braces [readability/braces] [4]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L584: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L584
If/else bodies with multiple statements require braces [readability/braces] [4]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L684: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L684
Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L712: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L712
Do not use variable-length arrays. Use an appropriately named ('k' followed by CamelCase) compile-time constant for the size. [runtime/arrays] [1]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L726: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L726
Single-parameter constructors should be marked explicit. [runtime/explicit] [5]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L862: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L862
Using C-style cast. Use static_cast<int>(...) instead [readability/casting] [4]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L872: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L872
Using C-style cast. Use static_cast<int>(...) instead [readability/casting] [4]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L884: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L884
Using C-style cast. Use static_cast<int>(...) instead [readability/casting] [4]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L922: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L922
Do not use variable-length arrays. Use an appropriately named ('k' followed by CamelCase) compile-time constant for the size. [runtime/arrays] [1]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L958: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L958
Do not use namespace using-directives. Use using-declarations instead. [build/namespaces] [5]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L989: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L989
Do not use namespace using-directives. Use using-declarations instead. [build/namespaces] [5]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L1006: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L1006
Do not use namespace using-directives. Use using-declarations instead. [build/namespaces] [5]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L1018: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L1018
Do not use namespace using-directives. Use using-declarations instead. [build/namespaces] [5]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L1143: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L1143
Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
[cpplint] onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L1151: onnxruntime/core/mlas/lib/x86_64/jblas/jblas/xbyak/xbyak_util.h#L1151
Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]