Review Handle class
Bug #1209828 reported by
Ueli Hofstetter
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
JQuantLib |
Fix Released
|
High
|
Richard Gomes |
Bug Description
there are one or two a little bit suspicious aspects in the handle related classes..
1: why does Handle implement observable? in ql observers aren't registered at the Handle itself but at the link
2. why does linkTo(setLink) of the link class notify the observers of the observable and not the observers of the link?
3 ...
=============
Relationships
=============
has duplicate http://
child of http://
To post a comment you must log in.
I've changed
1. getLink(...) to currentLink(...) and
2. setLink(...) to linkTo(...)
These changes will make classes Handle and Link look closer to QuantLib/C++ but there are still structural differences.
I've tried to removed "implements Observable" from class Handle and move all responsibilities to class Link but it didn't not work. A new tentative is needed after I figure out what would be the responsibilities of Link in this new scenario (I mean: in the original scenario defined by Quantlib/C++).