Bug 1891 - Pass the main task, and not the process, as the forked observer param
Summary: Pass the main task, and not the process, as the forked observer param
Status: RESOLVED FIXED
Alias: None
Product: frysk
Classification: Unclassified
Component: general (show other bugs)
Version: unspecified
: P1 normal
Target Milestone: ---
Assignee: Andrew Cagney
URL:
Keywords:
Depends on:
Blocks: 1516
  Show dependency treegraph
 
Reported: 2005-11-20 18:21 UTC by Andrew Cagney
Modified: 2005-11-20 20:15 UTC (History)
0 users

See Also:
Host:
Target:
Build:
Last reconfirmed:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Cagney 2005-11-20 18:21:00 UTC
All the code I've looked at ends up calling Proc.getTasks().getFirst() to pull
out the main task.  Might as well give clients want they want.
Comment 1 Andrew Cagney 2005-11-20 19:35:48 UTC
Index: frysk-core/frysk/proc/ChangeLog
2005-11-20  Andrew Cagney  <cagney@redhat.com>

        * LinuxHost.java: Pass the forked process's only task to Task
        .performeForked.
        * TaskObserver.java: Change .Forked.updateForked fork parameter to
        a Task.
        * TaskState.java: Update.
        * Task.java: Update.

Index: frysk-gui/frysk/gui/monitor/ChangeLog
2005-11-20  Andrew Cagney  <cagney@redhat.com>

        * observers/TaskForkedObserver.java: Update,
        frysk.proc.TaskObserver .Forked .updateForked passes in the
        processes Task.
        * EventLogger.java: Ditto.
Comment 2 Andrew Cagney 2005-11-20 20:15:07 UTC
Resultant cleanup

Index: frysk-core/frysk/proc/ChangeLog
2005-11-20  Andrew Cagney  <cagney@redhat.com>

        * Task.java: Refactor single constructor into separate unattached,
        attached clone, and attached main constructors.
        * Proc.java: Re-organize constructor code.  Delete
        sendNewAttachedTask.
        * LinuxTask.java: Ditto.
        * LinuxProc.java: Update.
        * LinuxHost.java: Update.  Explicitly create an attached processes
        first task.