Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: src/DefaultFileSystem.h

Issue 29499630: Issue 4938 - fix race conditions and get rid of hacks related to DefaultFileSystem (Closed) Base URL: https://github.com/adblockplus/libadblockplus.git
Patch Set: Created July 27, 2017, 11:15 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « include/AdblockPlus/JsEngine.h ('k') | src/DefaultFileSystem.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/DefaultFileSystem.h
diff --git a/src/DefaultFileSystem.h b/src/DefaultFileSystem.h
index 19b6abfa9870ec02b40700ce79aa4f511e337f58..a17de3fe5e05e8ba5e6df41bdcb54913d146b4b8 100644
--- a/src/DefaultFileSystem.h
+++ b/src/DefaultFileSystem.h
@@ -19,6 +19,7 @@
#define ADBLOCK_PLUS_DEFAULT_FILE_SYSTEM_H
#include <AdblockPlus/IFileSystem.h>
+#include <AdblockPlus/Scheduler.h>
#ifdef _WIN32
#define PATH_SEPARATOR '\\'
@@ -57,7 +58,7 @@ namespace AdblockPlus
class DefaultFileSystem : public IFileSystem
{
public:
- explicit DefaultFileSystem(std::unique_ptr<DefaultFileSystemSync> syncImpl);
+ explicit DefaultFileSystem(const Scheduler& scheduler, std::unique_ptr<DefaultFileSystemSync> syncImpl);
void Read(const std::string& path,
const ReadCallback& callback) const;
void Write(const std::string& path,
@@ -72,6 +73,7 @@ namespace AdblockPlus
std::string Resolve(const std::string& path) const;
private:
+ Scheduler scheduler;
std::unique_ptr<DefaultFileSystemSync> syncImpl;
};
}
« no previous file with comments | « include/AdblockPlus/JsEngine.h ('k') | src/DefaultFileSystem.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld