Hi Marius,

On May 15, 2012, at 7:54 PM, Marius Dumitru Florea wrote:

> Hi devs,
> 
> I just committed a new Velocity uberspector that automatically
> converts String method arguments to Enum when possible. Please review
> it and tell me if there's something wrong.

Seems ok, can't find anything that wouldn't work for the moment :)

> The idea is to be able to call #someMethod(SomeEnum) with
> 
> $obj.someMethod('VALUE')

What if I call #someMethod(String, SomeEnum)

with

$obj.someMethod(string, myEnum)

Does it work? (same question with any number of arguments)

If not then maybe it's a bit inconsistent and some users may not understand why 
it works and then suddenly fail to work in the following case: imagine that the 
method has one param initially (the enum) and a second param is added later on.

Thanks
-Vincent

> if SomeEnum.VALUE is a valid constant.
> 
> See 
> https://github.com/xwiki/xwiki-commons/commit/dbf1e06cc4f949c6f0b5a03c57067174cc620888
> .
> 
> Thanks,
> Marius
_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs

Reply via email to