False positive with GenericWhitespace - Array about checkstyle HOT 3 CLOSED

About the second case:

1. There is specific comment in the checkstyle source that this exact case should be reported:
   https://github.com/checkstyle/checkstyle/blob/master/src/main/java/com/puppycrawl/tools/checkstyle/checks/whitespace/GenericWhitespaceCheck.java#L89
2. I use this check to catch files that were not autoformatted. And autoformatter preferences for IDEA at my work (slightly modified defaults) remove this space if it is added.

So should we change the second case at all?

from checkstyle.

Hi @isopov , thanks for investigation, after attentive look at this case :

ListResult<String> result = new ListResult<String>(Collections.<String> emptyList());

I do not think it is good idea to have space at this place. With space it looks like type of some new form of method reference/call.

I am voting for ignoring that request (to have a space after ">") and fix the issue.

from checkstyle.

Done.

from checkstyle.