[ 
https://issues.apache.org/jira/browse/IGNITE-15348?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17444323#comment-17444323
 ] 

Andrei Demidov commented on IGNITE-15348:
-----------------------------------------

[~timonin.maksim], hi!

Could you, please, clarify one thing about code style and this issue. Is it 
proposed to eliminate blank spaces even after cast methods? 

It is said in Ignite code style [1] that "Source code has special spacing rules 
(mostly inherited from Sun's Java source code guidelines" but in mentioned 
Sun's Java source code guidelines it is said " Casts should be followed by a 
blank"

Thanks!

[1] 
[https://cwiki.apache.org/confluence/display/IGNITE/Coding+Guidelines#CodingGuidelines-Spacing|#CodingGuidelines-Spacing]

> Checkstyle should check whitespace after cast token. 
> -----------------------------------------------------
>
>                 Key: IGNITE-15348
>                 URL: https://issues.apache.org/jira/browse/IGNITE-15348
>             Project: Ignite
>          Issue Type: New Feature
>            Reporter: Maksim Timonin
>            Assignee: Andrei Demidov
>            Priority: Major
>              Labels: newbie
>
> According to Ignite code style [1]  there shouldn't a whitespace after the 
> ")" token. Let's add check for that.
>  
> Solution is add the TYPECAST token to the NoWhitespaceAfter  module
> <module name="NoWhitespaceAfter">
>  <property name="tokens" value="AT, INC, DEC, UNARY_MINUS, UNARY_PLUS, BNOT, 
> LNOT, DOT, ARRAY_DECLARATOR,
>  INDEX_OP, METHOD_REF, TYPECAST"/>
> </module>
>  
> Also it is required to fix all issues within repo.
>  
> [1] 
> https://cwiki.apache.org/confluence/display/IGNITE/Coding+Guidelines#CodingGuidelines-Whitespacesandemptylines



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to