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

Side by Side Diff: src/ports/SkMutex_win.h

Issue 19808007: Split atomic and mutex implementations and make inlinable. (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: Address dependency comments. Created 7 years 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 | Annotate | Revision Log
« no previous file with comments | « src/ports/SkMutex_pthread.h ('k') | src/ports/SkThread_none.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 /*
2 * Copyright 2013 Google Inc.
3 *
4 * Use of this source code is governed by a BSD-style license that can be
5 * found in the LICENSE file.
6 */
7
8 #ifndef SkMutex_win_DEFINED
9 #define SkMutex_win_DEFINED
10
11 /** Windows CriticalSection based mutex. */
12
13 #ifndef WIN32_LEAN_AND_MEAN
14 # define WIN32_LEAN_AND_MEAN
15 # define WIN32_IS_MEAN_WAS_LOCALLY_DEFINED
16 #endif
17 #ifndef NOMINMAX
18 # define NOMINMAX
19 # define NOMINMAX_WAS_LOCALLY_DEFINED
20 #endif
21 #
22 #include <windows.h>
23 #
24 #ifdef WIN32_IS_MEAN_WAS_LOCALLY_DEFINED
25 # undef WIN32_IS_MEAN_WAS_LOCALLY_DEFINED
26 # undef WIN32_LEAN_AND_MEAN
27 #endif
28 #ifdef NOMINMAX_WAS_LOCALLY_DEFINED
29 # undef NOMINMAX_WAS_LOCALLY_DEFINED
30 # undef NOMINMAX
31 #endif
32
33 // TODO: this exists because SK_DECLARE_STATIC_ONCE in methods is currently
34 // relying on a compiler bug which allows the '=' to work.
35 // All use of SK_DECLARE_STATIC_ONCE in methods is unsafe, and must be removed.
36 // To find these cases, make SkMutex's copy and assignement private directly.
37 class SkNoncopyableMutex {
38 public:
39 SkNoncopyableMutex() { }
40
41 private:
42 SkNoncopyableMutex(const SkNoncopyableMutex&);
43 SkNoncopyableMutex& operator=(const SkNoncopyableMutex&);
44 };
45
46 // On Windows, SkBaseMutex and SkMutex are the same thing,
47 // we can't easily get rid of static initializers.
48 class SkMutex : SkNoncopyableMutex {
49 public:
50 SkMutex() {
51 InitializeCriticalSection(&fStorage);
52 }
53
54 ~SkMutex() {
55 DeleteCriticalSection(&fStorage);
56 }
57
58 void acquire() {
59 EnterCriticalSection(&fStorage);
60 }
61
62 void release() {
63 LeaveCriticalSection(&fStorage);
64 }
65
66 private:
67 CRITICAL_SECTION fStorage;
68 };
69
70 typedef SkMutex SkBaseMutex;
71
72 // Windows currently provides no documented means of POD initializing a CRITICAL _SECTION.
73 #define SK_DECLARE_STATIC_MUTEX(name) static SkBaseMutex name
74 #define SK_DECLARE_GLOBAL_MUTEX(name) SkBaseMutex name
75
76 #endif
OLDNEW
« no previous file with comments | « src/ports/SkMutex_pthread.h ('k') | src/ports/SkThread_none.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698