Mailing-List: contact commits-help@jackrabbit.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@jackrabbit.apache.org
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: svn commit: r1170272 -
 /jackrabbit/sandbox/jackrabbit-mk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/ItemImpl.java
Date: Tue, 13 Sep 2011 17:58:21 -0000
To: commits@jackrabbit.apache.org
From: mduerig@apache.org
Message-Id: <20110913175821.984B823888EA@eris.apache.org>

Author: mduerig
Date: Tue Sep 13 17:58:21 2011
New Revision: 1170272

URL: http://svn.apache.org/viewvc?rev=1170272&view=rev
Log:
Microkernel based Jackrabbit prototype (WIP) 
comment

Modified:
    jackrabbit/sandbox/jackrabbit-mk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/ItemImpl.java

Modified: jackrabbit/sandbox/jackrabbit-mk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/ItemImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/sandbox/jackrabbit-mk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/ItemImpl.java?rev=1170272&r1=1170271&r2=1170272&view=diff
==============================================================================
--- jackrabbit/sandbox/jackrabbit-mk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/ItemImpl.java (original)
+++ jackrabbit/sandbox/jackrabbit-mk/jackrabbit-jcr2spi/src/main/java/org/apache/jackrabbit/jcr2spi/ItemImpl.java Tue Sep 13 17:58:21 2011
@@ -265,9 +265,7 @@ public abstract class ItemImpl implement
         }
 
         switch (state.getStatus()) {
-            // Nothing to do for
-            // - Status#EXISTING : modifications reverted or saved
-            //   inform listeners about an update (status was MODIFIED before)
+            //   Notify listeners about an update (status was MODIFIED before)
             //   or a simple refresh without modification (status was INVALIDATED).
             case EXISTING:
                 if (previousStatus == Status.INVALIDATED || previousStatus == Status.MODIFIED) {
@@ -275,21 +273,7 @@ public abstract class ItemImpl implement
                 }
                 break;
 
-            // Nothing to do for
-            // - Status#EXISTING_MODIFIED : transient modification
-            // - Status#STALE_MODIFIED : external modifications while transient changes pending
-            // - Status#STALE_DESTROYED : external modifications while transient changes pending
-            // - Status#MODIFIED : externally modified -> marker for sessionISM states only
-            // - Status#EXISTING_REMOVED : transient removal
-            case EXISTING_MODIFIED:
-            case STALE_MODIFIED:
-            case STALE_DESTROYED:
-            case MODIFIED:
-            case EXISTING_REMOVED:
-                break;
-
-            // Notify listeners that this item is transiently or permanently
-            // destroyed.
+            // Notify listeners that this item is transiently or permanently destroyed.
             // - Status#REMOVED : permanent removal. item will never get back to life
             case REMOVED:
                 state.removeListener(this);
@@ -301,6 +285,13 @@ public abstract class ItemImpl implement
                 // should never happen.
                 log.error("invalid state change to STATUS_NEW");
                 break;
+
+            // Nothing to do here 
+            case EXISTING_MODIFIED:   // transient modification
+            case STALE_MODIFIED:      // external modifications while transient changes pending
+            case STALE_DESTROYED:     // external modifications while transient changes pending
+            case MODIFIED:            // externally modified -> marker for sessionISM states only
+            case EXISTING_REMOVED:    // transient removal
             case _UNDEFINED_:
             case INVALIDATED:
                 break;