Skip to main content



      Home
Home » Eclipse Projects » EclipseLink » IndirectSet triggers raiseAddChangeEvent even when no element was added
IndirectSet triggers raiseAddChangeEvent even when no element was added [message #666290] Tue, 19 April 2011 14:52 Go to next message
Eclipse UserFriend
Hi,

I was debugging some code and noticed that when adding an element into the IndirectSet, an event raiseAddChangeEvent is being triggered even when the object is actually not added, causing IMHO unnecessary overhead.

Here is the snippet :
    public boolean add(Object element) {
        boolean added = true;
        // PERF: If not instantiated just record the add to avoid the instantiation.
        if (shouldAvoidInstantiation()) {
            if (hasRemovedElements() && getRemovedElements().contains(element)) {
                getRemovedElements().remove(element);
            } else if (getAddedElements().contains(element)) {
                // Must avoid recursion for relationship maintenance.                
                return false;
            } else {
                getAddedElements().add(element);
            }
        } else {
            added = getDelegate().add(element);
        }
        raiseAddChangeEvent(element);
        return added;
    }


Note the line

    added = getDelegate().add(element);


Variable added is set to false if the element is already present in the delegated set but at the end of the method the event raiseAddChangeEvent is triggered regardless the added variable

I think the event should only be triggered when the element was actually added.

Jakub
Re: IndirectSet triggers raiseAddChangeEvent even when no element was added [message #666662 is a reply to message #666290] Thu, 21 April 2011 09:37 Go to previous message
Eclipse UserFriend
Sounds correct, please log a bug for this.
Previous Topic:postgres and CursoredStream
Next Topic:Transfer entities between entitymanagers
Goto Forum:
  


Current Time: Sat Jul 05 03:08:23 EDT 2025

Powered by FUDForum. Page generated in 0.02873 seconds
.:: Contact :: Home ::.

Powered by: FUDforum 3.0.2.
Copyright ©2001-2010 FUDforum Bulletin Board Software

Back to the top