Re: [cdt-dev] event breakpoints

[Elena Laskavaia <ELaskavaia@xxxxxxx>]

I guess I updated too quick, I did it again and it works.

John Cortell wrote:
>
> Team synchronization shows no deltas. I ran it twice just to be sure.
> Let me investigate...
>
> John
>
> At 04:41 PM 4/25/2008, Elena Laskavaia wrote:
> >Code is not compilable now. Did you commit everything? I don't have
> >ICDIEventBreakpoint I still have ICDICatchpoint there
> >
> >John Cortell wrote:
> > >
> > > I've committed a refactoring of the code, as per my intentions
> > > announced on the TWIKI in the thread on this matter
> > >
> > > http://wiki.eclipse.org/Talk:CDT:_Debug:_Catchpoints_support
> > >
> > > All method names, types, variables, etc now refer to
> > > "EventBreakpoint" instead of "Catchpoint". I validated my work by
> > > running the JUnit tests and with manual testing as well.
> > >
> > > John
> > >
> > > At 12:54 PM 4/25/2008, Elena Laskavaia wrote:
> > > >Since no one replied and I don't want to wait anymore I commit the
> > > >changes, please take a look.
> > > >There is Add Event Breakpoint (C/C++) menu action in breakpoints 
> toolbar
> > > >menu. If you don't have mi it will show error message, if you 
> have mi
> > > >plugins it will show dialog with type selection.
> > > >
> > > >TODO:
> > > >- Icons - I draw initial one but I am not a graphic designer
> > > >- Figure out if there is any way to disable view action without 
> loading
> > > >plugin if it is not applicable
> > > >- Gdb/Mi - look at 6.8 to see if it supports more of advertised 
> event
> > > >breakpoints and fix mi engine
> > > >- Breakpoint error notification - currently there is no way to show
> > > >"error" marker on breakpoint to see why it failed to set
> > > >- Find a proper location for action or at least similar location 
> as Add
> > > >Watchpoint
> > > >
> > > >Elena Laskavaia wrote:
> > > > >
> > > > > I am ready to commit event breakpoint support.
> > > > > I made an MI implementation, but it works only for catch and 
> throw
> > > > > without arguments in my gdb version.
> > > > > The only open question left is where to put action in UI, we
> > > likely to
> > > > > move it next release if we get bug 227394 approved.
> > > > > The reasonable places:
> > > > > a) add to context menu where the current Add Watchpoint is or
> > > > > b) add to View menu, and move Add Watchpoint there too, 
> because as
> > > > > common agreement it should not be in context menu
> > > > >
> > > > >
> > > > > _______________________________________________
> > > > > cdt-dev mailing list
> > > > > cdt-dev@xxxxxxxxxxx
> > > > > https://dev.eclipse.org/mailman/listinfo/cdt-dev
> > > > >
> > > >_______________________________________________
> > > >cdt-dev mailing list
> > > >cdt-dev@xxxxxxxxxxx
> > > >https://dev.eclipse.org/mailman/listinfo/cdt-dev
> > >
> > > _______________________________________________
> > > cdt-dev mailing list
> > > cdt-dev@xxxxxxxxxxx
> > > https://dev.eclipse.org/mailman/listinfo/cdt-dev
> > >
> >_______________________________________________
> >cdt-dev mailing list
> >cdt-dev@xxxxxxxxxxx
> >https://dev.eclipse.org/mailman/listinfo/cdt-dev
>
> _______________________________________________
> cdt-dev mailing list
> cdt-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/cdt-dev
>