Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cdt-dev]WhywasAbstractCLaunchDelegate.getProgramArgumentsArray() made static?

Sorry, just forgot to delete "static". Done.

----- Original Message ----- From: "Ploett, Norbert" <norbert.ploett@xxxxxxxxxxx>
To: "CDT General developers list." <cdt-dev@xxxxxxxxxxx>
Sent: Thursday, May 18, 2006 2:47 AM
Subject: AW: [cdt-dev]WhywasAbstractCLaunchDelegate.getProgramArgumentsArray() made static?


Hi Mikhail,

almost perfect, and it's my fault for making a little mistake because I was actually overriding getProgramArguments() (rather than getProgramArgumentsArray()).

Now the implementation of getProgramArguments() was already moved to the static counterpart in LaunchUtils, but getProgramArguments() itself has also remained static.

Can change this one to be static too.

Thanks for being cooperative


Norbert

-----Urspr√ľngliche Nachricht-----
Von: cdt-dev-bounces@xxxxxxxxxxx [mailto:cdt-dev-bounces@xxxxxxxxxxx] Im Auftrag von Mikhail Khodjaiants
Gesendet: Donnerstag, 18. Mai 2006 00:10
An: CDT General developers list.
Betreff: Re: [cdt-dev] WhywasAbstractCLaunchDelegate.getProgramArgumentsArray() made static?

Norbert,

I reverted the changes and submitted it in CVS. Please, check if it's OK.

Sorry,
Mikhail
----- Original Message ----- From: "Ploett, Norbert" <norbert.ploett@xxxxxxxxxxx>
To: "CDT General developers list." <cdt-dev@xxxxxxxxxxx>
Sent: Wednesday, May 17, 2006 6:39 AM
Subject: [cdt-dev] Why wasAbstractCLaunchDelegate.getProgramArgumentsArray()
made static?


Hi folks,

I was using the fact that getProgramArgumentsArray() is an instance
method. I overrode it in my own launch delegate class derived from
LocalRunLaunchDelegate so I could twist the program arguments in
addition to what LocalRunLaunchDelegate was doing for me. Like so:

public class MyLaunchDelegate extends LocalRunLaunchDelegate  {

public String[] getProgramArgumentsArray( ... )  {
super( ... );
// put twisting code here
}
}

In this way I didn't even have to override the launch() method since the
LocalRunLaunchDelegate was perfect for me, apart from the twist ... :-)

In later revisions (i.e. from 1.40, checked in by Mikhail Khodjaiants
:-) ) the method was changed into a static and the check-in comment does
not give a profound reason. But static methods cannot be overridden.

Can I have my instance method back? Any opinions on that? (Huh,
Mikhail?)

Thanks,


Norbert Ploett

_______________________________________________
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


Back to the top