public inbox for frysk@sourceware.org
 help / color / mirror / Atom feed
* [patch] Enable tests for resolved bug #3737
@ 2008-03-13 22:27 Mark Wielaard
  0 siblings, 0 replies; only message in thread
From: Mark Wielaard @ 2008-03-13 22:27 UTC (permalink / raw)
  To: frysk

Hi,

Found two old (duplicate) bugs #3737 and #3936 which were resolved some
time ago by:

2007-04-26  Mark Wielaard  <mark@klomp.org>

        * frysk-core/frysk/proc/LinuxPtraceTaskState.java (detaching):
        Add handleDisappearedEvent.

They even had testcases. But those weren't enabled yet. This enables
them:

frysk-core/frysk/proc/ChangeLog
2008-03-13  Mark Wielaard  <mwielaard@redhat.com>

       * TestTaskObserver.java (testDetachExitingMainTask): Enabled
       already fixed test for bug #3737.
       (detachExitingTask): Likewise.

Passes on both x86 and x86_64 fedora 8.

Committed and pushed,

Mark

diff --git a/frysk-core/frysk/proc/TestTaskObserver.java b/frysk-core/frysk/proc
index 09e7b51..051e752 100644
--- a/frysk-core/frysk/proc/TestTaskObserver.java
+++ b/frysk-core/frysk/proc/TestTaskObserver.java
@@ -256,15 +256,11 @@ public class TestTaskObserver
     /** {@link #detachExitingTask} */
     public void testDetachExitingMainTask ()
     {
-       if (unresolvedOnUtrace(3737))
-           return;
        detachExitingTask (0, true);
     }
     /** {@link #detachExitingTask} */
     public void testDetachExitingOtherTask ()
     {
-       if (unresolvedOnUtrace(3737))
-           return;
        detachExitingTask (1, false);
     }


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

only message in thread, other threads:[~2008-03-13 22:27 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-03-13 22:27 [patch] Enable tests for resolved bug #3737 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).