Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 352230 - Proposed patch to add daemon flag to Thread Overview and Details
Summary: Proposed patch to add daemon flag to Thread Overview and Details
Status: RESOLVED DUPLICATE of bug 353517
Alias: None
Product: MAT
Classification: Tools
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Krum Tsvetkov CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-15 12:06 EDT by Kevin Grigorenko CLA
Modified: 2011-10-21 05:45 EDT (History)
1 user (show)

See Also:


Attachments
Patch to add daemon column (4.33 KB, patch)
2011-07-15 12:08 EDT, Kevin Grigorenko CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kevin Grigorenko CLA 2011-07-15 12:06:13 EDT
Attached is a proposed patch to add an "Is Daemon?" column to Java Basics > Thread Overview and Thread Details. It has been tested successfully on both HPROF and IBM System Dumps.

This is my first fully-formed patch -- is a bug report the best way to send in a patch or through the mat-dev mailing list as an attachment?
Comment 1 Kevin Grigorenko CLA 2011-07-15 12:08:06 EDT
Created attachment 199756 [details]
Patch to add daemon column
Comment 2 Krum Tsvetkov CLA 2011-07-18 02:08:48 EDT
thanks for the patch!
I find your proposal good. I'll give the patch a try and keep you updated if/when it is submitted.

>> is a bug report the best way to send in a patch ...
Yes. Please use Bugzilla for sending us patches if you have some further ideas.

Krum
Comment 3 Kevin Grigorenko CLA 2011-08-01 13:38:33 EDT
Superseded by Bug 353517
Comment 4 Andrew Johnson CLA 2011-10-21 05:45:11 EDT
fixed by 353517

*** This bug has been marked as a duplicate of bug 353517 ***