Jody Garnett a écrit : > Hi Guys: > > Emily found out that my new "Concatenate" function did not work - > because the number of arguments was treated as an upper limit :-) If > you extend FunctionImpl this check is not performed; I have changed > Concatenate to use FunctionImpl as a superclass (rather than > FunctionExpressionImpl). > > If possible I would like to leave the check in FunctionExpressionImpl > (as that class is already deprecated and should be shown the door...). > > Johann is this okay with you? You had asked for the limit to be > removed previously, you will notice that the CategorizeFunction I > asked you to review previously does not user any super class. > > Jody At this time I removed the check because the factory got crazy (she tryes to create a function with no parameter) and it was not the upper limit but the lower limit. I have no opinion, number of parameter may be static or limited, with a minimum/maximum or not, it depends on the function. We cant impose something for every function.
------------------------------------------------------------------------- This SF.Net email is sponsored by the Moblin Your Move Developer's challenge Build the coolest Linux based applications with Moblin SDK & win great prizes Grand prize is a trip for two to an Open Source event anywhere in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/ _______________________________________________ Geotools-devel mailing list Geotools-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
