Unified Diff: src/AppInfoJsObject.cpp
Issue 29410664:
Issue 5013 - Use const JsValue and pass reference where applicable (Closed)
Base URL: https://hg.adblockplus.org/libadblockplus/
Patch Set:
Created April 12, 2017, 3:24 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« include/AdblockPlus/JsValue.h ('K')
|
« src/AppInfoJsObject.h ('k')
|
src/ConsoleJsObject.h » ('j')
|
src/FilterEngine.cpp » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: src/AppInfoJsObject.cpp |
| =================================================================== |
| --- a/src/AppInfoJsObject.cpp |
| +++ b/src/AppInfoJsObject.cpp |
| @@ -18,18 +18,18 @@ |
| #include <AdblockPlus/AppInfo.h> |
| #include <AdblockPlus/JsValue.h> |
| #include "AppInfoJsObject.h" |
| #include "Utils.h" |
| using namespace AdblockPlus; |
| -JsValuePtr AppInfoJsObject::Setup(const AppInfo& appInfo, const JsValuePtr& obj) |
| +JsValue& AppInfoJsObject::Setup(const AppInfo& appInfo, JsValue& obj) |
| { |
| - obj->SetProperty("version", appInfo.version); |
| - obj->SetProperty("name", appInfo.name); |
| - obj->SetProperty("application", appInfo.application); |
| - obj->SetProperty("applicationVersion", appInfo.applicationVersion); |
| - obj->SetProperty("locale", appInfo.locale); |
| - obj->SetProperty("developmentBuild", appInfo.developmentBuild); |
| + obj.SetProperty("version", appInfo.version); |
| + obj.SetProperty("name", appInfo.name); |
| + obj.SetProperty("application", appInfo.application); |
| + obj.SetProperty("applicationVersion", appInfo.applicationVersion); |
| + obj.SetProperty("locale", appInfo.locale); |
| + obj.SetProperty("developmentBuild", appInfo.developmentBuild); |
| return obj; |
| } |
« include/AdblockPlus/JsValue.h ('K')
|
« src/AppInfoJsObject.h ('k')
|
src/ConsoleJsObject.h » ('j')
|
src/FilterEngine.cpp » ('J')
