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

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

Issue 29361445: Issue 4399 - Add WebView inheritor with ad blocking (Closed)
Patch Set: renamings, updated README Created Dec. 19, 2016, 3:04 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
Index: libadblockplus-android/jni/JniFilter.cpp
diff --git a/libadblockplus-android/jni/JniFilter.cpp b/libadblockplus-android/jni/JniFilter.cpp
index b3daec386640c00d0805d850f60cbbf2a40c6ce5..636deb45f6a9b45b96d1485359a8b3ab9df0df10 100644
--- a/libadblockplus-android/jni/JniFilter.cpp
+++ b/libadblockplus-android/jni/JniFilter.cpp
@@ -28,7 +28,9 @@ static jlong JNICALL JniCtor(JNIEnv* env, jclass clazz, jlong jsValue)
{
try
{
- return JniPtrToLong(new AdblockPlus::FilterPtr(new AdblockPlus::Filter(JniGetJsValuePtr(jsValue))));
+ AdblockPlus::JsValue *jsValuePtr = JniGetJsValue(jsValue);
+ return JniPtrToLong(new AdblockPlus::FilterPtr(
+ new AdblockPlus::Filter(std::move(*jsValuePtr))));
}
CATCH_THROW_AND_RETURN(env, 0)
}

Powered by Google App Engine
This is Rietveld