[
https://issues.apache.org/jira/browse/MAHOUT-233?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12796629#action_12796629
]
Sean Owen commented on MAHOUT-233:
----------------------------------
Really, I've never seen Java code with this convention. So it's like "public
void foo(int i,int j)" and "foo(bar,baz)". All the Mahout code doesn't do this.
I'd vote to not take this convention, but don't feel strongly enough to argue
about it. It will be a big ol' patch to change that.
Yes but how does one fixed indent value like "81" possibly work for all code? I
probably don't understand the value.
> 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.
