commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Stephen Colebourne" <scolebou...@btopenworld.com>
Subject Re: [collections] ObservedCollection checkin
Date Thu, 28 Aug 2003 21:28:41 GMT
This one is tough. The internals of the Standard event mechanism currently
rely on one listener for both. It can also get complicated with a single
addListener method trying to figure out what has been added. Plus there are
the masks to consider.

Merging the two into one made implementation easier. But if the user API
feels wrong then it could be changed.

Stephen

----- Original Message -----
From: "Michael Heuer" <heuermh@acm.org>
> What about
>
> StandardModificationListener
> {
>   void modificationOccurred(Evt);
> }
> VetoableModificationListener
> {
>   void modificationOccurring(Evt) throws ModificationVetoedException;
> }
>
> and
>
> ObservedCollection
> {
> static ObservedCollection decorate(Collection coll,
>                                    VetoableModificationListener l) {}
> static ObservedCollection decorate(Collection coll,
>                                    StandardModificationListener l) {}
> }
>
> instead of
>
> StandardModificationListener
> {
>   void modificationOccurred(Evt);
>   void modificationOccurring(Evt) throws ModificationVetoedException;
> }
>
> and
>
> ObservedCollection
> {
> static ObservedCollection decorate(Collection coll,
>                                    StandardModificationListener l) {}
> static ObservedCollection decoratePostEventsOnly(Collection coll,
>                                    StandardModificationListener l) {}
> }
>
> ?
>
>    michael
>
> On Thu, 28 Aug 2003, Stephen Colebourne wrote:
>
> > First part of ObservedCollection checked in, allowing an event sending
> > wrapper to be written around a collection.
> >
> > TODO:
> > - Validate that the design is OK
> > - Implement ObservedList subList
> > - Test List iterators
> > - Add Bag, SortedSet, SortedBag implementations
> > - Add Fast event mechansim
> > - Add Rich event mechanism
> >
> > Stephen
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> > For additional commands, e-mail: commons-dev-help@jakarta.apache.org
> >
> >
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: commons-dev-help@jakarta.apache.org
>


Mime
View raw message