From 87ef146b4c6ad5b43b48b95af759329898bb6e01 Mon Sep 17 00:00:00 2001 From: Dave Brosius Date: Wed, 1 Apr 2015 19:16:40 -0400 Subject: [PATCH] use short circuiting ops --- .../cassandra/io/util/UnbufferedDataOutputStreamPlus.java | 2 +- .../org/apache/cassandra/stress/generate/PartitionIterator.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/java/org/apache/cassandra/io/util/UnbufferedDataOutputStreamPlus.java b/src/java/org/apache/cassandra/io/util/UnbufferedDataOutputStreamPlus.java index ac3bae5d5a..c05b5b2096 100644 --- a/src/java/org/apache/cassandra/io/util/UnbufferedDataOutputStreamPlus.java +++ b/src/java/org/apache/cassandra/io/util/UnbufferedDataOutputStreamPlus.java @@ -256,7 +256,7 @@ public abstract class UnbufferedDataOutputStreamPlus extends DataOutputStreamPlu for (int i = 0 ; i < length ; i++) { int ch = str.charAt(i); - if ((ch > 0) & (ch <= 127)) + if ((ch > 0) && (ch <= 127)) utfCount += 1; else if (ch <= 2047) utfCount += 2; diff --git a/tools/stress/src/org/apache/cassandra/stress/generate/PartitionIterator.java b/tools/stress/src/org/apache/cassandra/stress/generate/PartitionIterator.java index 3a8693d373..357699de00 100644 --- a/tools/stress/src/org/apache/cassandra/stress/generate/PartitionIterator.java +++ b/tools/stress/src/org/apache/cassandra/stress/generate/PartitionIterator.java @@ -298,7 +298,7 @@ public abstract class PartitionIterator implements Iterator for (int i = 0 ; i <= depth ; i++) { int p = currentRow[i], l = lastRow[i], r = clusteringComponents[i].size(); - if ((p == l) | (r == 1)) + if ((p == l) || (r == 1)) continue; return p - l; }