On Thu, Nov 6, 2008 at 12:33 AM, Ibrar Ahmed <[EMAIL PROTECTED]> wrote:
>
> Hi,
>
> I am currently working on (Issue 4152:           Porting directory_lister to
> POSIX.). While looking at the code a question bumped into my mind that
> why we are creating thread using
>
>  unsigned thread_id;
>  thread_ = reinterpret_cast<HANDLE>(
>      _beginthreadex(NULL, 0, DirectoryLister::ThreadFunc, this, 0,
>                     &thread_id));
>
> AFAIK there are classes to manage threads in code base. Why we have
> not used these classes
>
> I.e
>
>
>  thread_.reset(new base::Thread("director_lister"));
>  thread_->Start();
>  message_loop_ = thread_->message_loop();
>
>  message_loop_->PostTask(FROM_HERE, NewRunnableMethod(
>        this, &DirectoryLister::ThreadFunc));
>
>
> Can we replace code with this piece of code?

Perhaps because the Thread class has a message loop that
we don't need for directory_lister?  It seems that we can use
the PlatformThread class though.

Wan-Teh

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Chromium-dev" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/chromium-dev?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to