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

Side by Side Diff: src/platform/update_engine/file_writer.h

Issue 466036: AU: Beginnings of delta support (Closed)
Patch Set: Created 11 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
OLDNEW
1 // Copyright (c) 2009 The Chromium OS Authors. All rights reserved. 1 // Copyright (c) 2009 The Chromium OS 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 UPDATE_ENGINE_FILE_WRITER_H__ 5 #ifndef CHROMEOS_PLATFORM_UPDATE_ENGINE_FILE_WRITER_H__
6 #define UPDATE_ENGINE_FILE_WRITER_H__ 6 #define CHROMEOS_PLATFORM_UPDATE_ENGINE_FILE_WRITER_H__
7 7
8 #include <sys/types.h> 8 #include <sys/types.h>
9 #include <sys/stat.h> 9 #include <sys/stat.h>
10 #include <fcntl.h> 10 #include <fcntl.h>
11 #include <unistd.h> 11 #include <unistd.h>
12 #include "base/logging.h" 12 #include "chromeos/obsolete_logging.h"
13 #include "update_engine/utils.h"
13 14
14 // FileWriter is a class that is used to (synchronously, for now) write to 15 // FileWriter is a class that is used to (synchronously, for now) write to
15 // a file. This file is a thin wrapper around open/write/close system calls, 16 // a file. This file is a thin wrapper around open/write/close system calls,
16 // but provides and interface that can be customized by subclasses that wish 17 // but provides and interface that can be customized by subclasses that wish
17 // to filter the data. 18 // to filter the data.
18 19
19 namespace chromeos_update_engine { 20 namespace chromeos_update_engine {
20 21
21 class FileWriter { 22 class FileWriter {
22 public: 23 public:
(...skipping 11 matching lines...) Expand all
34 }; 35 };
35 36
36 // Direct file writer is probably the simplest FileWriter implementation. 37 // Direct file writer is probably the simplest FileWriter implementation.
37 // It calls the system calls directly. 38 // It calls the system calls directly.
38 39
39 class DirectFileWriter : public FileWriter { 40 class DirectFileWriter : public FileWriter {
40 public: 41 public:
41 DirectFileWriter() : fd_(-1) {} 42 DirectFileWriter() : fd_(-1) {}
42 virtual ~DirectFileWriter() {} 43 virtual ~DirectFileWriter() {}
43 44
44 virtual int Open(const char* path, int flags, mode_t mode) { 45 virtual int Open(const char* path, int flags, mode_t mode);
45 CHECK_EQ(-1, fd_); 46 virtual int Write(const void* bytes, size_t count);
46 fd_ = open(path, flags, mode); 47 virtual int Close();
47 if (fd_ < 0)
48 return -errno;
49 return 0;
50 }
51 48
52 virtual int Write(const void* bytes, size_t count) { 49 int fd() const { return fd_; }
53 CHECK_GE(fd_, 0);
54 ssize_t rc = write(fd_, bytes, count);
55 if (rc < 0)
56 return -errno;
57 return rc;
58 }
59
60 virtual int Close() {
61 CHECK_GE(fd_, 0);
62 int rc = close(fd_);
63
64 // This can be any negative number that's not -1. This way, this FileWriter
65 // won't be used again for another file.
66 fd_ = -2;
67
68 if (rc < 0)
69 return -errno;
70 return rc;
71 }
72 50
73 private: 51 private:
74 int fd_; 52 int fd_;
75 }; 53 };
76 54
55 class ScopedFileWriterCloser {
56 public:
57 explicit ScopedFileWriterCloser(FileWriter* writer) : writer_(writer) {}
58 ~ScopedFileWriterCloser() {
59 int err = writer_->Close();
60 if (err)
61 LOG(ERROR) << "FileWriter::Close failed: "
62 << utils::ErrnoNumberAsString(-err);
63 }
64 private:
65 FileWriter* writer_;
66 };
67
77 } // namespace chromeos_update_engine 68 } // namespace chromeos_update_engine
78 69
79 #endif // UPDATE_ENGINE_FILE_WRITER_H__ 70 #endif // CHROMEOS_PLATFORM_UPDATE_ENGINE_FILE_WRITER_H__
OLDNEW
« no previous file with comments | « src/platform/update_engine/download_action_unittest.cc ('k') | src/platform/update_engine/file_writer_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698