Hi Todd,
>>    private static class ElementListenerList extends
>> ListenerList<ElementListener>
> Yep - you can go ahead and make this change.
Done.


> And a last warning on Element:
>> org.apache.pivot.wtk.text.Element$NodeOffsetComparator implements
>> Comparator but not Serializable
> It does not need to implement Serializable, as far as I know.
>From the Comparator docs:

Note: It is generally a good idea for comparators to also implement
java.io.Serializable, as they may be used as ordering methods in
serializable data structures (like TreeSet, TreeMap). In order for the
data structure to serialize successfully, the comparator (if provided)
must implement Serializable.

So i think i could do also this, ok ?

Bye

Reply via email to