public inbox for mauve-patches@sourceware.org
 help / color / mirror / Atom feed
* Harness fixlet
@ 2006-07-05 17:46 Anthony Balkissoon
  0 siblings, 0 replies; only message in thread
From: Anthony Balkissoon @ 2006-07-05 17:46 UTC (permalink / raw)
  To: mauve-patches

[-- Attachment #1: Type: text/plain, Size: 280 bytes --]

Small fix for yesterday's ErrorStreamPrinter commit.  If a test times
out the ESP needs to be restarted.

2006-07-05  Anthony Balkissoon  <abalkiss@redhat.com>

	* Harness.java:
	(ErrorStreamPrinter.run): If the readLine() returns null restart the 
	error stream printer.

--Tony

[-- Attachment #2: ErrorStreamWriterFix.diff --]
[-- Type: text/x-patch, Size: 821 bytes --]

Index: Harness.java
===================================================================
RCS file: /cvs/mauve/mauve/Harness.java,v
retrieving revision 1.18
diff -u -r1.18 Harness.java
--- Harness.java	4 Jul 2006 20:06:54 -0000	1.18
+++ Harness.java	5 Jul 2006 17:44:10 -0000
@@ -1118,7 +1118,17 @@
       try
       {
         while (true)
-          System.err.println(in.readLine());
+          {
+            String temp = in.readLine();
+            if (temp == null)
+              {
+                // This means the RunnerProcess was restarted (because of a 
+                // timeout) and we need to restart the error stream writer.
+                restartESP = true;
+                break;
+              }
+            System.err.println(temp);
+          }
       }
       catch (IOException ioe)
       {

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2006-07-05 17:46 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-07-05 17:46 Harness fixlet Anthony Balkissoon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).