From 289b6c8d560c0856b42fdd0acbec996b56d27c24 Mon Sep 17 00:00:00 2001 From: Rene Gollent Date: Sat, 14 Mar 2009 00:18:23 +0000 Subject: [PATCH] Remove invalid TODO. git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@29504 a95241bf-73f2-0310-859d-f6bbb57e9c96 --- src/system/kernel/messaging/MessagingService.cpp | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) diff --git a/src/system/kernel/messaging/MessagingService.cpp b/src/system/kernel/messaging/MessagingService.cpp index de685b41d4..02b17ed3fb 100644 --- a/src/system/kernel/messaging/MessagingService.cpp +++ b/src/system/kernel/messaging/MessagingService.cpp @@ -450,14 +450,7 @@ MessagingService::_AllocateCommand(int32 commandWhat, int32 size, // delete the discarded areas (save one) ObjectDeleter discardedAreaDeleter; MessagingArea *discardedArea = NULL; - // TODO: this loop still has a flaw - // - if the first area in the list is full, - // we will always allocate a new area to add the next message to, - // even if we already have another area in the chain with enough space - // if the allocation fails this probably needs to walk the list - // and keep trying for each non-empty area it finds until it either - // succeeds or hits the last area. Only in the latter case should we - // be allocating a new one. + while (fFirstArea != fLastArea) { area = fFirstArea; area->Lock();