* Exception line number fix
@ 2006-07-26 21:39 Anthony Balkissoon
0 siblings, 0 replies; only message in thread
From: Anthony Balkissoon @ 2006-07-26 21:39 UTC (permalink / raw)
To: mauve-patches
[-- Attachment #1: Type: text/plain, Size: 324 bytes --]
Roman pointed out that the line number we were returning as the FAIL
line number didn't always make sense. This patch fixes the problem.
2006-07-26 Anthony Balkissoon <abalkiss@redhat.com>
* RunnerProcess.java:
(exceptionDetails): Line number doesn't need to come from the test()
method, removed that "if".
--Tony
[-- Attachment #2: ExceptionLineNumber.diff --]
[-- Type: text/x-patch, Size: 626 bytes --]
Index: RunnerProcess.java
===================================================================
RCS file: /cvs/mauve/mauve/RunnerProcess.java,v
retrieving revision 1.12
diff -u -r1.12 RunnerProcess.java
--- RunnerProcess.java 25 Jul 2006 23:11:16 -0000 1.12
+++ RunnerProcess.java 26 Jul 2006 21:37:22 -0000
@@ -821,8 +821,7 @@
// exception.
for (int i = 0; i < st.length; i++)
{
- if (st[i].getClassName().equals(name)
- && st[i].getMethodName().equals("test"))
+ if (st[i].getClassName().equals(name))
{
lineOrigin = st[i].getLineNumber();
break;
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2006-07-26 21:39 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-07-26 21:39 Exception line number fix 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).