aboutsummaryrefslogtreecommitdiffstats
path: root/container-search/src/main/java
diff options
context:
space:
mode:
authorJon Bratseth <bratseth@verizonmedia.com>2019-07-03 16:21:46 -0700
committerJon Bratseth <bratseth@verizonmedia.com>2019-07-03 16:21:46 -0700
commit66e2f98fc71338caaa9b0f72ee9109c209b910c6 (patch)
tree9229d7350b1bef51a4f7ed00f92dcef2fd4e7c3b /container-search/src/main/java
parent0a0f2cc1692fc9419accae1570d885a186380e3d (diff)
Nonfunctional changes only
Diffstat (limited to 'container-search/src/main/java')
-rw-r--r--container-search/src/main/java/com/yahoo/prelude/fastsearch/ByteField.java2
-rw-r--r--container-search/src/main/java/com/yahoo/prelude/hitfield/RawData.java18
-rw-r--r--container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java1
3 files changed, 10 insertions, 11 deletions
diff --git a/container-search/src/main/java/com/yahoo/prelude/fastsearch/ByteField.java b/container-search/src/main/java/com/yahoo/prelude/fastsearch/ByteField.java
index 22069d0270c..8fdc093122e 100644
--- a/container-search/src/main/java/com/yahoo/prelude/fastsearch/ByteField.java
+++ b/container-search/src/main/java/com/yahoo/prelude/fastsearch/ByteField.java
@@ -24,7 +24,7 @@ public class ByteField extends DocsumField {
if (value == EMPTY_VALUE) {
return NanNumber.NaN;
} else {
- return Byte.valueOf(value);
+ return value;
}
}
diff --git a/container-search/src/main/java/com/yahoo/prelude/hitfield/RawData.java b/container-search/src/main/java/com/yahoo/prelude/hitfield/RawData.java
index a0c9b10c519..2e9d9d8cad9 100644
--- a/container-search/src/main/java/com/yahoo/prelude/hitfield/RawData.java
+++ b/container-search/src/main/java/com/yahoo/prelude/hitfield/RawData.java
@@ -8,27 +8,27 @@ package com.yahoo.prelude.hitfield;
*/
public final class RawData {
- private byte[] content;
+ private final byte[] content;
/**
- * Constructor, takes ownership
+ * Constructor, takes ownership of the given byte array.
+ *
* @param content some bytes, handover
*/
public RawData(byte[] content) {
this.content = content;
}
- /**
- * @return internal byte array containing the actual data received
- **/
+ /** Returns the internal byte array containing the actual data received */
public byte[] getInternalData() {
return content;
}
/**
- * an ascii string; non-ascii data is escaped with hex notation
- * NB: not always uniquely reversible
- **/
+ * An ascii string; non-ascii data is escaped with hex notation.
+ * NB: not always uniquely reversible.
+ */
+ @Override
public String toString() {
StringBuilder buf = new StringBuilder();
for (byte b : content) {
@@ -46,7 +46,7 @@ public final class RawData {
} else {
// XXX maybe we should only do this? creates possibly-invalid XML though.
buf.append("&");
- buf.append(Integer.toString(i));
+ buf.append(i);
buf.append(";");
}
}
diff --git a/container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java b/container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java
index 54dfbfe1a85..af453983f89 100644
--- a/container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java
+++ b/container-search/src/main/java/com/yahoo/search/rendering/JsonRenderer.java
@@ -776,7 +776,6 @@ public class JsonRenderer extends AsynchronousSectionedRenderer<Result> {
} else {
renderInspectorDirect(data);
}
-
}
private void renderInspectorDirect(Inspector data) throws IOException {