I saw the following comment go by:
> r30783 | johann.sorel | 2008-06-22 03:41:45 -0700 (Sun, 22 Jun 2008) | 
> 1 line
>
> removed parameters lenght check at categorize function creation, this 
> makes the factory go crazy.
Can you please be more specific? Which factories any why? Chances are
the factories are broken ...

The diff shows the following check was removed:
>  public CategorizeFunction(List<Expression> parameters, Literal 
> fallback) {
>      if(parameters == null || parameters.size() <2){
>          throw new IllegalArgumentException("Categorize function must 
> have at least 2 parameters : lookup value and the value");
>      }
>      this.parameters = parameters;
>      this.fallback = fallback;
>  }
Specifically I am not wanting to support someone creating the function
and changing the parameters afterwords; if there is a factory making
that assumption I want to know about it now; with this new function that
not many people use yet.

So yeah - what factory was going crazy.
Jody



-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to