From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 5170 invoked by alias); 15 Dec 2007 01:08:32 -0000 Received: (qmail 5130 invoked by uid 367); 15 Dec 2007 01:08:28 -0000 Date: Sat, 15 Dec 2007 01:08:00 -0000 Message-ID: <20071215010828.5115.qmail@sourceware.org> From: cagney@sourceware.org To: frysk-cvs@sourceware.org Subject: [SCM] master: expunit.EofException -> EndOfFileException; expand exception messages. X-Git-Refname: refs/heads/master X-Git-Reftype: branch X-Git-Oldrev: 9bbfe5c000a1fc3116d8eb919d1ebd061ebbf1f4 X-Git-Newrev: 523f56213ae2f667cce2af311f7a837c2e22188b Mailing-List: contact frysk-cvs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: frysk-cvs-owner@sourceware.org Reply-To: frysk@sourceware.org X-SW-Source: 2007-q4/txt/msg00596.txt.bz2 The branch, master has been updated via 523f56213ae2f667cce2af311f7a837c2e22188b (commit) from 9bbfe5c000a1fc3116d8eb919d1ebd061ebbf1f4 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email. - Log ----------------------------------------------------------------- commit 523f56213ae2f667cce2af311f7a837c2e22188b Author: Andrew Cagney Date: Fri Dec 14 20:07:35 2007 -0500 expunit.EofException -> EndOfFileException; expand exception messages. frysk-core/frysk/hpd/ChangeLog 2007-12-14 cagney * HpdTestbed.java: Update; EofException renamed to EndOfFileException. * TestPreprocessor.java: Ditto. frysk-sys/frysk/expunit/ChangeLog 2007-12-14 cagney * EndOfFileException.java: Rename EofException.java. (EndOfFileException(Match[], String)): Replace EndOfFileException(). * TimeoutException.java (TimeoutException(long, Match[], String)): Replace TimeoutException(long). * Expect.java: Update. ----------------------------------------------------------------------- Summary of changes: frysk-core/frysk/hpd/ChangeLog | 5 ++ frysk-core/frysk/hpd/HpdTestbed.java | 8 ++-- frysk-core/frysk/hpd/TestPreprocessor.java | 4 +- frysk-sys/frysk/expunit/ChangeLog | 8 +++ ...ationException.java => EndOfFileException.java} | 31 +++++++---- frysk-sys/frysk/expunit/EofException.java | 54 -------------------- frysk-sys/frysk/expunit/Expect.java | 9 ++-- frysk-sys/frysk/expunit/TimeoutException.java | 22 +++++++- 8 files changed, 63 insertions(+), 78 deletions(-) copy frysk-sys/frysk/expunit/{TerminationException.java => EndOfFileException.java} (79%) delete mode 100644 frysk-sys/frysk/expunit/EofException.java First 500 lines of diff: diff --git a/frysk-core/frysk/hpd/ChangeLog b/frysk-core/frysk/hpd/ChangeLog index cae4d71..c8afbd5 100644 --- a/frysk-core/frysk/hpd/ChangeLog +++ b/frysk-core/frysk/hpd/ChangeLog @@ -1,3 +1,8 @@ +2007-12-14 cagney + + * HpdTestbed.java: Update; EofException renamed to EndOfFileException. + * TestPreprocessor.java: Ditto. + 2007-12-13 Sami Wagiaalla Moved frysk.rt.Line to frysk.scopes.SourceLocation. diff --git a/frysk-core/frysk/hpd/HpdTestbed.java b/frysk-core/frysk/hpd/HpdTestbed.java index 9914ee5..2cec313 100644 --- a/frysk-core/frysk/hpd/HpdTestbed.java +++ b/frysk-core/frysk/hpd/HpdTestbed.java @@ -45,7 +45,7 @@ import frysk.testbed.CoreFileAtSignal; import frysk.expunit.Expect; import frysk.expunit.Match; import frysk.expunit.Regex; -import frysk.expunit.EofException; +import frysk.expunit.EndOfFileException; import frysk.expunit.TimeoutException; import java.io.File; import frysk.testbed.TearDownExpect; @@ -105,7 +105,7 @@ public class HpdTestbed } } }); - } catch (EofException e) { + } catch (EndOfFileException e) { TestCase.fail(why + " got: "); } catch (TimeoutException t) { TestCase.fail(why + " got: "); @@ -119,7 +119,7 @@ public class HpdTestbed public HpdTestbed expectPrompt() { try { expect(prompt); - } catch (EofException e) { + } catch (EndOfFileException e) { TestCase.fail("expecting: <" + prompt + "> got: EOF"); } catch (TimeoutException t) { TestCase.fail("expecting: <" + prompt + "> got: TIMEOUT"); @@ -197,7 +197,7 @@ public class HpdTestbed } } }); - } catch (EofException e) { + } catch (EndOfFileException e) { TestCase.fail("Expecting got: "); } catch (TimeoutException t) { TestCase.fail("Expecting got: "); diff --git a/frysk-core/frysk/hpd/TestPreprocessor.java b/frysk-core/frysk/hpd/TestPreprocessor.java index 84e4ffd..000d643 100644 --- a/frysk-core/frysk/hpd/TestPreprocessor.java +++ b/frysk-core/frysk/hpd/TestPreprocessor.java @@ -40,7 +40,7 @@ package frysk.hpd; import frysk.expunit.Regex; -import frysk.expunit.EofException; +import frysk.expunit.EndOfFileException; /** * Test the pre-processor which unpacks stuff like @@ -68,7 +68,7 @@ public class TestPreprocessor extends TestLib { fail("Unexpected input: <<" + group() + ">>"); } }); - } catch (EofException e) { + } catch (EndOfFileException e) { eof = true; } assertTrue("eof", eof); diff --git a/frysk-sys/frysk/expunit/ChangeLog b/frysk-sys/frysk/expunit/ChangeLog index 16d4b15..1173518 100644 --- a/frysk-sys/frysk/expunit/ChangeLog +++ b/frysk-sys/frysk/expunit/ChangeLog @@ -1,3 +1,11 @@ +2007-12-14 cagney + + * EndOfFileException.java: Rename EofException.java. + (EndOfFileException(Match[], String)): Replace EndOfFileException(). + * TimeoutException.java (TimeoutException(long, Match[], String)): Replace TimeoutException(long). + * Expect.java: Update. + + 2007-12-04 Andrew Cagney Merged frysk.sys.Sig into frysk.sys.Signal. diff --git a/frysk-sys/frysk/expunit/TerminationException.java b/frysk-sys/frysk/expunit/EndOfFileException.java similarity index 79% copy from frysk-sys/frysk/expunit/TerminationException.java copy to frysk-sys/frysk/expunit/EndOfFileException.java index 7c69c9f..e815132 100644 --- a/frysk-sys/frysk/expunit/TerminationException.java +++ b/frysk-sys/frysk/expunit/EndOfFileException.java @@ -43,19 +43,26 @@ package frysk.expunit; * Thrown an an end-of-file is encountered and nothing else matches. */ -public class TerminationException - extends RuntimeException -{ +public class EndOfFileException extends RuntimeException { static final long serialVersionUID = 1; - static private String expecting (int expectedStatus) - { - if (expectedStatus >= 0) - return " (expecting exit with status " + expectedStatus + ")"; - else - return " (expecting kill with signal " + expectedStatus + ")"; + + private static String message(Match[] matches, String output) { + StringBuffer msg = new StringBuffer(); + msg.append("End-of-file"); + if (matches != null) { + msg.append("; expecting: "); + for (int i = 0; i < matches.length; i++) { + msg.append(" <<"); + msg.append(matches.toString()); + msg.append(">>"); + } + } + msg.append("; buffer <<"); + msg.append(output); + msg.append(">>"); + return msg.toString(); } - TerminationException (int expectedStatus, String s) - { - super (s + expecting (expectedStatus)); + EndOfFileException(Match[] matches, String output) { + super(message(matches, output)); } } diff --git a/frysk-sys/frysk/expunit/EofException.java b/frysk-sys/frysk/expunit/EofException.java deleted file mode 100644 index 19ec3fd..0000000 --- a/frysk-sys/frysk/expunit/EofException.java +++ /dev/null @@ -1,54 +0,0 @@ -// This file is part of the program FRYSK. -// -// Copyright 2007, Red Hat Inc. -// -// FRYSK is free software; you can redistribute it and/or modify it -// under the terms of the GNU General Public License as published by -// the Free Software Foundation; version 2 of the License. -// -// FRYSK is distributed in the hope that it will be useful, but -// WITHOUT ANY WARRANTY; without even the implied warranty of -// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU -// General Public License for more details. -// -// You should have received a copy of the GNU General Public License -// along with FRYSK; if not, write to the Free Software Foundation, -// Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA. -// -// In addition, as a special exception, Red Hat, Inc. gives You the -// additional right to link the code of FRYSK with code not covered -// under the GNU General Public License ("Non-GPL Code") and to -// distribute linked combinations including the two, subject to the -// limitations in this paragraph. Non-GPL Code permitted under this -// exception must only link to the code of FRYSK through those well -// defined interfaces identified in the file named EXCEPTION found in -// the source code files (the "Approved Interfaces"). The files of -// Non-GPL Code may instantiate templates or use macros or inline -// functions from the Approved Interfaces without causing the -// resulting work to be covered by the GNU General Public -// License. Only Red Hat, Inc. may make changes or additions to the -// list of Approved Interfaces. You must obey the GNU General Public -// License in all respects for all of the FRYSK code and other code -// used in conjunction with FRYSK except the Non-GPL Code covered by -// this exception. If you modify this file, you may extend this -// exception to your version of the file, but you are not obligated to -// do so. If you do not wish to provide this exception without -// modification, you must delete this exception statement from your -// version and license this file solely under the GPL without -// exception. - -package frysk.expunit; - -/** - * Thrown an an end-of-file is encountered and nothing else matches. - */ - -public class EofException - extends RuntimeException -{ - static final long serialVersionUID = 1; - EofException () - { - super ("End-of-file encountered"); - } -} diff --git a/frysk-sys/frysk/expunit/Expect.java b/frysk-sys/frysk/expunit/Expect.java index 83eecd0..c39918f 100644 --- a/frysk-sys/frysk/expunit/Expect.java +++ b/frysk-sys/frysk/expunit/Expect.java @@ -232,12 +232,13 @@ public class Expect } if (eof) { logger.log (Level.FINE, "{0} match EOF\n", this); - throw new EofException (); + throw new EndOfFileException(matches, output); } long timeRemaining = endTime - System.currentTimeMillis (); if (timeRemaining <= 0) { logger.log (Level.FINE, "{0} match TIMEOUT\n", this); - throw new TimeoutException (timeoutMilliseconds / 1000); + throw new TimeoutException(timeoutMilliseconds / 1000, + matches, output); } logger.log (Level.FINE, @@ -351,7 +352,7 @@ public class Expect try { expect (); } - catch (EofException e) { + catch (EndOfFileException e) { // Just what the doctor ordered. } } @@ -366,7 +367,7 @@ public class Expect try { expect (); } - catch (EofException e) { + catch (EndOfFileException e) { // This is blocking; which probably isn't good. pid.blockingWait (new WaitObserver (status)); } diff --git a/frysk-sys/frysk/expunit/TimeoutException.java b/frysk-sys/frysk/expunit/TimeoutException.java index c916760..4f64398 100644 --- a/frysk-sys/frysk/expunit/TimeoutException.java +++ b/frysk-sys/frysk/expunit/TimeoutException.java @@ -48,8 +48,26 @@ public class TimeoutException extends RuntimeException { static final long serialVersionUID = 1; - TimeoutException (long millisecondTimeout) + private static String message(long millisecondTimeout, Match[] matches, String output) { + StringBuffer msg = new StringBuffer(); + msg.append("Timeout of " + millisecondTimeout + " expired"); + if (matches != null) { + msg.append("; expecting: "); + for (int i = 0; i < matches.length; i++) { + msg.append(" <<"); + msg.append(matches.toString()); + msg.append(">>"); + } + } + msg.append("; buffer <<"); + msg.append(output); + msg.append(">>"); + return msg.toString(); + } + + TimeoutException (long millisecondTimeout, Match[] matches, String output) { - super ("Timeout of " + millisecondTimeout + " expired"); + super (message(millisecondTimeout, matches, output)); + } } hooks/post-receive -- frysk system monitor/debugger