You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by eh...@apache.org on 2007/05/03 18:38:26 UTC

svn commit: r534929 - /incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/RequestCycle.java

Author: ehillenius
Date: Thu May  3 09:38:26 2007
New Revision: 534929

URL: http://svn.apache.org/viewvc?view=rev&rev=534929
Log:
removed stale comments

Modified:
    incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/RequestCycle.java

Modified: incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/RequestCycle.java
URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/RequestCycle.java?view=diff&rev=534929&r1=534928&r2=534929
==============================================================================
--- incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/RequestCycle.java (original)
+++ incubator/wicket/trunk/jdk-1.4/wicket/src/main/java/org/apache/wicket/RequestCycle.java Thu May  3 09:38:26 2007
@@ -1058,26 +1058,11 @@
 					break;
 				}
 				case PROCESS_EVENTS : {
-					// determine what kind of synchronization is to be used, and
-					// handle any events with that and generate a response in
-					// that same block
-					// NOTE: because of synchronization, we need to take the
-					// steps PROCESS_EVENTS and RESPOND together
 					processEventsAndRespond();
 					break;
 				}
 				case RESPOND : {
 					// generate a response
-					// NOTE: We have to do sync here because the
-					// processEventsAndRespond step will be unrolled by a
-					// RestartXX Exception. And if this is not the case
-					// then still it is not a problem to have 2 locks on
-					// the same session this will not cause a deadlock.
-					// So only if the request targets are different and the
-					// getLock() doesn't return 2 times the same object a
-					// deadlock could maybe occur. But we use the session
-					// as the lock at all times maybe use a variable inside
-					// RequestCycle to know that a lock is still in place?
 					respond();
 					break;
 				}