You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cassandra.apache.org by "Dave Brosius (Updated) (JIRA)" <ji...@apache.org> on 2012/02/28 06:33:25 UTC

[jira] [Updated] (CASSANDRA-3971) [patch] make hasSameReplication safer/more readable

     [ https://issues.apache.org/jira/browse/CASSANDRA-3971?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dave Brosius updated CASSANDRA-3971:
------------------------------------

    Attachment: clean_up_hasSameReplication.diff
    
> [patch] make hasSameReplication safer/more readable
> ---------------------------------------------------
>
>                 Key: CASSANDRA-3971
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-3971
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Core
>            Reporter: Dave Brosius
>            Priority: Trivial
>         Attachments: clean_up_hasSameReplication.diff
>
>
> code was iterating over passed in list parameter, and using the list indices on a seemingly unrelated list. However, in the one case where this method is used the passed in list and the referenced list in the method were actually the same. Change code to just reference the method parameter list, so it's clear what's going on.
> patch against trunk.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira