Using an explicit constructor for courgette:MemoryAllocator is incompatible with the implementati...

courgette/memory_allocator.h

 // 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.
 
 #ifndef COURGETTE_MEMORY_ALLOCATOR_H_
 #define COURGETTE_MEMORY_ALLOCATOR_H_
 
 #include <memory>
 
 #include "base/basictypes.h"
@@ skipping 163 matching lines @@
 
   template<class OtherT>
   struct rebind {
     // convert an MemoryAllocator<T> to a MemoryAllocator<OtherT>
     typedef MemoryAllocator<OtherT> other;
   };
 
   MemoryAllocator() _THROW0() {
   }
 
-  explicit MemoryAllocator(const MemoryAllocator<T>& other) _THROW0() {
+  // We can't use an explicit constructor here, as dictated by our style guide.
+  // The implementation of basic_string in Visual Studio 2010 prevents this.
+  MemoryAllocator(const MemoryAllocator<T>& other) _THROW0() {
   }
 
   template<class OtherT>
   explicit MemoryAllocator(const MemoryAllocator<OtherT>& other) _THROW0() {
   }
 
   ~MemoryAllocator() {
   }
 
   void deallocate(pointer ptr, size_type size) {
@@ skipping 61 matching lines @@
 template<class T>
 class MemoryAllocator : public std::allocator<T> {
  public:
 };
 
 #endif  // OS_WIN
 
 }  // namespace courgette
 
 #endif  // COURGETTE_MEMORY_ALLOCATOR_H_