[jira] [Commented] (OFBIZ-11886) Allow single line statements in checkstyle

[Nicolas Malin (Jira)]

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

Suraj Khurana commented on OFBIZ-11886:
---------------------------------------

Thanks Jacques.

I have corrected some straight forward occurrences of single line if statements, so I am planning to commit both, checkstyle patch and corrected occurrences.

Please confirm if this looks good to you as well.

> Allow single line statements in checkstyle
> ------------------------------------------
>
>                 Key: OFBIZ-11886
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-11886
>             Project: OFBiz
>          Issue Type: Sub-task
>          Components: ALL COMPONENTS
>            Reporter: Suraj Khurana
>            Assignee: Suraj Khurana
>            Priority: Major
>         Attachments: OFBIZ-11886-format-LoginServices-checkNewPassword.patch, OFBIZ-11886-plugins.patch, OFBIZ-11886.patch, OFBIZ-11886.patch
>
>
> We need to add this module in checkstyle.xml
> <module name="NeedBraces">
>     <property name="allowSingleLineStatement" value="true"/>
>  </module>
> After this, single line statements like:
> if (Debug.verboseOn()) Debug.logVerbose(...)
> Will be considered safe.

--
This message was sent by Atlassian Jira
(v8.3.4#803005)