Bug 15939 – Add currentThread API

• Status: RESOLVED
• Resolution: FIXED
• Component: Platform

[Justin Haygood]

in order to get rid of all direct useages of pthread, we need a currentThread API

[Justin Haygood]

Created attachment 17179 [details]
Implements currentThread API

[Adam Roben (:aroben)]

Comment on attachment 17179 [details]
Implements currentThread API

+static ThreadIdentifier identifierByPthreadHandle(pthread_t& pthreadhandle)

The parameter should be marked const (ditto for the ThreadingGtk version).

+    for (; i != threadMap().end(); ++i) {
+        if (i->second == thread))
+            return i->first;
+    }

You don't need the braces around the body of the loop.

r=me

[Adam Roben (:aroben)]

(In reply to comment #2)
> (From update of attachment 17179 [details] [edit])
> +    for (; i != threadMap().end(); ++i) {
> +        if (i->second == thread))
> +            return i->first;
> +    }
> 
> You don't need the braces around the body of the loop.

Please disregard this. The braces are correct per our style guidelines <http://webkit.org/coding/coding-style.html>.

[Mark Rowe (bdash)]

Landed in r27707 after some fixes to the gthreads version and improvements to match the style guidelines.

[Alexey Proskuryakov]

*** Bug 15935 has been marked as a duplicate of this bug. ***