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

Bug 306427

Summary: Remove obsolete catchpoint types from CDT.
Product: [Tools] CDT Reporter: John Cortell <john.cortell>
Component: cdt-debugAssignee: John Cortell <john.cortell>
Status: RESOLVED FIXED QA Contact: Ken Ryall <ken.ryall>
Severity: normal    
Priority: P3 CC: elaskavaia.cdt, pawel.1.piech
Version: 6.0   
Target Milestone: 7.0   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Fix john.cortell: iplog-

Description John Cortell CLA 2010-03-18 15:14:31 EDT 
The event breakpoint feature was based on the gdb catchpoints listed in 'help catch' in gdb 6.8. Many of them were unimplemented in gdb and were later removed in 7.0. CDT still makes those available. We should remove them. The final list should be whatever 'help catch' says in gdb 7.0.

Note that event breakpoints are a general concept and apply equally to non-gdb debuggers. Non-gdb solutions (e.g., EDC on Windows) can contribute additional event types via an extension, so this move would not limit what EDC or other vendors can support.
Comment 1 John Cortell CLA 2010-04-06 19:14:04 EDT 
Specifically, the list of catchpoints supported by stock CDT will be reduced to:

    * catch
    * exec
    * fork
    * syscall [name/number]
    * throw
    * vfork

as per the list provided by gdb 7.0's 'help catch'. There are two Ada related events that we won't support in CDT for obvious reasons (assert and exception).

Additionally, 'help catch throw' and 'help catch catch' indicate an exception name can be specified as an argument, but MinGW's gdb not only doesn't support such an argument, but specifying it causes the catchpoint command to fail; so, we'll leave the current support in CDT as is in this respect (don't allow the user to specify an exception name)
Comment 2 John Cortell CLA 2010-04-12 15:01:04 EDT 
Created attachment 164616 [details]
Fix
Comment 3 John Cortell CLA 2010-04-12 15:01:31 EDT 
Committed to HEAD.
Comment 4 John Cortell CLA 2010-06-17 12:49:26 EDT 
Note that this change was motivated by input from a gdb insider. See http://dev.eclipse.org/mhonarc/lists/cdt-dev/msg17835.html