You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Matteo Bertozzi (JIRA)" <ji...@apache.org> on 2014/11/20 10:39:33 UTC

[jira] [Commented] (HBASE-12527) Remove padding around parens

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

Matteo Bertozzi commented on HBASE-12527:
-----------------------------------------

do we really need to do this as a single patch? it will be painful then to cherry-pick stuff...
can't we just keep fixing those kind of things when we change the code around them as we do today?

> Remove padding around parens
> ----------------------------
>
>                 Key: HBASE-12527
>                 URL: https://issues.apache.org/jira/browse/HBASE-12527
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Sean Busbey
>            Priority: Minor
>
> We have checkstyle [enforcing no padding around parenthesis|http://checkstyle.sourceforge.net/config_whitespace.html#ParenPad] and that matches [our ref guide|http://hbase.apache.org/book.html#common.patch.feedback.space.invaders].
> Right now we have ~150 instances of not meeting this standard.
> list impacted files:
> {code}
> grep -rl -E "(\( |[^ ] \))" */src/* | grep java | grep -v generated
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)