See: src/blocks/forms/java/org/apache/cocoon/forms/samples/DateTestJavaSelectionList.java
In the buld method() there where originally 3 nulls instead of the "fast fixed" (String)null.
Without the casting of the null the compiler in 1.4.2_07 said it was not able to decide wich of this 2 methods use:
protected void addItem(Object value, String label) protected void addItem(Object value, XMLizable label)
Yep.
This is why I casted them to (String). I don't carefully checked the code, I just wanted the code to compile and test other things. Since we are near to a release I am doing full builds now (Sorry, my fault).
All in all, Since the class is under the o.a.c.forms.samples package, then I don't think it is an important one. Hence I thought will be OK a quick fix here.
If someone know better where is used this code and why, please review the fix and change it as the best it can be.
(String) null is the proper fix, so all is ok now.
Vadim
