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