Fixing NPE bug

zhourenjian · zhourenjian · commit b3fee8e84cc2 · 2008-11-29T09:53:13.000Z
diff --git a/sources/net.sf.j2s.ajax/ajaxpipe/net/sf/j2s/ajax/SimplePipeHelper.java b/sources/net.sf.j2s.ajax/ajaxpipe/net/sf/j2s/ajax/SimplePipeHelper.java
@@ -228,8 +228,12 @@ static boolean notifyPipeStatus(String key, boolean live) {
 	 * Wait some more seconds to check whether the pipe is to be closed or not.
 	 * @param runnable
 	 * @return whether the pipe is to be closed or not.
+	 * @j2sIgnore
 	 */
 	static boolean waitAMomentForClosing(final SimplePipeRunnable runnable) {
+		if (runnable == null) {
+			return true;
+		}
 		long extra = runnable.pipeWaitClosingInterval();
 		long interval = runnable.pipeMonitoringInterval();
 		if (interval <= 0) {
diff --git a/sources/net.sf.j2s.ajax/ajaxpipe/net/sf/j2s/ajax/SimplePipeHttpServlet.java b/sources/net.sf.j2s.ajax/ajaxpipe/net/sf/j2s/ajax/SimplePipeHttpServlet.java
@@ -156,7 +156,7 @@ protected void doPipe(final HttpServletResponse resp, String key, String type)
 			}
 			int size = vector.size();
 			if (size > 0) {
-				for (int i = size - 1; i >= 0; i--) {
+				for (int i = 0; i < size; i++) {
 					SimpleSerializable ss = null;
 					/*
 					 * Still need to check vector size!

Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,7 @@ protected void doPipe(final HttpServletResponse resp, String key, String type)`
`156`	`156`	`}`
`157`	`157`	`int size = vector.size();`
`158`	`158`	`if (size > 0) {`
`159`		`- for (int i = size - 1; i >= 0; i--) {`
	`159`	`+ for (int i = 0; i < size; i++) {`
`160`	`160`	`SimpleSerializable ss = null;`
`161`	`161`	`/*`
`162`	`162`	`* Still need to check vector size!`