RFR: 6635: Add whitespace rule to JMC jcheck configuration

Marcus Hirt hirt at openjdk.org
Fri Nov 15 22:19:25 UTC 2019


On Fri, 15 Nov 2019 21:51:58 GMT, Christoph Langer <clanger at openjdk.org> wrote:

> This is my first PR with skara :)
> 
> I think it would be nice if jcheck would also check for unnecessary whitespace.
> 
> ----------------
> 
> Commits:
>  - e201163f: 6635: Add whitespace rule to JMC jcheck configuration
> 
> Changes: https://git.openjdk.java.net/jmc/pull/4/files
>  Webrev: https://webrevs.openjdk.java.net/jmc/4/webrev.00
>   Issue: https://bugs.openjdk.java.net/browse/JMC-6635
>   Stats: 4 lines in 1 file changed: 3 ins; 0 del; 1 mod
>   Patch: https://git.openjdk.java.net/jmc/pull/4.diff
>   Fetch: git fetch https://git.openjdk.java.net/jmc pull/4/head:pull/4

When I discussed this with Erik (Helin), he told me there could be a problem with us using tabs instead of spaces. I think we should instead try to make spotless part of the process.

PR: https://git.openjdk.java.net/jmc/pull/4


More information about the jmc-dev mailing list