[vim/vim] Exclude generated files from style checking (PR #15309)

K.Takata

unread,

Jul 19, 2024, 9:17:26 PM (2 days ago) Jul 19

to vim/vim, Subscribed

Some OLE-related auto-generated files may contain space errors:
https://ci.appveyor.com/project/chrisbra/vim-win32-installer/builds/50248542/job/w45ve9yd6qmmws8t#L11475

	From test_codestyle.vim:
	Found errors in Test_source_files():
	command line..script C:/projects/vim-win32-installer/vim/src/testdir/runtest.vim[607]..function RunTheTest[57]..Test_source_files[8]..<SNR>8_PerformCheck[11]..<SNR>8_ReportError line 2: ../dlldata.c line 2: trailing white space
	command line..script C:/projects/vim-win32-installer/vim/src/testdir/runtest.vim[607]..function RunTheTest[57]..Test_source_files[8]..<SNR>8_PerformCheck[11]..<SNR>8_ReportError line 2: ../iid_ole.c line 12: trailing white space
	command line..script C:/projects/vim-win32-installer/vim/src/testdir/runtest.vim[607]..function RunTheTest[57]..Test_source_files[6]..<SNR>8_PerformCheck[11]..<SNR>8_ReportError line 2: ../if_ole.h line 60: space before Tab
	command line..script C:/projects/vim-win32-installer/vim/src/testdir/runtest.vim[607]..function RunTheTest[57]..Test_source_files[8]..<SNR>8_PerformCheck[11]..<SNR>8_ReportError line 2: ../if_ole.h line 10: trailing white space

Exclude them from style checking.

You can view, comment on, or merge this pull request online at:

https://github.com/vim/vim/pull/15309

Commit Summary

a5d7ab2 Exclude generated files from style checking

File Changes

(1 file)

M src/testdir/test_codestyle.vim (7)

Patch Links:

—
Reply to this email directly, view it on GitHub.
You are receiving this because you are subscribed to this thread.

Christian Brabandt

unread,

Jul 20, 2024, 5:55:20 AM (2 days ago) Jul 20

to vim/vim, Subscribed

I noticed this one before and wonder why this sometimes happens. I wanted to exclude it from the codestyle test, but forgot about it again :/.

Thanks!

—
Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.

Christian Brabandt

unread,

Jul 20, 2024, 5:58:08 AM (2 days ago) Jul 20

to vim/vim, Subscribed

Closed #15309 via c8a582a.

—
Reply to this email directly, view it on GitHub.

You are receiving this because you are subscribed to this thread.

K.Takata

unread,

Jul 20, 2024, 7:48:36 AM (2 days ago) Jul 20

to vim/vim, Subscribed

This should be a timestamp issue. If if_ole.idl is newer than other if_ole files, the files are regenerated and it causes the problem.