You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Robert Stupp (JIRA)" <ji...@apache.org> on 2015/11/22 14:22:10 UTC
[jira] [Created] (CASSANDRA-10750) Minor code improvements
Robert Stupp created CASSANDRA-10750:
----------------------------------------
Summary: Minor code improvements
Key: CASSANDRA-10750
URL: https://issues.apache.org/jira/browse/CASSANDRA-10750
Project: Cassandra
Issue Type: Improvement
Reporter: Robert Stupp
Assignee: Robert Stupp
Priority: Minor
Went though several IDE inspections and found some places in the code that could be improved. These are just minor improvements and no bug fixes (except one minor "theoretical" thing).
The [branch on github against trunk|https://github.com/snazy/cassandra/tree/10750-code-opts-trunk] contains a series of commits:
* simplify Mutation.apply to remove the casts
* "minor code improvements" just replaces some expressions that are effectively constant
* remove unused assignments (probably just cosmetic)
* collapse identical if-branches (probably just cosmetic)
* empty array constants
* fix printf usage (could potentially raise an exception in printf)
* replace tail-recursion in some critical sections (as the JVM cannot optimize that AFAIK)
* remove methods identical to their super methods (probably just cosmetic)
[cassci results here|http://cassci.datastax.com/view/Dev/view/snazy/search/?q=snazy-10750-]
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)