RFR: 8295729: Remove trailing whitespace from non-value lines in properties files [v3]
Magnus Ihse Bursie
ihse at openjdk.org
Mon Oct 24 19:21:07 UTC 2022
> Properties files is essentially source code. It should have the same whitespace checks as all other source code, so we don't get spurious trailing whitespace changes.
>
> With the new Skara jcheck, it is possible to increase the coverage of the whitespace checks (in the old mercurial version, this was more or less impossible).
>
> The only manual change is to `.jcheck/conf`. All other changes were made by running `find . -type f -iname "*.properties" | xargs gsed -i -e 's/[ \t]*$//'`.
Magnus Ihse Bursie has updated the pull request incrementally with two additional commits since the last revision:
- Revert "Remove check for .properties from jcheck"
This reverts commit c91fdaa19dc06351598bd1c0614e1af3bfa08ae2.
- Change trailing space and tab in values to unicode encoding
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/10792/files
- new: https://git.openjdk.org/jdk/pull/10792/files/c91fdaa1..f4f94341
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=10792&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=10792&range=01-02
Stats: 412 lines in 131 files changed: 0 ins; 0 del; 412 mod
Patch: https://git.openjdk.org/jdk/pull/10792.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/10792/head:pull/10792
PR: https://git.openjdk.org/jdk/pull/10792
More information about the compiler-dev
mailing list