[
https://issues.apache.org/jira/browse/MAHOUT-233?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12796635#action_12796635
]
Robin Anil commented on MAHOUT-233:
-----------------------------------
No. relaxing the space occurs only in checkstyle. In the code formatter, does
keep space for method params. But it strips space from Map<String,String>
bq. Yes but how does one fixed indent value like "81" possibly work for all
code? I probably don't understand the value.
try the Eclipse Code style in the editor
those indentation values are Eclipse specific, what it does is this.
{noformat}
String foo(int param1,
Object param2,
Float param3) throws Exception1,
Exception2
Exception3
{noformat}
> 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.
