You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by ju...@apache.org on 2012/07/28 13:49:27 UTC

svn commit: r1366653 - /jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/value/ValueImpl.java

Author: jukka
Date: Sat Jul 28 11:49:26 2012
New Revision: 1366653

URL: http://svn.apache.org/viewvc?rev=1366653&view=rev
Log:
OAK-200: Replace Commons Collections with Guava

Avoid triggering Value state change on getString().

Modified:
    jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/value/ValueImpl.java

Modified: jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/value/ValueImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/value/ValueImpl.java?rev=1366653&r1=1366652&r2=1366653&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/value/ValueImpl.java (original)
+++ jackrabbit/oak/trunk/oak-jcr/src/main/java/org/apache/jackrabbit/oak/jcr/value/ValueImpl.java Sat Jul 28 11:49:26 2012
@@ -187,12 +187,11 @@ class ValueImpl implements Value {
                             "In this case a new Value instance must be acquired in order to successfully call this method.");
                 }
                 try {
-                    stream = getNewStream();
                     return CharStreams.toString(CharStreams.newReaderSupplier(
                             new InputSupplier<InputStream>() {
                                 @Override
                                 public InputStream getInput() {
-                                    return stream;
+                                    return value.getNewStream();
                                 }
                             }, Charsets.UTF_8));
                 } catch (IOException e) {