Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 352230

Summary: Proposed patch to add daemon flag to Thread Overview and Details
Product: [Tools] MAT Reporter: Kevin Grigorenko <kevin.grigorenko>
Component: CoreAssignee: Krum Tsvetkov <krum.tsvetkov>
Status: RESOLVED DUPLICATE QA Contact:
Severity: enhancement    
Priority: P3 CC: krum.tsvetkov
Version: unspecified   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Patch to add daemon column none

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 ***