Hi,

> I also guess that if you just make ElementListenerList a "static
> class", the warning/error will go away (and you make it one reference
> smaller on the heap, and a slightly smaller constructor code).
Right, changing the inner class definition to:

//    private class ElementListenerList extends ListenerList<ElementListener>
    private static class ElementListenerList extends
ListenerList<ElementListener>

the warning/error has gone out ... but is this correct (as a behavior)
for the Element class ?


And a last warning on Element:
org.apache.pivot.wtk.text.Element$NodeOffsetComparator implements
Comparator but not Serializable
So, this is ok ?

//    private static class NodeOffsetComparator implements Comparator<Node> {
    private static class NodeOffsetComparator implements
Comparator<Node>, Serializable {
                private static final long serialVersionUID = 0L;



Thanks,
Sandro

Reply via email to