|
1 | 1 | #!/bin/bash |
2 | 2 |
|
3 | | -set -e |
4 | | - |
| 3 | +echo |
| 4 | +echo "Checking For MaNGOS Coding Standards:" |
5 | 5 | echo "Starting Codestyling Script:" |
6 | 6 | echo |
7 | 7 |
|
8 | 8 | declare -A singleLineRegexChecks=( |
9 | 9 | ["[[:blank:]]$"]="Remove whitespace at the end of the lines above" |
10 | 10 | ["\t"]="Replace tabs with 4 spaces in the lines above" |
| 11 | + ["^[[:blank:]]*(?:[a-zA-Z_][a-zA-Z_0-9]*[[:blank:]]+)*[a-zA-Z_][a-zA-Z_0-9]*[[:blank:]]*\([^)]*\)[[:blank:]]*\{[[:blank:]]*$"]="Move opening brace to a new line after function definition" |
| 12 | + ["\{[[:blank:]]*\S"]="Opening brace must be on its own line (no code after '{')" |
| 13 | + ["\S[[:blank:]]*\}"]="Closing brace must be on its own line (no code before '}')" |
| 14 | + ["^[[:blank:]]*if\s*\(.*\)[[:blank:]]*(?!\{)"]="if statement must use braces" |
| 15 | + ["^[[:blank:]]*else[[:blank:]]*(?!if|\{)"]="else statement must use braces" |
| 16 | + ["\bif\("]="Missing space between 'if' and '('" |
11 | 17 | ) |
12 | 18 |
|
13 | | -for check in ${!singleLineRegexChecks[@]}; do |
14 | | - echo " Checking RegEx: '${check}'" |
15 | | - |
16 | | - if grep -P -r -I -n ${check} src; then |
17 | | - echo |
18 | | - echo "${singleLineRegexChecks[$check]}" |
19 | | - exit 1 |
| 19 | +# Ignore directories |
| 20 | +grep_exclude_args=( |
| 21 | + --exclude-dir="Eluna" |
| 22 | + --exclude-dir="tools" |
| 23 | +) |
| 24 | + |
| 25 | +# Accept multiple input paths |
| 26 | +input_paths=("$@") |
| 27 | +if [[ ${#input_paths[@]} -eq 0 ]]; then |
| 28 | + input_paths=("src") # fallback |
| 29 | +fi |
| 30 | + |
| 31 | +# Track errors and triggered rules |
| 32 | +hadError=0 |
| 33 | +declare -A results |
| 34 | +declare -a triggeredDescriptions |
| 35 | + |
| 36 | +# Perform checks |
| 37 | +for check in "${!singleLineRegexChecks[@]}"; do |
| 38 | + matches=$(grep -P -r -I -n "${grep_exclude_args[@]}" "${input_paths[@]}" -e "$check" 2>/dev/null) |
| 39 | + if [[ -n "$matches" ]]; then |
| 40 | + results["$check"]="$matches" |
| 41 | + triggeredDescriptions+=("${singleLineRegexChecks[$check]}") |
| 42 | + hadError=1 |
20 | 43 | fi |
21 | 44 | done |
22 | 45 |
|
23 | | -# declare -A multiLineRegexChecks=( |
24 | | -# ["\n\n\n"]="Multiple blank lines detected, keep only one. Check the files above" |
25 | | -# ) |
26 | | - |
27 | | -# for check in ${!multiLineRegexChecks[@]}; do |
28 | | -# echo " Checking RegEx: '${check}'" |
| 46 | +# Display results |
| 47 | +if [[ $hadError -eq 1 ]]; then |
| 48 | + echo |
| 49 | + for check in "${!results[@]}"; do |
| 50 | + echo "== Rule: ${singleLineRegexChecks[$check]} ==" |
| 51 | + echo "${results[$check]}" |
| 52 | + echo |
| 53 | + done |
29 | 54 |
|
30 | | -# if grep -Pzo -r -I ${check} src; then |
31 | | -# echo |
32 | | -# echo |
33 | | -# echo "${multiLineRegexChecks[$check]}" |
34 | | -# exit 1 |
35 | | -# fi |
36 | | -# done |
| 55 | + echo "------------------------------------------" |
| 56 | + echo "Summary of Triggered Rules:" |
| 57 | + for rule in "${triggeredDescriptions[@]}"; do |
| 58 | + echo "- $rule" |
| 59 | + done |
| 60 | + echo "------------------------------------------" |
37 | 61 |
|
38 | | -echo |
39 | | -echo "Awesome! No issues..." |
| 62 | + exit 1 |
| 63 | +else |
| 64 | + echo "All checks passed. No issues found." |
| 65 | + exit 0 |
| 66 | +fi |
0 commit comments