i2p/haiku-beta6
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Removed some unnecessary TODO items.

Browse Source 
git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@16913 a95241bf-73f2-0310-859d-f6bbb57e9c96
This commit is contained in:
Axel Dörfler
2006-03-28 14:52:56 +00:00
parent 090dcfd4a8
commit ad236123aa
1 changed files with 5 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/kits/app/Handler.cpp
+5 -9
View File
@@ -90,6 +90,7 @@ class _ObserverList {
public: public:
_ObserverList(void); _ObserverList(void);
~_ObserverList(void); ~_ObserverList(void);
status_t SendNotices(unsigned long, BMessage const *); status_t SendNotices(unsigned long, BMessage const *);
status_t StartObserving(BHandler *, unsigned long); status_t StartObserving(BHandler *, unsigned long);
status_t StartObserving(const BMessenger&, unsigned long); status_t StartObserving(const BMessenger&, unsigned long);
@@ -123,11 +124,10 @@ BHandler::~BHandler()
delete (BMessageFilter*)fFilters->ItemAtFast(i); delete (BMessageFilter*)fFilters->ItemAtFast(i);
delete fFilters; delete fFilters;
} }
// remove all observers
if (fObserverList) { // remove all observers (the observer list manages itself)
// TODO ... ?!? delete fObserverList;
delete fObserverList;
}
// free rest // free rest
free(fName); free(fName);
gDefaultTokens.RemoveToken(fToken); gDefaultTokens.RemoveToken(fToken);
@@ -268,7 +268,6 @@ BHandler::SetNextHandler(BHandler *handler)
{ {
if (!fLooper) { if (!fLooper) {
debugger("handler must belong to looper before setting NextHandler"); debugger("handler must belong to looper before setting NextHandler");
fNextHandler = NULL;
return; return;
} }
@@ -282,9 +281,6 @@ BHandler::SetNextHandler(BHandler *handler)
return; 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; fNextHandler = handler;
} }