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

Side by Side Diff: base/scoped_temp_dir.h

Issue 6042009: Added WARN_UNUSED_RESULT to ScopedTempDir methods. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: merge with trunk Created 9 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 | Annotate | Revision Log
« no previous file with comments | « no previous file | base/scoped_temp_dir_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 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_SCOPED_TEMP_DIR_H_ 5 #ifndef BASE_SCOPED_TEMP_DIR_H_
6 #define BASE_SCOPED_TEMP_DIR_H_ 6 #define BASE_SCOPED_TEMP_DIR_H_
7 #pragma once 7 #pragma once
8 8
9 // An object representing a temporary / scratch directory that should be cleaned 9 // An object representing a temporary / scratch directory that should be cleaned
10 // up (recursively) when this object goes out of scope. Note that since 10 // up (recursively) when this object goes out of scope. Note that since
(...skipping 10 matching lines...) Expand all
21 class ScopedTempDir { 21 class ScopedTempDir {
22 public: 22 public:
23 // No directory is owned/created initially. 23 // No directory is owned/created initially.
24 ScopedTempDir(); 24 ScopedTempDir();
25 25
26 // Recursively delete path. 26 // Recursively delete path.
27 ~ScopedTempDir(); 27 ~ScopedTempDir();
28 28
29 // Creates a unique directory in TempPath, and takes ownership of it. 29 // Creates a unique directory in TempPath, and takes ownership of it.
30 // See file_util::CreateNewTemporaryDirectory. 30 // See file_util::CreateNewTemporaryDirectory.
31 bool CreateUniqueTempDir(); 31 bool CreateUniqueTempDir() WARN_UNUSED_RESULT;
32 32
33 // Creates a unique directory under a given path, and takes ownership of it. 33 // Creates a unique directory under a given path, and takes ownership of it.
34 bool CreateUniqueTempDirUnderPath(const FilePath& path); 34 bool CreateUniqueTempDirUnderPath(const FilePath& path) WARN_UNUSED_RESULT;
35 35
36 // Takes ownership of directory at |path|, creating it if necessary. 36 // Takes ownership of directory at |path|, creating it if necessary.
37 // Don't call multiple times unless Take() has been called first. 37 // Don't call multiple times unless Take() has been called first.
38 bool Set(const FilePath& path); 38 bool Set(const FilePath& path) WARN_UNUSED_RESULT;
39 39
40 // Deletes the temporary directory wrapped by this object. 40 // Deletes the temporary directory wrapped by this object.
41 bool Delete(); 41 bool Delete() WARN_UNUSED_RESULT;
42 42
43 // Caller takes ownership of the temporary directory so it won't be destroyed 43 // Caller takes ownership of the temporary directory so it won't be destroyed
44 // when this object goes out of scope. 44 // when this object goes out of scope.
45 FilePath Take(); 45 FilePath Take();
46 46
47 const FilePath& path() const { return path_; } 47 const FilePath& path() const { return path_; }
48 48
49 // Returns true if path_ is non-empty and exists. 49 // Returns true if path_ is non-empty and exists.
50 bool IsValid() const; 50 bool IsValid() const;
51 51
52 private: 52 private:
53 FilePath path_; 53 FilePath path_;
54 54
55 DISALLOW_COPY_AND_ASSIGN(ScopedTempDir); 55 DISALLOW_COPY_AND_ASSIGN(ScopedTempDir);
56 }; 56 };
57 57
58 #endif // BASE_SCOPED_TEMP_DIR_H_ 58 #endif // BASE_SCOPED_TEMP_DIR_H_
OLDNEW
« no previous file with comments | « no previous file | base/scoped_temp_dir_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698