RFR: 6635: Add whitespace rule to JMC jcheck configuration

[Christoph Langer]

On Fri, 15 Nov 2019 23:16:27 GMT, Marcus Hirt <hirt at openjdk.org> wrote:

> On Fri, 15 Nov 2019 22:19:25 GMT, Marcus Hirt <hirt at openjdk.org> wrote:
> 
>> 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.
> 
> I added a PR for a first trial run with spotless here: #5, let me know what you think!

> 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.

What could be the problems? I think we would not want to have tabs in java code, would we?

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