diff options
author | Jon Marius Venstad <jonmv@users.noreply.github.com> | 2024-01-25 14:27:02 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-01-25 14:27:02 +0100 |
commit | 6a53e91ea2c4fcc6fde43004473ce985cfdca8dd (patch) | |
tree | c75f62b0b981972e513b1664d40151b6b5ff682c /document | |
parent | ecc147f0e2b7823501855500e32444d716dcab9e (diff) |
Apply suggestions from code review
Diffstat (limited to 'document')
-rw-r--r-- | document/src/main/java/com/yahoo/document/json/LazyTokenBuffer.java | 5 | ||||
-rw-r--r-- | document/src/main/java/com/yahoo/document/json/readers/TensorReader.java | 2 |
2 files changed, 5 insertions, 2 deletions
diff --git a/document/src/main/java/com/yahoo/document/json/LazyTokenBuffer.java b/document/src/main/java/com/yahoo/document/json/LazyTokenBuffer.java index 7798cd93909..0fbdd0b28c7 100644 --- a/document/src/main/java/com/yahoo/document/json/LazyTokenBuffer.java +++ b/document/src/main/java/com/yahoo/document/json/LazyTokenBuffer.java @@ -39,7 +39,10 @@ public class LazyTokenBuffer extends TokenBuffer { return null; Token token = buffered.get(); - if (token == null) tokens.add(token = nextToken()); + if (token == null) { + token = nextToken(); + tokens.add(token); + } localNesting += nestingOffset(token.token); return token; } diff --git a/document/src/main/java/com/yahoo/document/json/readers/TensorReader.java b/document/src/main/java/com/yahoo/document/json/readers/TensorReader.java index 27179ef32de..1fd4029b1a5 100644 --- a/document/src/main/java/com/yahoo/document/json/readers/TensorReader.java +++ b/document/src/main/java/com/yahoo/document/json/readers/TensorReader.java @@ -209,7 +209,7 @@ public class TensorReader { if (buffer.current() != JsonToken.START_ARRAY) return false; Supplier<Token> lookahead = buffer.lookahead(); Token next; - while ((next = lookahead.get()).token == JsonToken.START_ARRAY); + while ((next = lookahead.get()).token == JsonToken.START_ARRAY) { } return next.token == JsonToken.START_OBJECT; } |