Add "explicit" to ctors where appropriate.
authorRui Ueyama <ruiu@google.com>
Wed, 3 Jul 2013 07:32:40 +0000 (07:32 +0000)
committerRui Ueyama <ruiu@google.com>
Wed, 3 Jul 2013 07:32:40 +0000 (07:32 +0000)
llvm-svn: 185517

lld/lib/ReaderWriter/PECOFF/Atoms.h
lld/lib/ReaderWriter/PECOFF/ReaderCOFF.cpp
lld/lib/ReaderWriter/PECOFF/WriterPECOFF.cpp

index f8b14fc..f7a99be 100644 (file)
@@ -30,7 +30,7 @@ using llvm::object::coff_symbol;
 /// to be fixed up so that the address points to atom Y's address.
 class COFFReference LLVM_FINAL : public Reference {
 public:
-  COFFReference(Kind kind) : _target(nullptr), _offsetInAtom(0) {
+  explicit COFFReference(Kind kind) : _target(nullptr), _offsetInAtom(0) {
     _kind = kind;
   }
 
index 61d864f..41be72d 100644 (file)
@@ -351,7 +351,8 @@ private:
 
 class ReaderCOFF : public Reader {
 public:
-  ReaderCOFF(const TargetInfo &ti) : Reader(ti), _readerArchive(ti, *this) {}
+  explicit ReaderCOFF(const TargetInfo &ti)
+      : Reader(ti), _readerArchive(ti, *this) {}
 
   error_code parseFile(std::unique_ptr<MemoryBuffer> &mb,
                        std::vector<std::unique_ptr<File> > &result) const {
index 055f925..f30dcd7 100644 (file)
@@ -67,7 +67,7 @@ public:
     kindSection
   };
 
-  Chunk(Kind kind) : _kind(kind), _size(0), _align(1) {}
+  explicit Chunk(Kind kind) : _kind(kind), _size(0), _align(1) {}
   virtual ~Chunk() {};
   virtual void write(uint8_t *fileBuffer) = 0;
 
@@ -131,7 +131,7 @@ private:
 /// A PEHeaderChunk represents PE header including COFF header.
 class PEHeaderChunk : public HeaderChunk {
 public:
-  PEHeaderChunk(const PECOFFTargetInfo &targetInfo) : HeaderChunk() {
+  explicit PEHeaderChunk(const PECOFFTargetInfo &targetInfo) : HeaderChunk() {
     // Set the size of the chunk and initialize the header with null bytes.
     _size = sizeof(llvm::COFF::PEMagic) + sizeof(_coffHeader)
         + sizeof(_peHeader);
@@ -585,7 +585,7 @@ private:
   }
 
 public:
-  ExecutableWriter(const PECOFFTargetInfo &targetInfo)
+  explicit ExecutableWriter(const PECOFFTargetInfo &targetInfo)
       : _PECOFFTargetInfo(targetInfo) {}
 
   // Create all chunks that consist of the output file.