[
https://issues.apache.org/jira/browse/MAHOUT-233?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12796627#action_12796627
]
Robin Anil commented on MAHOUT-233:
-----------------------------------
bq. does the checkstyle rule now say no space after comma? that seems
uncovnentional
Its as per the new lucene code style found on the wiki
bq. what are the new, large alignment values in codestyle?
the large alignment values forces function and method parameters in to each
line and aligns it at the opening bracket. They prevent the codestyle alignment
error.
> Modifying Mahout Check style to match our current coding style
> --------------------------------------------------------------
>
> Key: MAHOUT-233
> URL: https://issues.apache.org/jira/browse/MAHOUT-233
> Project: Mahout
> Issue Type: Task
> Affects Versions: 0.3
> Reporter: Robin Anil
> Fix For: 0.3
>
> Attachments: checkstyle-xml.patch, Eclipse-Codestyle.diff,
> Eclipse-Mahout-Codestyle.xml
>
>
> Checkstyle currently throws a lot of errors for small things like
> private static Logger log
> if(booleanVar == false) // Required for readability
> This task is created to track
> how to match the checkstyle to relax certain rules and
> changes in code for the strict ones
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
