Add empty check to ObserverList in finalize() |
||
Issue descriptionWe could add an optional empty check to ObserverList in finalize() to push back against memory bloat and odd errors. The C++ version already has this, and it is used a lot.
,
Apr 7 2017
(sorry, for context that would essentially be after Easter)
,
Feb 8 2018
,
Feb 8 2018
Are we ok with adding a finalizer() to ObserverList? IIRC that adds some overhead (although maybe not much).
,
Feb 12 2018
I think we discussed this and decided against it, primarily because of the overhead. It may be useful if we could find a way that the finalizer is present only in debug builds. |
||
►
Sign in to add a comment |
||
Comment 1 by peconn@chromium.org
, Apr 7 2017