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

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

Issue 29347012: Issue 4188 - Update libadblockplus-android to use the latest libadblockplus (Closed)
Patch Set: Changes in libadblockplus-android repo Created June 24, 2016, 2:38 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/JniSubscription.cpp
===================================================================
--- a/libadblockplus-android/jni/JniSubscription.cpp
+++ b/libadblockplus-android/jni/JniSubscription.cpp
@@ -23,17 +23,19 @@
{
return JniLongToTypePtr<AdblockPlus::SubscriptionPtr>(ptr)->get();
}
static jlong JNICALL JniCtor(JNIEnv* env, jclass clazz, jlong jsValue)
{
try
{
- return JniPtrToLong(new AdblockPlus::SubscriptionPtr(new AdblockPlus::Subscription(JniGetJsValuePtr(jsValue))));
+ AdblockPlus::JsValue *jsValuePtr = JniGetJsValue(jsValue);
+ return JniPtrToLong(new AdblockPlus::SubscriptionPtr(new AdblockPlus::Subscription(
+ std::move(*jsValuePtr))));
}
CATCH_THROW_AND_RETURN(env, 0)
}
static jboolean JNICALL JniIsListed(JNIEnv* env, jclass clazz, jlong ptr)
{
try
{
« libadblockplus-android/jni/JniFilter.cpp ('K') | « libadblockplus-android/jni/JniFilter.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld