[cdt-dev] RE: Knowing we are a CDT launch

[Marc Khouzam <marc.khouzam@xxxxxxxxxxxx>]

Hi,

just a note that I did _not_ add a new ICLaunch interface.
Toni had the good idea of using a launch attribute instead
and it worked great.

Marc


> -----Original Message-----
> From: cdt-dev-bounces@xxxxxxxxxxx 
> [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Marc Khouzam
> Sent: April-18-10 10:43 AM
> To: CDT DEV (cdt-dev@xxxxxxxxxxx)
> Subject: [cdt-dev] Knowing we are a CDT launch
> 
> Hi,
> 
> as part of 
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=248627 Add an 
> option to show full paths for the stack frame source location
> 
> I need to be able to know if the 'show full path' option 
> should be enabled.  This is a global option, which should be enabled
> as soon as we have a launch that supports it.
> The way it is handled now, is to see if the debug view 
> contains a debug target that is an ICDebugElement.
> The problem is that for DSF, we don't use ICDebugElement, so 
> this option is always disabled for DSF.
> 
> My thought was that we need a way to see if there is a CDT 
> launch currently active.
> To do that we would need to add a new empty interface, 
> ICLaunch, which CDT launches would implement (CDI and DSF).
> 
> The other option is to add a more specific interface, 
> IShowFullPathTarget, which CDT launches that want to support
> that option, would implement.  This solution is more 
> future-proof, because, if we have other options that need to 
> be handled in
> the same way, it would allow each debugger integration to 
> pick and choose which options to support.  For example, CDI
> could implement IShowFullPath and IMyOtherOption, while DSF 
> would only implement IShowFullPath, if there was no
> support for the MyOtherOption.  Having a single ICLaunch 
> interface is simpler but less flexible.
> 
> I like the second option better, but I'm worried I may be 
> overly specific, which could lead to a bunch of interfaces.
> 
> Anyone have an opinion?
> And does anyone object to one or both of these new interfaces?
> 
> Thanks
> 
> Marc
> _______________________________________________
> cdt-dev mailing list
> cdt-dev@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/cdt-dev
>