Implement File API

include/AdblockPlus/FileSystem.h

Index: include/AdblockPlus/FileSystem.h
===================================================================
rename from include/AdblockPlus/FileReader.h
rename to include/AdblockPlus/FileSystem.h
--- a/include/AdblockPlus/FileReader.h
+++ b/include/AdblockPlus/FileSystem.h
@@ -1,5 +1,5 @@
-#ifndef ADBLOCKPLUS_FILE_READER_H
-#define ADBLOCKPLUS_FILE_READER_H
+#ifndef ADBLOCKPLUS_FILE_SYSTEM_H
+#define ADBLOCKPLUS_FILE_SYSTEM_H
 
 #include <istream>
 #include <string>
@@ -7,11 +7,24 @@
 
 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;
   };
 }