Summary: | Adding a java breakpoint exception should enable it | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] JDT | Reporter: | Pascal Rapicault <pascal> | ||||||
Component: | Debug | Assignee: | Curtis Windatt <curtis.windatt.public> | ||||||
Status: | VERIFIED FIXED | QA Contact: | |||||||
Severity: | normal | ||||||||
Priority: | P3 | CC: | curtis.windatt.public, Michael_Rennie | ||||||
Version: | 3.3 | Keywords: | bugday, contributed | ||||||
Target Milestone: | 3.6 M5 | Flags: | Michael_Rennie:
review+
|
||||||
Hardware: | PC | ||||||||
OS: | Windows XP | ||||||||
Whiteboard: | |||||||||
Attachments: |
|
Description
Pascal Rapicault
2007-05-28 15:18:56 EDT
Consider post 3.3 Created attachment 153293 [details]
Proposed patch
Fairly simple fix by appending some code to org.eclipse.jdt.internal.debug.ui.breakpoints.AddExceptionAction.java
However, I am a bit unsure about some exception handling (see comment in patched code). Hopefully someone with a little more familiarity can clear this up.
Created attachment 154272 [details]
Updated Fix
There were some issues with your patch:
1. Removed some unecessary logic (the javadoc for setEnabled explicitly states that it does nothing if already enabled)
2. Corrected string NLS, rather than have a constant in the class for strings (which doesn't add any benefit), Eclipse uses a localization class so that different translations can be used. Look at the patch for example usage. If you enter a literal string, it should be marked as a warning with a quick fix to open the externalization wizard that will do the work for you.
3. Handled the core exception the same way as other jobs in the class do. If there is a problem interacting with the breakpoint model the job will fail and display a status dialog to the user.
Will commit for M5. Fixed in HEAD. Mike, please verify. looks fine, I applied a small update to make the breakpoint jobs system + interactive. |