* [patch] Make TestDebugInfoStackTrace.frameAssertions() more robust
@ 2008-03-19 14:53 Mark Wielaard
0 siblings, 0 replies; only message in thread
From: Mark Wielaard @ 2008-03-19 14:53 UTC (permalink / raw)
To: frysk
Hi,
While working on more unwind issues I noticed that
TestDebugInfoStackTrace.frameAssertions() didn't check for the actual
kill() and syscall() functions (only the internal library names). With
my new work the actual function names show up sometimes (not committed
yet), so these also need to be matched.
frysk-core/frysk/debuginfo/ChangeLog
2008-03-19 Mark Wielaard <mwielaard@redhat.com>
* TestDebugInfoStackTrace (frameAssertions): Also check for function
names starting with kill or syscall.
Committed and pushed,
Mark
index 1939bb9..e932858 100644
--- a/frysk-core/frysk/debuginfo/TestDebugInfoStackTrace.java
+++ b/frysk-core/frysk/debuginfo/TestDebugInfoStackTrace.java
@@ -407,8 +407,8 @@ public class TestDebugInfoStackTrace
String frameName = this.frameTracker[i][index][2];
while (frameName == null
- || frameName.indexOf("kill") > 0
- || frameName.indexOf("syscall") > 0)
+ || frameName.indexOf("kill") >= 0
+ || frameName.indexOf("syscall") >= 0)
{
index++;
frameName = this.frameTracker[i][index][2];
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2008-03-19 14:53 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-03-19 14:53 [patch] Make TestDebugInfoStackTrace.frameAssertions() more robust Mark Wielaard
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).