Implement File API

src/JsEngine.cpp

 #include <AdblockPlus.h>
 #include <sstream>
 
 #include "GlobalJsObject.h"
 #include "Utils.h"
 
 namespace
 {
   v8::Handle<v8::Context> CreateContext(
     AdblockPlus::FileSystem& fileSystem,
@@ skipping 66 matching lines @@
   const v8::Handle<v8::Script> script = CompileScript(source, filename);
   CheckTryCatch(tryCatch);
   v8::Local<v8::Value> result = script->Run();
   CheckTryCatch(tryCatch);
   v8::String::Utf8Value resultString(result);
   return std::string(*resultString);
 }
 
 void AdblockPlus::JsEngine::Load(const std::string& scriptPath)
 {
-  const std::auto_ptr<std::istream> file = fileSystem->Read(scriptPath);
-  if (!file.get() || !*file)

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

I would suspect that !file.get() is the same as !file (IMHO smart pointers
overload that operation).

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

No, auto_ptr doesn't. Surprised me as well.

+  const std::tr1::shared_ptr<std::istream> file = fileSystem->Read(scriptPath);
+  if (!file || !*file)
     throw std::runtime_error("Unable to load script " + scriptPath);
   Evaluate(Utils::Slurp(*file));
 }
 
 void AdblockPlus::JsEngine::Gc()
 {
   while (!v8::V8::IdleNotification());
 }