]> granicus.if.org Git - taglib/commitdiff
Revert "Removed an ambiguous cast from FileNameHandle to FileName."
authorTsuda Kageyu <tsuda.kageyu@gmail.com>
Fri, 22 Aug 2014 15:01:11 +0000 (00:01 +0900)
committerTsuda Kageyu <tsuda.kageyu@gmail.com>
Fri, 22 Aug 2014 15:04:33 +0000 (00:04 +0900)
This reverts commit 9af7601baee66ed8fa7cac01b5dc272dd2544ee5.

taglib/toolkit/tfilestream.cpp

index 0195302a2cdab3a8f2b5b627672fac9c5fd246a1..93078ae950eb1421fc9a243f1f53cb24f94f93c5 100644 (file)
@@ -85,14 +85,10 @@ namespace
 
 #else   // _WIN32
 
-  class FileNameHandle
+  struct FileNameHandle : public std::string
   {
-  private:
-    std::string name;
-
-  public:
-    FileNameHandle(FileName n) : name(n) {}
-    FileName toFileName() const { return name.c_str(); }
+    FileNameHandle(FileName name) : std::string(name) {}
+    operator FileName () const { return c_str(); }
   };
 
   typedef FILE* FileHandle;
@@ -126,10 +122,12 @@ namespace
 class FileStream::FileStreamPrivate
 {
 public:
-  FileStreamPrivate(const FileName &fileName) :
-    file(InvalidFileHandle),
-    name(fileName),
-    readOnly(true) {}
+  FileStreamPrivate(const FileName &fileName)
+    : file(InvalidFileHandle)
+    , name(fileName)
+    , readOnly(true)
+  {
+  }
 
   FileHandle file;
   FileNameHandle name;
@@ -140,8 +138,8 @@ public:
 // public members
 ////////////////////////////////////////////////////////////////////////////////
 
-FileStream::FileStream(FileName fileName, bool openReadOnly) :
-  d(new FileStreamPrivate(fileName))
+FileStream::FileStream(FileName fileName, bool openReadOnly)
+  d(new FileStreamPrivate(fileName))
 {
   // First try with read / write mode, if that fails, fall back to read only.
 
@@ -158,7 +156,7 @@ FileStream::FileStream(FileName fileName, bool openReadOnly) :
 # ifdef _WIN32
     debug("Could not open file " + fileName.toString());
 # else
-    debug("Could not open file " + String(d->name.toFileName()));
+    debug("Could not open file " + String(static_cast<const char *>(d->name)));
 # endif
   }
 }
@@ -173,11 +171,7 @@ FileStream::~FileStream()
 
 FileName FileStream::name() const
 {
-# ifdef _WIN32
   return d->name;
-# else
-  return d->name.toFileName();
-# endif
 }
 
 ByteVector FileStream::readBlock(ulong length)