Bug 4243 - Removed Porcs/Threads are not handled
Summary: Removed Porcs/Threads are not handled
Status: RESOLVED FIXED
Alias: None
Product: frysk
Classification: Unclassified
Component: general (show other bugs)
Version: unspecified
: P2 normal
Target Milestone: ---
Assignee: Unassigned
URL:
Keywords:
: 4259 (view as bug list)
Depends on:
Blocks: 1632
  Show dependency tree / graph
 
Reported: 2007-03-21 20:30 UTC by Sami Wagiaalla
Modified: 2007-03-29 16:54 UTC (History)
2 users (show)

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

Attachments
Frysk Monitor: Removing terminated threads (27.22 KB, image/png)
2007-03-23 13:58 UTC, Sami Wagiaalla 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sami Wagiaalla 2007-03-21 20:30:58 UTC 
the new event viewer does not handle dead procs/threads.

One way to do so is display a darker display and provide a button to remove it
from the view.
Comment 1 Sami Wagiaalla 2007-03-22 15:14:43 UTC 
Where do people think the button should go ?
Options:

* to the left the the time line underneath the label.
* in a tool bar.
* to the right of the time line which will
  cause the timelines to shrink to the right
  a little to provide space. We could also have
  the buttons present at all times to remove dead
  threads and uninteresting procs ?

Thoughts ?
Comment 2 Sami Wagiaalla 2007-03-22 15:24:30 UTC 
*** Bug 4259 has been marked as a duplicate of this bug. ***
Comment 3 Sami Wagiaalla 2007-03-23 13:58:23 UTC 
Created attachment 1646 [details]
Frysk Monitor: Removing terminated threads 

What do people think of the button on the right hand side ?
Comment 4 Sami Wagiaalla 2007-03-26 17:13:12 UTC 
Add a way to indicate the life time of a thread/proc:
suggestions:
   - draw the life time area brighter than the dead area.
   - cut off the line in the button with a couple black ticks
* see what project management software does to represent a lifetime of a task
(gant).

2007-03-26  Sami Wagiaalla  <swagiaal@redhat.com>

        * test/TestPrototypeCopying.java: Added test for copying of filter
        boolean value.
        * test/TestObserverSaveLoad.java: Added test for filter boolean value.
        * gladedir/procpop.glade: Added "Observer Discription" feild to Edit
        Observer Dialog.

2007-03-26  Sami Wagiaalla  <swagiaal@redhat.com>

        * eventviewer/TimeLineStartEvent.java: New file.
        * eventviewer/TimeLineEndEvent.java: New file.
        * eventviewer/TimeLine.java: Added button for removing timelines.
        * eventviewer/ProcTimeLine.java (getGuiProc): made it public.
        * eventviewer/ProcBox.java: Added handling for removing dead threads.
Fix for
        bz 4243.
        * eventviewer/EventViewer2.java: Added and expose listiner to handle
redrawing
        fix for bz# 4269.
        Added handling for removing dead processes. Fix for bz 4243.
        * ObserversDialog.java: Added 'Observer Description' field, and handling.
        * EditObserverDialog.java: Renamed observerDescriptionTextView to 
        editObserverDialogObserverDescriptionTextView.
Comment 5 Sami Wagiaalla 2007-03-29 17:54:03 UTC 
Fixed on 2007-03-26 17:13
Enhancements requested moved to:
  http://sourceware.org/bugzilla/show_bug.cgi?id=4296