[Cdk-devel] [ cdk-Patches-3334528 ] real fixes for setAtoms() change event throwing

[SourceForge.net <no...@so...>]

Patches item #3334528, was opened at 2011-06-26 10:52
Message generated for change (Comment added) made by rajarshi
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3334528&group_id=20024

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: cdk-1.4.x
>Group: Accepted
>Status: Closed
>Resolution: Duplicate
Priority: 5
Private: No
Submitted By: Egon Willighagen (egonw)
Assigned to: Nobody/Anonymous (nobody)
Summary: real fixes for setAtoms() change event throwing

Initial Comment:
I messed up in my previous patch... I am positive I ran the unit tests, but I seem to have mixed up stuff... so, none less than *two* fixes for earlier patches:

patch 0001:
Unregister the listeners for the global atoms, not the local one

patch 0002:
The setAtoms() method itself throws a change even too, so the listener must be reset *after* that call.

----------------------------------------------------------------------

>Comment By: Rajarshi Guha (rajarshi)
Date: 2011-06-26 12:37

Message:
D'oh. Should have coffee first thing in the morning. Ran tests now and it
works. Applied and pushed

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=320024&aid=3334528&group_id=20024