Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(213)

Side by Side Diff: src/ports/SkTLS_pthread.cpp

Issue 2003163004: pthread_once -> SkOnce (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: simpler Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright 2013 Google Inc. 2 * Copyright 2013 Google Inc.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license that can be 4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file. 5 * found in the LICENSE file.
6 */ 6 */
7 7
8 #include "SkTLS.h" 8 #include "SkTLS.h"
9 #include "SkOnce.h"
9 10
10 #include <pthread.h> 11 #include <pthread.h>
11 12
12 static pthread_key_t gSkTLSKey; 13 static pthread_key_t gSkTLSKey;
13 static pthread_once_t gSkTLSKey_Once = PTHREAD_ONCE_INIT;
14
15 static void sk_tls_make_key() {
16 (void)pthread_key_create(&gSkTLSKey, SkTLS::Destructor);
17 }
18 14
19 void* SkTLS::PlatformGetSpecific(bool forceCreateTheSlot) { 15 void* SkTLS::PlatformGetSpecific(bool forceCreateTheSlot) {
20 // should we use forceCreateTheSlot to potentially skip calling pthread_once 16 // should we use forceCreateTheSlot to potentially just return nullptr if
21 // and just return nullptr if we've never been called with 17 // we've never been called with forceCreateTheSlot==true ?
22 // forceCreateTheSlot==true ? 18 static SkOnce once;
23 19 once(pthread_key_create, &gSkTLSKey, SkTLS::Destructor);
24 (void)pthread_once(&gSkTLSKey_Once, sk_tls_make_key);
25 return pthread_getspecific(gSkTLSKey); 20 return pthread_getspecific(gSkTLSKey);
26 } 21 }
27 22
28 void SkTLS::PlatformSetSpecific(void* ptr) { 23 void SkTLS::PlatformSetSpecific(void* ptr) {
29 (void)pthread_setspecific(gSkTLSKey, ptr); 24 (void)pthread_setspecific(gSkTLSKey, ptr);
30 } 25 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698