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 th...@apache.org on 2012/05/10 17:43:58 UTC

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

Author: thomasm
Date: Thu May 10 15:43:58 2012
New Revision: 1336751

URL: http://svn.apache.org/viewvc?rev=1336751&view=rev
Log:
OAK-16 Proper ValueFactory implementation and Value handling (re-add javadocs as requested by Angela)

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=1336751&r1=1336750&r2=1336751&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 Thu May 10 15:43:58 2012
@@ -37,7 +37,7 @@ import java.util.Calendar;
 import java.util.TimeZone;
 
 /**
- * The implementation of JCR values.
+ * ValueImpl...
  */
 class ValueImpl implements Value {
 
@@ -67,12 +67,17 @@ class ValueImpl implements Value {
     }
 
     //--------------------------------------------------------------< Value >---
-
+    /**
+     * @see javax.jcr.Value#getType()
+     */
     @Override
     public int getType() {
         return value.getType();
     }
 
+    /**
+     * @see javax.jcr.Value#getBoolean()
+     */
     @Override
     public boolean getBoolean() throws RepositoryException {
         if (getType() == PropertyType.STRING || getType() == PropertyType.BINARY || getType() == PropertyType.BOOLEAN) {
@@ -82,6 +87,9 @@ class ValueImpl implements Value {
         }
     }
 
+    /**
+     * @see javax.jcr.Value#getDate()
+     */
     @Override
     public Calendar getDate() throws RepositoryException {
         Calendar cal;
@@ -101,6 +109,9 @@ class ValueImpl implements Value {
         return cal;
     }
 
+    /**
+     * @see javax.jcr.Value#getDecimal()
+     */
     @Override
     public BigDecimal getDecimal() throws RepositoryException {
         try {
@@ -116,6 +127,9 @@ class ValueImpl implements Value {
         }
     }
 
+    /**
+     * @see javax.jcr.Value#getDouble()
+     */
     @Override
     public double getDouble() throws RepositoryException {
         try {
@@ -131,6 +145,9 @@ class ValueImpl implements Value {
         }
     }
 
+    /**
+     * @see javax.jcr.Value#getLong()
+     */
     @Override
     public long getLong() throws RepositoryException {
         try {
@@ -146,6 +163,9 @@ class ValueImpl implements Value {
         }
     }
 
+    /**
+     * @see javax.jcr.Value#getString()
+     */
     @Override
     public String getString() throws RepositoryException {
         switch (getType()) {
@@ -167,6 +187,9 @@ class ValueImpl implements Value {
         }
     }
 
+    /**
+     * @see javax.jcr.Value#getStream()
+     */
     @Override
     public InputStream getStream() throws IllegalStateException, RepositoryException {
         if (stream == null) {
@@ -187,13 +210,18 @@ class ValueImpl implements Value {
         return stream;
     }
 
+    /**
+     * @see javax.jcr.Value#getBinary()
+     */
     @Override
     public Binary getBinary() throws RepositoryException {
         return new BinaryImpl(this);
     }
 
     //-------------------------------------------------------------< Object >---
-
+    /**
+     * @see Object#equals(Object)
+     */
     @Override
     public boolean equals(Object obj) {
         if (obj instanceof ValueImpl) {
@@ -203,7 +231,10 @@ class ValueImpl implements Value {
         }
     }
 
-     @Override
+    /**
+     * @see Object#hashCode()
+     */
+    @Override
     public int hashCode() {
         return value.hashCode();
     }