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

Unified Diff: include/AdblockPlus/DefaultFileSystem.h

Issue 29409580: Issue 5013 - Make parameter const ref when applicable. (Closed) Base URL: https://hg.adblockplus.org/libadblockplus/
Patch Set: Created April 11, 2017, 1:58 p.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 | « no previous file | include/AdblockPlus/FileSystem.h » ('j') | include/AdblockPlus/JsEngine.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: include/AdblockPlus/DefaultFileSystem.h
===================================================================
--- a/include/AdblockPlus/DefaultFileSystem.h
+++ b/include/AdblockPlus/DefaultFileSystem.h
@@ -33,17 +33,17 @@
* system's file system.
* All paths are considered relative to the base path, or to the current
* working directory if no base path is set (see `SetBasePath()`).
*/
class DefaultFileSystem : public FileSystem
{
public:
std::shared_ptr<std::istream> Read(const std::string& path) const;
- void Write(const std::string& path, std::shared_ptr<std::istream> data);
+ void Write(const std::string& path, const std::shared_ptr<std::istream>& data);
void Move(const std::string& fromPath, const std::string& toPath);
void Remove(const std::string& path);
StatResult Stat(const std::string& path) const;
std::string Resolve(const std::string& path) const;
/**
* Sets the base path, all paths are considered relative to it.
* @param path Base path.
« no previous file with comments | « no previous file | include/AdblockPlus/FileSystem.h » ('j') | include/AdblockPlus/JsEngine.h » ('J')

Powered by Google App Engine
This is Rietveld