Anthony Barbier | 6ff3b19 | 2017-09-04 18:44:23 +0100 | [diff] [blame] | 1 | #!/bin/bash |
| 2 | |
| 3 | set -e |
| 4 | |
Moritz Pflanzer | a09de0c | 2017-09-01 20:41:12 +0100 | [diff] [blame] | 5 | DIRECTORIES="./arm_compute ./src ./examples ./tests ./utils ./support" |
Anthony Barbier | 6ff3b19 | 2017-09-04 18:44:23 +0100 | [diff] [blame] | 6 | |
Pablo Tello | eb82fd2 | 2018-02-23 13:43:50 +0000 | [diff] [blame] | 7 | grep -HrnP --exclude-dir=assembly --exclude-dir=convolution --exclude-dir=arm_gemm "/\*\*$" $DIRECTORIES | tee bad_style.log |
Anthony Barbier | 6ff3b19 | 2017-09-04 18:44:23 +0100 | [diff] [blame] | 8 | if (( `cat bad_style.log | wc -l` > 0 )) |
| 9 | then |
| 10 | echo "" |
| 11 | echo "ERROR: Doxygen comments should start on the first line: \"/** My comment\"" |
| 12 | exit -1 |
| 13 | fi |
| 14 | |
Pablo Tello | eb82fd2 | 2018-02-23 13:43:50 +0000 | [diff] [blame] | 15 | grep -Hnr --exclude-dir=assembly --exclude-dir=convolution --exclude-dir=arm_gemm --exclude=Doxyfile "@brief" $DIRECTORIES | tee bad_style.log |
Anthony Barbier | 6ff3b19 | 2017-09-04 18:44:23 +0100 | [diff] [blame] | 16 | if (( `cat bad_style.log | wc -l` > 0 )) |
| 17 | then |
| 18 | echo "" |
| 19 | echo "ERROR: Doxygen comments shouldn't use '@brief'" |
| 20 | exit -1 |
| 21 | fi |
| 22 | |
Pablo Tello | eb82fd2 | 2018-02-23 13:43:50 +0000 | [diff] [blame] | 23 | grep -HnRE --exclude-dir=assembly --exclude-dir=convolution --exclude-dir=arm_gemm "\buint " --exclude-dir=cl_kernels --exclude-dir=cs_shaders $DIRECTORIES | tee bad_style.log |
Anthony Barbier | 6ff3b19 | 2017-09-04 18:44:23 +0100 | [diff] [blame] | 24 | if [[ $(cat bad_style.log | wc -l) > 0 ]] |
| 25 | then |
| 26 | echo "" |
| 27 | echo "ERROR: C/C++ don't define 'uint'. Use 'unsigned int' instead." |
| 28 | exit -1 |
| 29 | fi |
| 30 | |
Pablo Tello | eb82fd2 | 2018-02-23 13:43:50 +0000 | [diff] [blame] | 31 | grep -HnR --exclude-dir=assembly --exclude-dir=convolution --exclude-dir=arm_gemm "float32_t" $DIRECTORIES | tee bad_style.log |
Anthony Barbier | 6ff3b19 | 2017-09-04 18:44:23 +0100 | [diff] [blame] | 32 | if [[ $(cat bad_style.log | wc -l) > 0 ]] |
| 33 | then |
| 34 | echo "" |
| 35 | echo "ERROR: C/C++ don't define 'float32_t'. Use 'float' instead." |
| 36 | exit -1 |
| 37 | fi |
| 38 | |
Pablo Tello | eb82fd2 | 2018-02-23 13:43:50 +0000 | [diff] [blame] | 39 | grep -Hnir --exclude-dir=assembly --exclude-dir=convolution --exclude-dir=arm_gemm "arm[_ ]\?cv" $DIRECTORIES | tee bad_style.log |
Anthony Barbier | 6ff3b19 | 2017-09-04 18:44:23 +0100 | [diff] [blame] | 40 | if [[ $(cat bad_style.log | wc -l) > 0 ]] |
| 41 | then |
| 42 | echo "" |
| 43 | echo "ERROR: Reference to arm_cv detected in the files above (Replace with arm_compute)" |
| 44 | exit -1 |
| 45 | fi |
| 46 | |
Pablo Tello | eb82fd2 | 2018-02-23 13:43:50 +0000 | [diff] [blame] | 47 | grep -Hnir --exclude-dir=assembly --exclude-dir=convolution --exclude-dir=arm_gemm "#.*if.*defined[^(]" $DIRECTORIES | tee bad_style.log |
Anthony Barbier | ac69aa1 | 2017-07-03 17:39:37 +0100 | [diff] [blame] | 48 | if [[ $(cat bad_style.log | wc -l) > 0 ]] |
| 49 | then |
| 50 | echo "" |
| 51 | echo "ERROR: use parenthesis after #if defined(MY_PREPROCESSOR)" |
| 52 | exit -1 |
| 53 | fi |
| 54 | |
Pablo Tello | eb82fd2 | 2018-02-23 13:43:50 +0000 | [diff] [blame] | 55 | grep -Hnir --exclude-dir=assembly --exclude-dir=convolution --exclude-dir=arm_gemm "#else$\|#endif$" $DIRECTORIES | tee bad_style.log |
Anthony Barbier | ac69aa1 | 2017-07-03 17:39:37 +0100 | [diff] [blame] | 56 | if [[ $(cat bad_style.log | wc -l) > 0 ]] |
| 57 | then |
| 58 | echo "" |
Ioan-Cristian Szabo | 33fd07b | 2017-10-26 15:42:24 +0100 | [diff] [blame] | 59 | echo "ERROR: #else and #endif should be followed by a comment of the guard they refer to (e.g /* ARM_COMPUTE_AARCH64_V8_2 */ )" |
Anthony Barbier | ac69aa1 | 2017-07-03 17:39:37 +0100 | [diff] [blame] | 60 | exit -1 |
| 61 | fi |
| 62 | |
Pablo Tello | eb82fd2 | 2018-02-23 13:43:50 +0000 | [diff] [blame] | 63 | grep -Hnir --exclude-dir=assembly --exclude-dir=convolution --exclude-dir=arm_gemm "ARM_COMPUTE_AARCH64_V8_2" ./tests/validation/CL | tee bad_style.log |
Moritz Pflanzer | 4dfc235 | 2017-08-02 14:51:36 +0100 | [diff] [blame] | 64 | if [[ $(cat bad_style.log | wc -l) > 0 ]] |
| 65 | then |
| 66 | echo "" |
Ioan-Cristian Szabo | 33fd07b | 2017-10-26 15:42:24 +0100 | [diff] [blame] | 67 | echo "ERROR: Found ARM_COMPUTE_AARCH64_V8_2 in CL tests though armv8.2 features (FP16) are always supported for OpenCL" |
Moritz Pflanzer | 4dfc235 | 2017-08-02 14:51:36 +0100 | [diff] [blame] | 68 | exit -1 |
| 69 | fi |
Anthony Barbier | ac69aa1 | 2017-07-03 17:39:37 +0100 | [diff] [blame] | 70 | |
Anthony Barbier | 6ff3b19 | 2017-09-04 18:44:23 +0100 | [diff] [blame] | 71 | spdx_missing=0 |
| 72 | for f in $(find $DIRECTORIES -type f) |
| 73 | do |
| 74 | if [[ $(grep SPDX $f | wc -l) == 0 ]] |
| 75 | then |
| 76 | # List of exceptions: |
| 77 | case `basename $f` in |
| 78 | "arm_compute_version.embed");; |
| 79 | ".clang-format");; |
| 80 | ".clang-tidy");; |
| 81 | #It's an error for other files to not contain the MIT header: |
| 82 | *) |
| 83 | spdx_missing=1 |
| 84 | echo $f; |
| 85 | ;; |
| 86 | esac |
| 87 | fi; |
| 88 | done |
| 89 | |
| 90 | if [[ $spdx_missing > 0 ]] |
| 91 | then |
| 92 | echo "" |
| 93 | echo "ERROR: MIT Copyright header missing from the file(s) above." |
| 94 | exit -1 |
| 95 | fi |