RFR: 8339336: Fix build system whitespace to adhere to coding conventions [v2]
Magnus Ihse Bursie
ihse at openjdk.org
Sun Sep 1 21:35:02 UTC 2024
> The build system code has unfortunately diverted in some places from the conventions as described in https://openjdk.org/groups/build/doc/code-conventions.html.
>
> Instead of trying to fix these when touching code nearby, I'd like to make an effort to fix all issues at once and separately. Incremental fixes has their benefit, but they can also muddy the actual fix and are not always appreciated.
>
> The updates in this patch have all been discovered using automated tools, but each and every change has been manually scrutinized. Those that the automatic tools pointed out that, but that were not obviously or clear-cut safe (e.g. adding spaces after comma, in `subst` or similar situations) were reverted before I pushed. I chose to err on the "First, do no harm" side, so there might be places that could have been corrected, but were not.
>
> I have made a single type of change per commit in this branch. It might be easier to review this by looking at one commit at a time.
Magnus Ihse Bursie has updated the pull request incrementally with one additional commit since the last revision:
Fix bad change in BuildMicrobenchmarks
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/20798/files
- new: https://git.openjdk.org/jdk/pull/20798/files/0cbd0923..f25fa856
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=20798&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=20798&range=00-01
Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/jdk/pull/20798.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/20798/head:pull/20798
PR: https://git.openjdk.org/jdk/pull/20798
More information about the build-dev
mailing list