Bram Moolenaar | ebfec1c | 2023-01-22 21:14:53 +0000 | [diff] [blame] | 1 | " Test for checking the source code style. |
| 2 | |
Bram Moolenaar | abc8130 | 2023-06-04 16:55:27 +0100 | [diff] [blame] | 3 | def s:ReportError(fname: string, lnum: number, msg: string) |
| 4 | if lnum > 0 |
| 5 | assert_report(fname .. ' line ' .. lnum .. ': ' .. msg) |
| 6 | endif |
| 7 | enddef |
| 8 | |
Christian Brabandt | b147d31 | 2023-09-01 17:58:35 +0100 | [diff] [blame] | 9 | def s:PerformCheck(fname: string, pattern: string, msg: string, skip: string) |
John Marriott | 78d742a | 2024-04-02 20:26:01 +0200 | [diff] [blame] | 10 | var prev_lnum = 1 |
Christian Brabandt | b147d31 | 2023-09-01 17:58:35 +0100 | [diff] [blame] | 11 | var lnum = 1 |
| 12 | while (lnum > 0) |
| 13 | cursor(lnum, 1) |
| 14 | lnum = search(pattern, 'W', 0, 0, skip) |
John Marriott | 78d742a | 2024-04-02 20:26:01 +0200 | [diff] [blame] | 15 | if (prev_lnum == lnum) |
| 16 | break |
| 17 | endif |
| 18 | prev_lnum = lnum |
Christian Brabandt | b147d31 | 2023-09-01 17:58:35 +0100 | [diff] [blame] | 19 | if (lnum > 0) |
John Marriott | 78d742a | 2024-04-02 20:26:01 +0200 | [diff] [blame] | 20 | ReportError(fname, lnum, msg) |
Christian Brabandt | b147d31 | 2023-09-01 17:58:35 +0100 | [diff] [blame] | 21 | endif |
| 22 | endwhile |
| 23 | enddef |
| 24 | |
Bram Moolenaar | ebfec1c | 2023-01-22 21:14:53 +0000 | [diff] [blame] | 25 | def Test_source_files() |
Christian Brabandt | b147d31 | 2023-09-01 17:58:35 +0100 | [diff] [blame] | 26 | for fname in glob('../*.[ch]', 0, 1) + ['../xxd/xxd.c'] |
Bram Moolenaar | bf63011 | 2023-05-19 21:41:02 +0100 | [diff] [blame] | 27 | bwipe! |
Bram Moolenaar | abc8130 | 2023-06-04 16:55:27 +0100 | [diff] [blame] | 28 | g:ignoreSwapExists = 'e' |
Bram Moolenaar | ebfec1c | 2023-01-22 21:14:53 +0000 | [diff] [blame] | 29 | exe 'edit ' .. fname |
| 30 | |
Ken Takata | c8a582a | 2024-07-20 11:55:13 +0200 | [diff] [blame] | 31 | # Some files are generated files and may contain space errors. |
| 32 | if fname =~ 'dlldata.c' |
| 33 | || fname =~ 'if_ole.h' |
| 34 | || fname =~ 'iid_ole.c' |
| 35 | continue |
| 36 | endif |
| 37 | |
Christian Brabandt | b147d31 | 2023-09-01 17:58:35 +0100 | [diff] [blame] | 38 | PerformCheck(fname, ' \t', 'space before Tab', '') |
Bram Moolenaar | ebfec1c | 2023-01-22 21:14:53 +0000 | [diff] [blame] | 39 | |
Christian Brabandt | b147d31 | 2023-09-01 17:58:35 +0100 | [diff] [blame] | 40 | PerformCheck(fname, '\s$', 'trailing white space', '') |
Bram Moolenaar | ebfec1c | 2023-01-22 21:14:53 +0000 | [diff] [blame] | 41 | |
| 42 | # some files don't stick to the Vim style rules |
| 43 | if fname =~ 'iscygpty.c' |
| 44 | continue |
| 45 | endif |
| 46 | |
Bram Moolenaar | ebfec1c | 2023-01-22 21:14:53 +0000 | [diff] [blame] | 47 | var skip = 'getline(".") =~ "condition) {" || getline(".") =~ "vimglob_func" || getline(".") =~ "{\"" || getline(".") =~ "{\\d" || getline(".") =~ "{{{"' |
Christian Brabandt | b147d31 | 2023-09-01 17:58:35 +0100 | [diff] [blame] | 48 | PerformCheck(fname, ')\s*{', 'curly after closing paren', skip) |
Bram Moolenaar | ebfec1c | 2023-01-22 21:14:53 +0000 | [diff] [blame] | 49 | |
Bram Moolenaar | ebfec1c | 2023-01-22 21:14:53 +0000 | [diff] [blame] | 50 | # Examples in comments use double quotes. |
| 51 | skip = "getline('.') =~ '\"'" |
Bram Moolenaar | bf63011 | 2023-05-19 21:41:02 +0100 | [diff] [blame] | 52 | |
Christian Brabandt | b147d31 | 2023-09-01 17:58:35 +0100 | [diff] [blame] | 53 | PerformCheck(fname, '}\s*else', 'curly before "else"', skip) |
Bram Moolenaar | ebfec1c | 2023-01-22 21:14:53 +0000 | [diff] [blame] | 54 | |
Christian Brabandt | b147d31 | 2023-09-01 17:58:35 +0100 | [diff] [blame] | 55 | PerformCheck(fname, 'else\s*{', 'curly after "else"', skip) |
Bram Moolenaar | c9471b1 | 2023-05-09 15:00:00 +0100 | [diff] [blame] | 56 | |
Christian Brabandt | b147d31 | 2023-09-01 17:58:35 +0100 | [diff] [blame] | 57 | PerformCheck(fname, '\<\(if\|while\|for\)(', 'missing white space after "if"/"while"/"for"', skip) |
Bram Moolenaar | ebfec1c | 2023-01-22 21:14:53 +0000 | [diff] [blame] | 58 | endfor |
| 59 | |
| 60 | bwipe! |
| 61 | enddef |
| 62 | |
Bram Moolenaar | 94722c5 | 2023-01-28 19:19:03 +0000 | [diff] [blame] | 63 | def Test_test_files() |
| 64 | for fname in glob('*.vim', 0, 1) |
Bram Moolenaar | abc8130 | 2023-06-04 16:55:27 +0100 | [diff] [blame] | 65 | g:ignoreSwapExists = 'e' |
Bram Moolenaar | 94722c5 | 2023-01-28 19:19:03 +0000 | [diff] [blame] | 66 | exe 'edit ' .. fname |
| 67 | |
| 68 | # some files intentionally have misplaced white space |
| 69 | if fname =~ 'test_cindent.vim' || fname =~ 'test_join.vim' |
| 70 | continue |
| 71 | endif |
| 72 | |
| 73 | # skip files that are known to have a space before a tab |
| 74 | if fname !~ 'test_comments.vim' |
| 75 | && fname !~ 'test_listchars.vim' |
| 76 | && fname !~ 'test_visual.vim' |
| 77 | cursor(1, 1) |
Christian Brabandt | 22105fd | 2024-07-15 20:51:11 +0200 | [diff] [blame] | 78 | var skip = 'getline(".") =~ "codestyle: ignore"' |
| 79 | var lnum = search(fname =~ "test_regexp_latin" ? '[^รก] \t' : ' \t', 'W', 0, 0, skip) |
Bram Moolenaar | abc8130 | 2023-06-04 16:55:27 +0100 | [diff] [blame] | 80 | ReportError('testdir/' .. fname, lnum, 'space before Tab') |
Bram Moolenaar | 94722c5 | 2023-01-28 19:19:03 +0000 | [diff] [blame] | 81 | endif |
| 82 | |
| 83 | # skip files that are known to have trailing white space |
| 84 | if fname !~ 'test_cmdline.vim' |
| 85 | && fname !~ 'test_let.vim' |
| 86 | && fname !~ 'test_tagjump.vim' |
| 87 | && fname !~ 'test_vim9_cmd.vim' |
Doug Kearns | dbe39ed | 2025-01-04 17:12:24 +0100 | [diff] [blame] | 88 | && fname !~ 'test_vim9_enum.vim' |
Bram Moolenaar | 94722c5 | 2023-01-28 19:19:03 +0000 | [diff] [blame] | 89 | cursor(1, 1) |
| 90 | var lnum = search( |
| 91 | fname =~ 'test_vim9_assign.vim' ? '[^=]\s$' |
| 92 | : fname =~ 'test_vim9_class.vim' ? '[^)]\s$' |
| 93 | : fname =~ 'test_vim9_script.vim' ? '[^,:3]\s$' |
| 94 | : fname =~ 'test_visual.vim' ? '[^/]\s$' |
| 95 | : '[^\\]\s$') |
Bram Moolenaar | abc8130 | 2023-06-04 16:55:27 +0100 | [diff] [blame] | 96 | ReportError('testdir/' .. fname, lnum, 'trailing white space') |
Bram Moolenaar | 94722c5 | 2023-01-28 19:19:03 +0000 | [diff] [blame] | 97 | endif |
| 98 | endfor |
| 99 | |
| 100 | bwipe! |
| 101 | enddef |
| 102 | |
h-east | d950984 | 2023-02-21 13:33:17 +0000 | [diff] [blame] | 103 | def Test_help_files() |
| 104 | var lnum: number |
| 105 | set nowrapscan |
| 106 | |
| 107 | for fpath in glob('../../runtime/doc/*.txt', 0, 1) |
Bram Moolenaar | abc8130 | 2023-06-04 16:55:27 +0100 | [diff] [blame] | 108 | g:ignoreSwapExists = 'e' |
h-east | d950984 | 2023-02-21 13:33:17 +0000 | [diff] [blame] | 109 | exe 'edit ' .. fpath |
| 110 | |
| 111 | var fname = fnamemodify(fpath, ":t") |
| 112 | |
| 113 | # todo.txt is for developers, it's not need a strictly check |
| 114 | # version*.txt is a history and large size, so it's not checked |
| 115 | if fname == 'todo.txt' || fname =~ 'version.*\.txt' |
| 116 | continue |
| 117 | endif |
| 118 | |
| 119 | # Check for mixed tabs and spaces |
| 120 | cursor(1, 1) |
| 121 | while 1 |
| 122 | lnum = search('[^/] \t') |
| 123 | if fname == 'visual.txt' && getline(lnum) =~ "STRING \tjkl" |
| 124 | || fname == 'usr_27.txt' && getline(lnum) =~ "\[^\? \t\]" |
| 125 | continue |
| 126 | endif |
Bram Moolenaar | abc8130 | 2023-06-04 16:55:27 +0100 | [diff] [blame] | 127 | ReportError(fpath, lnum, 'space before tab') |
h-east | d950984 | 2023-02-21 13:33:17 +0000 | [diff] [blame] | 128 | if lnum == 0 |
| 129 | break |
| 130 | endif |
| 131 | endwhile |
| 132 | |
| 133 | # Check for unnecessary whitespace at the end of a line |
| 134 | cursor(1, 1) |
| 135 | while 1 |
| 136 | lnum = search('[^/~\\]\s$') |
| 137 | # skip line that are known to have trailing white space |
| 138 | if fname == 'map.txt' && getline(lnum) =~ "unmap @@ $" |
| 139 | || fname == 'usr_12.txt' && getline(lnum) =~ "^\t/ \t$" |
| 140 | || fname == 'usr_41.txt' && getline(lnum) =~ "map <F4> o#include $" |
| 141 | || fname == 'change.txt' && getline(lnum) =~ "foobar bla $" |
| 142 | continue |
| 143 | endif |
Bram Moolenaar | abc8130 | 2023-06-04 16:55:27 +0100 | [diff] [blame] | 144 | ReportError('testdir' .. fpath, lnum, 'trailing white space') |
h-east | d950984 | 2023-02-21 13:33:17 +0000 | [diff] [blame] | 145 | if lnum == 0 |
| 146 | break |
| 147 | endif |
| 148 | endwhile |
| 149 | |
Bram Moolenaar | abc8130 | 2023-06-04 16:55:27 +0100 | [diff] [blame] | 150 | # # TODO: Check for line over 80 columns |
h-east | d950984 | 2023-02-21 13:33:17 +0000 | [diff] [blame] | 151 | # cursor(1, 1) |
| 152 | # while 1 |
| 153 | # lnum = search('\%>80v.*$') |
Bram Moolenaar | abc8130 | 2023-06-04 16:55:27 +0100 | [diff] [blame] | 154 | # ReportError(fpath, lnum, 'line over 80 columns') |
h-east | d950984 | 2023-02-21 13:33:17 +0000 | [diff] [blame] | 155 | # if lnum == 0 |
| 156 | # break |
| 157 | # endif |
| 158 | # endwhile |
| 159 | |
| 160 | endfor |
| 161 | |
| 162 | set wrapscan&vim |
| 163 | bwipe! |
| 164 | enddef |
| 165 | |
Naruhiko Nishino | c2a9000 | 2025-05-04 20:05:47 +0200 | [diff] [blame] | 166 | def Test_indent_of_source_files() |
| 167 | for fname in glob('../*.[ch]', 0, 1) + ['../xxd/xxd.c'] |
| 168 | execute 'tabnew ' .. fname |
| 169 | for lnum in range(1, line('$')) |
| 170 | var name: string = synIDattr(synID(lnum, 1, 0), 'name') |
| 171 | if -1 == index(['cComment', 'cCommentStart'], name) |
| 172 | var line: string = getline(lnum) |
| 173 | var indent: string = matchstr(line, '^\s*') |
| 174 | var tailing: string = matchstr(line, '\s*$') |
| 175 | if !empty(indent) |
| 176 | if indent !~# '^\t* \{0,7\}$' |
| 177 | ReportError('testdir/' .. fname, lnum, 'invalid indent') |
| 178 | endif |
| 179 | endif |
| 180 | if !empty(tailing) |
| 181 | ReportError('testdir/' .. fname, lnum, 'tailing spaces') |
| 182 | endif |
| 183 | endif |
| 184 | endfor |
| 185 | close |
| 186 | endfor |
| 187 | enddef |
Bram Moolenaar | ebfec1c | 2023-01-22 21:14:53 +0000 | [diff] [blame] | 188 | |
Christian Brabandt | 22105fd | 2024-07-15 20:51:11 +0200 | [diff] [blame] | 189 | " vim: shiftwidth=2 sts=2 expandtab nofoldenable |