Re: [cdt-dev] [DSF] SessionType

[Vladimir Prus <vladimir@xxxxxxxxxxxxxxxx>]

On Thursday 08 July 2010 19:48:59 Marc Khouzam wrote:

> > -----Original Message-----
> > From: cdt-dev-bounces@xxxxxxxxxxx 
> > [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Vladimir Prus
> > Sent: Thursday, July 08, 2010 6:17 AM
> > To: cdt-dev@xxxxxxxxxxx
> > Subject: [cdt-dev] [DSF] SessionType
> > 
> > 
> > Hi,
> > I'm looking at this definition from DSF:
> > 
> > 	public enum SessionType { LOCAL, REMOTE, CORE }
> 
> This is DSF-GDB specific.  (just to be fair to DSF :-))
> 
> > Unfortunately, despite quite some years of experience with gdb, I have
> > no idea what LOCAL and REMOTE means. 
> 
> REMOTE is when we connect to a gdbserver.
> LOCAL is when we use GDB on the host only.

And what if we connect to something that speaks gdb protocol, but is not
a gdbserver?

> > Would it not be better to remove session type completely, and use
> > more detailed switches, like 'should run or continue to be used
> > when starting program', or 'run can be used to restart'.
> 
> If the problem is using -exec-run or -exec-continue, then the
> service which dispatches those commands can be overridden to handle
> the cases you want to deal with.

I am becoming somewhat concerned :-( You seem to suggest that
overridding a service -- that is, writing my own service class
and doing what I want -- is a sensible approach. However,
suppose I want to go this route. I'll derive from GDBControl_7_0
and what is the next step? startOrRestart method is pretty big,
and all I want to do is modify a single if condition inside it.
Clearly, copy-pasting it is not a good approach and there
are no methods that encapsulate 'should I use run' decision.
Are you proposing to add 'useRun' method to GDBControl_7_0, or
there's some other approach?

Thanks,

-- 
Vladimir Prus
CodeSourcery
vladimir@xxxxxxxxxxxxxxxx
(650) 331-3385 x722