RFR: 8328074: Add jcheck whitespace checking for assembly files [v2]
Magnus Ihse Bursie
ihse at openjdk.org
Wed Mar 13 11:39:22 UTC 2024
> As part of the ongoing effort to enable jcheck whitespace checking to all text files, it is now time to address assembly (.S) files. The hotspot assembly files were fixed as part of the hotspot mapfile removal, so only a single incorrect jdk library remains.
>
> The main issue with `libjsvml` was inconsistent line starts, sometimes using tabs. I used the `expand` unix command line tool to replace these with spaces.
Magnus Ihse Bursie has updated the pull request incrementally with two additional commits since the last revision:
- Only show first 200 lines of long diffs
- Actually remove emptied file
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/18268/files
- new: https://git.openjdk.org/jdk/pull/18268/files/ce1d4c9f..49fb1016
Webrevs:
- full: Webrev is not available because diff is too large
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=18268&range=00-01
Stats: 9 lines in 3 files changed: 1 ins; 0 del; 8 mod
Patch: https://git.openjdk.org/jdk/pull/18268.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/18268/head:pull/18268
PR: https://git.openjdk.org/jdk/pull/18268
More information about the build-dev
mailing list