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;
}