Yair Zaslavsky has posted comments on this change.

Change subject: core : Add ability to listen for extension registration
......................................................................


Patch Set 5:

Please take a look at -
http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/6-b14/java/util/Observable.java

To my understanding, the changed flag is set to false at the CTOR, and 
notifyObservers will notify only if hasChanged() returns true. For that, you 
will need to call setChanged().

-- 
To view, visit http://gerrit.ovirt.org/27602
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Idd78e7a390c2a45062be1741548e370a205bfe24
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to