On Mon, 29 Oct 2007, Alexandru Stanoi wrote:

>  Review by Derick
>  ================
>  
> -[ ] I am not sure if the name for the new option constant for
> +[-] I am not sure if the name for the new option constant for
>       addUnorderedParameter() are good - the
>       ezcUrlConfiguration::AGGREGATE_ARGUMENTS one. Perhaps
>       MULTIPLE_ARRAY_ARGUMENTS would work? Or perhaps MULTIPLE_ARGUMENTS |
>       ARRAY_ARGUMENTS would be a viable alternative? I think the latter makes
>       most sense, but is slightly more to type.
> +
> +     I think the AGGREGATE_ARGUMENTS is better.

Please bring this one up on the mailinglist though.

regards,
Derick
-- 
svn-components mailing list
svn-components@lists.ez.no
http://lists.ez.no/mailman/listinfo/svn-components

Reply via email to