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

Side by Side Diff: base/mac/scoped_cffiledescriptorref.h

Issue 1551943002: Rewrite most of the scopers in //base/mac to use ScopedTypeRef or ScopedGeneric. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix iOS Created 4 years, 11 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
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef BASE_MAC_SCOPED_CFFILEDESCRIPTORREF_H_ 5 #ifndef BASE_MAC_SCOPED_CFFILEDESCRIPTORREF_H_
6 #define BASE_MAC_SCOPED_CFFILEDESCRIPTORREF_H_ 6 #define BASE_MAC_SCOPED_CFFILEDESCRIPTORREF_H_
7 7
8 #include <CoreFoundation/CoreFoundation.h> 8 #include <CoreFoundation/CoreFoundation.h>
9 9
10 #include "base/compiler_specific.h" 10 #include "base/scoped_generic.h"
11 #include "base/macros.h"
12 11
13 namespace base { 12 namespace base {
14 namespace mac { 13 namespace mac {
15 14
15 namespace internal {
16
17 struct ScopedCFFileDescriptorRefTraits {
18 static CFFileDescriptorRef InvalidValue() { return nullptr; }
19 static void Free(CFFileDescriptorRef ref) {
20 CFFileDescriptorInvalidate(ref);
21 CFRelease(ref);
22 }
23 };
24
25 } // namespace internal
26
16 // ScopedCFFileDescriptorRef is designed after ScopedCFTypeRef<>. On 27 // ScopedCFFileDescriptorRef is designed after ScopedCFTypeRef<>. On
17 // destruction, it will invalidate the file descriptor. 28 // destruction, it will invalidate the file descriptor.
18 // ScopedCFFileDescriptorRef (unlike ScopedCFTypeRef<>) does not support RETAIN 29 // ScopedCFFileDescriptorRef (unlike ScopedCFTypeRef<>) does not support RETAIN
19 // semantics, copying, or assignment, as doing so would increase the chances 30 // semantics, copying, or assignment, as doing so would increase the chances
20 // that a file descriptor is invalidated while still in use. 31 // that a file descriptor is invalidated while still in use.
21 class ScopedCFFileDescriptorRef { 32 using ScopedCFFileDescriptorRef =
22 public: 33 ScopedGeneric<CFFileDescriptorRef,
23 explicit ScopedCFFileDescriptorRef(CFFileDescriptorRef fdref = NULL) 34 internal::ScopedCFFileDescriptorRefTraits>;
24 : fdref_(fdref) {
25 }
26
27 ~ScopedCFFileDescriptorRef() {
28 if (fdref_) {
29 CFFileDescriptorInvalidate(fdref_);
30 CFRelease(fdref_);
31 }
32 }
33
34 void reset(CFFileDescriptorRef fdref = NULL) {
35 if (fdref_ == fdref)
36 return;
37 if (fdref_) {
38 CFFileDescriptorInvalidate(fdref_);
39 CFRelease(fdref_);
40 }
41 fdref_ = fdref;
42 }
43
44 bool operator==(CFFileDescriptorRef that) const {
45 return fdref_ == that;
46 }
47
48 bool operator!=(CFFileDescriptorRef that) const {
49 return fdref_ != that;
50 }
51
52 operator CFFileDescriptorRef() const {
53 return fdref_;
54 }
55
56 CFFileDescriptorRef get() const {
57 return fdref_;
58 }
59
60 CFFileDescriptorRef release() WARN_UNUSED_RESULT {
61 CFFileDescriptorRef temp = fdref_;
62 fdref_ = NULL;
63 return temp;
64 }
65
66 private:
67 CFFileDescriptorRef fdref_;
68
69 DISALLOW_COPY_AND_ASSIGN(ScopedCFFileDescriptorRef);
70 };
71 35
72 } // namespace mac 36 } // namespace mac
73 } // namespace base 37 } // namespace base
74 38
75 #endif // BASE_MAC_SCOPED_CFFILEDESCRIPTORREF_H_ 39 #endif // BASE_MAC_SCOPED_CFFILEDESCRIPTORREF_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698