Scheduling and other cleanup

RESOLVED FIXED in mozilla24

Status

()

defect
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: ekr, Assigned: ekr)

Tracking

unspecified
mozilla24
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

No description provided.
Attachment #765187 - Flags: review?(tterribe)
Comment on attachment 765187 [details] [diff] [review]
Scheduling and other cleanup

Review of attachment 765187 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with the requested comment.

Just curious, what's the other cleanup?

::: media/webrtc/trunk/webrtc/system_wrappers/source/thread_posix.cc
@@ +206,5 @@
>    }
>    int result = pthread_attr_setdetachstate(&attr_, PTHREAD_CREATE_DETACHED);
>    // Set the stack stack size to 1M.
>    result |= pthread_attr_setstacksize(&attr_, 1024 * 1024);
> +#if 0

Please add a comment pointing to either this bug or the webrtc.org issue to say why you're doing this.
Attachment #765187 - Flags: review?(tterribe) → review+
(In reply to Timothy B. Terriberry (:derf) from comment #3)
> Comment on attachment 765187 [details] [diff] [review]
> Scheduling and other cleanup
> 
> Review of attachment 765187 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> r=me with the requested comment.
> 
> Just curious, what's the other cleanup?

It's so clean you can't see it. (the log message is stale).


> ::: media/webrtc/trunk/webrtc/system_wrappers/source/thread_posix.cc
> @@ +206,5 @@
> >    }
> >    int result = pthread_attr_setdetachstate(&attr_, PTHREAD_CREATE_DETACHED);
> >    // Set the stack stack size to 1M.
> >    result |= pthread_attr_setstacksize(&attr_, 1024 * 1024);
> > +#if 0
> 
> Please add a comment pointing to either this bug or the webrtc.org issue to
> say why you're doing this.

Willdo.
https://hg.mozilla.org/mozilla-central/rev/7e6a46bbbbf6
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.