Jan-Jaap van der Geer wrote:
On Sun, 2009-11-22 at 12:19 -0800, Peter Naulls wrote:
Jan-Jaap van der Geer wrote:
This adds a third parameter to ab_create_app to indicate which
package.
This worked for me, although I'm not really sure if it is OK to add
a
third parameter when the second one can be empty. I've not seen that
kind of ab_create_app commands though, so I'm not sure if it is
relevant.
You'd want to nest such a check, since unless someone explicitly put
"" as a parameter, you'd never get an empty second parameter but
non-empty 3rd one.
Well, my reasoning was that all current packages that use ab_create_app
will either have one or two parameters. So by adding a third it could be
used there where appropriate, if it is using two parameters. Since you'd
have to change the code to make use of it, this wouldn't really be a
problem, you'd only use it if it makes sense.
Since you seem to confirm that you actually can call it with "" to
indicate an empty second parameter and a non-empty third parameter, I
don't see why there is a problem.
I may want to overload this differently, for the case of multiple apps
in one package. But anyway, apart from the mechanics, can you describe
what it is you are trying to archive here, since I'm not really sure of
that. There is probably a better way to do both things.
_______________________________________________
GCCSDK mailing list [email protected]
Bugzilla: http://www.riscos.info/bugzilla/index.cgi
List Info: http://www.riscos.info/mailman/listinfo/gcc
Main Page: http://www.riscos.info/index.php/GCCSDK
