Re: [Quantlib-dev] destructor performance, observables with large observer lists

[Ferdinando A. <na...@am...>]

On Sat, Jun 25, 2011 at 5:48 PM, Plamen Neykov
<pla...@re...> wrote:
> wouldn't it be simpler just to use std::set<Observer*> and save you the trouble with the long id or am I missing something here?

great minds think alike... as a matter of fact in the trunk it has
already been switched to std:set on June 7th :-)

see http://quantlib.svn.sourceforge.net/viewvc/quantlib?view=revision&revision=17788

I was more concerned with possible non-unique elements than
destructor's performance reason, and that's why I would keep it at
std::set instead of having it as template parameter.

Roland could you confirm that the trunk solution is OK for you?

BTW in the current trunk there is a MAJOR performance improvement if
you work in a real time environment with many changing rate quotes
between recalculations: I patched a bug which triggered many useless
notifications

thanks to all for the report and help: it's refreshing to have
contributors really stressing the library.

ciao -- Nando