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

Unified Diff: libadblockplus-android/jni/JniEventCallback.cpp

Issue 29422558: Issue 5167 - Update to use libadblockplus revision dca8df9af1a7 (Closed)
Patch Set: optimizations by serge, using newer libadblockplus-binaries Created May 5, 2017, 11:33 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
Index: libadblockplus-android/jni/JniEventCallback.cpp
diff --git a/libadblockplus-android/jni/JniEventCallback.cpp b/libadblockplus-android/jni/JniEventCallback.cpp
index 62733e5f91d8dbefba1ed30d6a8fc544a7033041..1601993c738f359f031d36b8733d79324601a572 100644
--- a/libadblockplus-android/jni/JniEventCallback.cpp
+++ b/libadblockplus-android/jni/JniEventCallback.cpp
@@ -36,7 +36,7 @@ JniEventCallback::JniEventCallback(JNIEnv* env, jobject callbackObject)
{
}
-void JniEventCallback::Callback(AdblockPlus::JsValueList& params)
+void JniEventCallback::Callback(AdblockPlus::JsValueList&& params)
{
JNIEnvAcquire env(GetJavaVM());
@@ -47,7 +47,7 @@ void JniEventCallback::Callback(AdblockPlus::JsValueList& params)
if (method)
{
- jobject jsList = JniJsValueListToArrayList(*env, params);
+ jobject jsList = JniJsValueListToArrayList(*env, std::move(params));
env->CallVoidMethod(GetCallbackObject(), method, jsList);
}
}

Powered by Google App Engine
This is Rietveld