From ad236123aa06305ad1264a97b22ebb1c43983085 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Axel=20D=C3=B6rfler?= Date: Tue, 28 Mar 2006 14:52:56 +0000 Subject: [PATCH] Removed some unnecessary TODO items. git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@16913 a95241bf-73f2-0310-859d-f6bbb57e9c96 --- src/kits/app/Handler.cpp | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/src/kits/app/Handler.cpp b/src/kits/app/Handler.cpp index 8a4881737f..67ed32c07b 100644 --- a/src/kits/app/Handler.cpp +++ b/src/kits/app/Handler.cpp @@ -90,6 +90,7 @@ class _ObserverList { public: _ObserverList(void); ~_ObserverList(void); + status_t SendNotices(unsigned long, BMessage const *); status_t StartObserving(BHandler *, unsigned long); status_t StartObserving(const BMessenger&, unsigned long); @@ -123,11 +124,10 @@ BHandler::~BHandler() delete (BMessageFilter*)fFilters->ItemAtFast(i); delete fFilters; } - // remove all observers - if (fObserverList) { - // TODO ... ?!? - delete fObserverList; - } + + // remove all observers (the observer list manages itself) + delete fObserverList; + // free rest free(fName); gDefaultTokens.RemoveToken(fToken); @@ -268,7 +268,6 @@ BHandler::SetNextHandler(BHandler *handler) { if (!fLooper) { debugger("handler must belong to looper before setting NextHandler"); - fNextHandler = NULL; return; } @@ -282,9 +281,6 @@ BHandler::SetNextHandler(BHandler *handler) return; } - // NOTE: I'm sure some sort of threading protection should happen here, - // hopefully the spec-mandated BLooper lock is sufficient. - // TODO: implement correctly fNextHandler = handler; }