RFR: 6635: Add whitespace rule to JMC jcheck configuration

[Christoph Langer]

On Mon, 18 Nov 2019 08:25:14 GMT, Christoph Langer <clanger at openjdk.org> wrote:

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

> I added a PR for a first trial run with spotless here: #5, let me know what you think!

Cool. I'll look into it.

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