You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Jonathan Ellis (JIRA)" <ji...@apache.org> on 2011/02/07 20:24:57 UTC

[jira] Commented: (CASSANDRA-2125) Convert to using properly nested inner-classes.

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

Jonathan Ellis commented on CASSANDRA-2125:
-------------------------------------------

I think I'm -1 on doing bulk refactoring to satisfy OCD at this point.  It's not worth the cost of making merging a bitch.

> Convert to using properly nested inner-classes.
> -----------------------------------------------
>
>                 Key: CASSANDRA-2125
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-2125
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Gary Dusbabek
>            Priority: Trivial
>              Labels: nit
>             Fix For: 0.8
>
>
> We do:
> class Foo{}
> class InnerFoo{}
> in Foo.java where we should be doing:
> class Foo{
>     private /*possibly static*/ class InnerFoo{}
> }
> The semantics of the former are not well-defined per the JLS and will get messy if we ever want a class called InnerFoo.

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira