Hi, i noticed some unused or duplicated classes in the code, that might be worth removing them. In the package org.exolab.castor.util the classes Iterator and OrderedMap are unused. The unsynchronized Stack class could be replaced by the ArrayStack from the Jakarta commons-collections package. The IteratorEnumeration class also exists in commons-collections and is duplicated as an inner class IteratorEnumerator in ListClassDescriptor. The List class could also be removed if the compatibility with the JDK 1.1.x is dropped (and i think it's already the case since there are JDK 1.2 classes in the source).

Emmanuel Bourg

----------------------------------------------------------- If you wish to unsubscribe from this mailing, send mail to
[EMAIL PROTECTED] with a subject of:
unsubscribe castor-dev

Reply via email to