RFR: 8240499: Enforce whitespace checking for additional source files
Kevin Rushforth
kcr at openjdk.java.net
Fri Sep 11 13:29:04 UTC 2020
The `.jcheck/conf` file is configured to check the same set of files as the old HG jcheck, namely files with the
following extensions:
.java, .c, .h, .cpp, .hpp
The Skara git jcheck allows us to evolve the rules for white space checking compatibly.
This PR adds the following additional file extensions to the list of source files that need to be kept whitespace-clean:
.cc, .css, .frag, .fxml, .g4, .gradle, .groovy, .hlsl, .jsl, .m, .metal, .mm, .stg, .vert
For ease of review, I have done the initial push as 2 commits. The first modifies the `.jcheck/conf` file and
`tools/scripts/checkWhiteSpace` script to add the additional extensions. The second fixes the whitespace errors in the
(39) files among those with the newly added extensions.
-------------
Commit messages:
- 8240499: Enforce whitespace checking for additional source files
- 8240499: Enforce whitespace checking for additional source files
Changes: https://git.openjdk.java.net/jfx/pull/301/files
Webrev: https://webrevs.openjdk.java.net/?repo=jfx&pr=301&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8240499
Stats: 148 lines in 41 files changed: 0 ins; 16 del; 132 mod
Patch: https://git.openjdk.java.net/jfx/pull/301.diff
Fetch: git fetch https://git.openjdk.java.net/jfx pull/301/head:pull/301
PR: https://git.openjdk.java.net/jfx/pull/301
More information about the openjfx-dev
mailing list