Implementation of an STL compatible allocator for Courgette on Windows....

courgette/memory_allocator.cc

Index: courgette/memory_allocator.cc
===================================================================
--- courgette/memory_allocator.cc	(revision 0)
+++ courgette/memory_allocator.cc	(revision 0)
@@ -0,0 +1,148 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "courgette/memory_allocator.h"
+
+#include "base/file_util.h"
+#include "base/logging.h"
+
+#ifdef OS_WIN
+
+namespace courgette {
+
+// TempFile
+
+TempFile::TempFile() : file_(base::kInvalidPlatformFileValue), size_(0) {
+}
+
+TempFile::~TempFile() {
+  Close();
+}
+
+void TempFile::Close() {
+  if (valid()) {
+    base::ClosePlatformFile(file_);
+    file_ = base::kInvalidPlatformFileValue;
+    size_ = 0;
+  }
+}
+
+bool TempFile::Create() {
+  DCHECK(file_ == base::kInvalidPlatformFileValue);
+  FilePath path;
+  file_util::CreateTemporaryFile(&path);

[grt (UTC plus 2), 2011/02/28 15:06:41]

what if this returns false?

[tommi (sloooow) - chröme, 2011/02/28 17:50:00]

Done.

+  bool created = false;
+  base::PlatformFileError error_code = base::PLATFORM_FILE_OK;
+  int flags = base::PLATFORM_FILE_OPEN_ALWAYS | base::PLATFORM_FILE_READ |
+              base::PLATFORM_FILE_WRITE | base::PLATFORM_FILE_DELETE_ON_CLOSE |
+              base::PLATFORM_FILE_TEMPORARY;
+  file_ = base::CreatePlatformFile(path, flags, &created, &error_code);

[grt (UTC plus 2), 2011/02/28 15:06:41]

PLOG_IF(ERROR, file_ == base::kInvalidPlatformFileValue) << ...?

[tommi (sloooow) - chröme, 2011/02/28 17:50:00]

Done.

+  return valid();
+}
+
+bool TempFile::valid() const {
+  return file_ != base::kInvalidPlatformFileValue;
+}
+
+base::PlatformFile TempFile::handle() const {
+  return file_;
+}
+
+size_t TempFile::size() const {
+  return size_;
+}
+
+bool TempFile::SetSize(size_t size) {
+  bool ret = base::TruncatePlatformFile(file_, size);
+  if (ret)
+    size_ = size;
+  return ret;
+}
+
+// FileMapping
+
+FileMapping::FileMapping() : mapping_(NULL), view_(NULL) {
+}
+
+FileMapping::~FileMapping() {
+  Close();
+}
+
+bool FileMapping::Create(HANDLE file, size_t size) {
+  DCHECK(file != INVALID_HANDLE_VALUE);
+  DCHECK(!valid());
+  mapping_ = ::CreateFileMapping(file, NULL, PAGE_READWRITE, 0, 0, NULL);
+  if (mapping_)
+    view_ = ::MapViewOfFile(mapping_, FILE_MAP_WRITE, 0, 0, size);
+
+  if (!valid()) {
+    PLOG(ERROR) << "Failed to map file";
+    Close();
+  } else {
+    char* write = reinterpret_cast<char*>(view_);
+    write[0] = 0;
+    write[size - 1] = 0;
+  }
+
+  return valid();
+}
+
+void FileMapping::Close() {
+  if (view_)
+    ::UnmapViewOfFile(view_);
+  if (mapping_)
+    ::CloseHandle(mapping_);
+  mapping_ = NULL;
+  view_ = NULL;
+}
+
+bool FileMapping::valid() const {
+  return view_ != NULL;
+}
+
+void* FileMapping::view() const {
+  return view_;
+}
+
+// TempMapping
+
+TempMapping::TempMapping() {
+}
+
+TempMapping::~TempMapping() {
+}
+
+bool TempMapping::Initialize(size_t size) {
+  size += sizeof(TempMapping*);  // NOLINT

[grt (UTC plus 2), 2011/02/28 15:06:41]

Style guide says "Use sizeof(varname) instead of sizeof(type) whenever
possible.", so how about sizeof(this)?

[tommi (sloooow) - chröme, 2011/02/28 17:50:00]

Done.

+  bool ret = file_.Create() && file_.SetSize(size) &&
+             mapping_.Create(file_.handle(), size);
+  if (ret) {
+    TempMapping** write = reinterpret_cast<TempMapping**>(mapping_.view());
+    *write = this;
+  }
+
+  return ret;
+}
+
+void* TempMapping::memory() const {
+  uint8* mem = reinterpret_cast<uint8*>(mapping_.view());
+  if (mem)
+    mem += sizeof(TempMapping*);  // NOLINT

[grt (UTC plus 2), 2011/02/28 15:06:41]

sizeof(this)?

[tommi (sloooow) - chröme, 2011/02/28 17:50:00]

Done.

+  DCHECK(mem);
+  return mem;
+}
+
+// static
+TempMapping* TempMapping::GetMappingFromPtr(void* mem) {
+  TempMapping* ret = NULL;
+  if (mem) {
+    ret = reinterpret_cast<TempMapping**>(mem)[-1];
+  }
+  DCHECK(ret);
+  return ret;
+}
+
+}  // namespace courgette
+
+#endif  // OS_WIN
Property changes on: courgette\memory_allocator.cc
___________________________________________________________________
Added: svn:eol-style
   + LF