Bug 343039 - Turn on monitors by default
Summary: Turn on monitors by default
Status: RESOLVED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Debug (show other bugs)
Version: 3.6.1   Edit
Hardware: All All
: P3 enhancement with 2 votes (vote)
Target Milestone: 3.8 M6   Edit
Assignee: Michael Rennie CLA
QA Contact:
URL:
Whiteboard:
Keywords: noteworthy
Depends on:
Blocks:
 
Reported: 2011-04-16 10:39 EDT by Jeff McAffer CLA
Modified: 2012-02-29 12:50 EST (History)
6 users (show)

See Also:

Attachments
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 Jeff McAffer CLA 2011-04-16 10:39:25 EDT 
The JDT debugger has the ability to show monitors owned by a thread.  this is turned off by default.  Dunno how long this option has been here but many people (including me) never knew it was there and would have greatly benefitted from its use in time-consuming race and deadlock debugging sessions.

Please consider turning it on by default.
Comment 1 Nobody - feel free to take it CLA 2011-04-18 04:12:31 EDT 
I agree this would be a better default.
Comment 2 Michael Rennie CLA 2011-04-18 09:30:15 EDT 
I also agree.

We'll change it at the start of 3.8, I don't think its a good idea to change a default setting at the end of a release.
Comment 3 Dani Megert CLA 2011-04-18 09:40:47 EDT 
(In reply to comment #2)
> I also agree.
> 
> We'll change it at the start of 3.8, I don't think its a good idea to change a
> default setting at the end of a release.

+1
Comment 4 Jeff McAffer CLA 2011-04-18 10:07:56 EDT 
Not sure why this is bad to change now.  consider it a polish item.  Presumably the function is stable as it is in the release so the risk is minimal.  What is going to change between now and 3.8 wrt this function?
Comment 5 Dani Megert CLA 2011-04-18 10:18:07 EDT 
(In reply to comment #4)
> Not sure why this is bad to change now.  consider it a polish item.  Presumably
> the function is stable as it is in the release so the risk is minimal.  What is
> going to change between now and 3.8 wrt this function?

We want to avoid prominent changes to the UI so late in the game (M7/RC0 is next week). Also note that this is in the 'Tips and Tricks' (maybe time to read it [again]? ;-).
Comment 6 Nobody - feel free to take it CLA 2011-04-18 11:45:18 EDT 
3.8 suits me, thanks