[cdt-patch] Re: Proposed CDT 1.2 patches

[Ashish Karkare <ashish.karkare@xxxxxxxxxxx>]

> From: "Mikhail Khodjaiants" <mikhailk@xxxxxxx>
>
> Looks good, though I thought that #2 was fixed in 1.2.
>  

I believe you are right.   The new getGDBProcess() method in MIPlugin 
seems to ensure  that  MIException will be rarely thrown  while 
executing the MISession constructor.

IMO,  the fix is still valid,  but its importance is greatly reduced.  I 
will add an appropriate note to the bugzilla report.

Regards,

Ashish.

> ----- Original Message -----
> From: "Ashish Karkare" <ashish.karkare@xxxxxxxxxxx>
>
> > 2. If gdb takes too long to load the application, the MISession can time
> >    
> out,
>  
>
> >    leaving behind orphan session resources, including possibly a running
> >    gdb process. This can lead to failure of subsequent launches, even
> >    
> after
>  
>
> >    the user increases timeout values.
> >
> >    The proposed patch is to catch the exception and terminate the
> >    
> session, and
>  
>
> >    then display the session time out message.
> >
> >