Implement File API

include/AdblockPlus/FileSystem.h

-#ifndef ADBLOCKPLUS_FILE_READER_H
-#define ADBLOCKPLUS_FILE_READER_H
+#ifndef ADBLOCKPLUS_FILE_SYSTEM_H
+#define ADBLOCKPLUS_FILE_SYSTEM_H
 
 #include <istream>
 #include <string>
 #include <memory>
 
 namespace AdblockPlus
 {
-  class FileReader
+  class FileSystem
   {
   public:
-    virtual ~FileReader();
+    struct StatResult
+    {
+      bool exists;
+      bool isDirectory;
+      bool isFile;
+      int lastModified;
+    };
+
+    virtual ~FileSystem() {}
     virtual std::auto_ptr<std::istream> Read(const std::string& path) const = 0;

[Wladimir Palant, 2013/04/12 16:10:35]

I guess the reason for not returning std::string here is memory efficiency?

[Felix Dahlke, 2013/04/15 03:43:34]

It's probably not really needed, as the whole string is slurped later anyway.
Would make things easier, any objections to making this return string?

+    virtual void Write(const std::string& path, const std::string& content) = 0;
+    virtual void Move(const std::string& fromPath,
+                      const std::string& toPath) = 0;
+    virtual void Remove(const std::string& path) = 0;
+    virtual StatResult Stat(const std::string& path) const = 0;
   };
 }
 
 #endif