Issue 5205 - Add Subscription.isDisabled()

libadblockplus-android/jni/JniSubscription.cpp

Index: libadblockplus-android/jni/JniSubscription.cpp
diff --git a/libadblockplus-android/jni/JniSubscription.cpp b/libadblockplus-android/jni/JniSubscription.cpp
index bbf2c22583d59e2aedbe943ddc51691061e66e00..db216036d4b25169e1b7d4b19590fc00694be31f 100644
--- a/libadblockplus-android/jni/JniSubscription.cpp
+++ b/libadblockplus-android/jni/JniSubscription.cpp
@@ -24,6 +24,24 @@ static AdblockPlus::Subscription* GetSubscriptionPtr(jlong ptr)
   return JniLongToTypePtr<AdblockPlus::Subscription>(ptr);
 }
 
+static jboolean JNICALL JniIsDisabled(JNIEnv* env, jclass clazz, jlong ptr)
+{
+  try
+  {
+    return GetSubscriptionPtr(ptr)->IsDisabled() ? JNI_TRUE : JNI_FALSE;
+  }
+  CATCH_THROW_AND_RETURN(env, JNI_FALSE)
+}
+
+static void JNICALL JniSetDisabled(JNIEnv* env, jclass clazz, jlong ptr, jboolean disabled)
+{
+  try
+  {
+    return GetSubscriptionPtr(ptr)->SetDisabled((bool)(disabled == JNI_TRUE));

[sergei, 2017/05/22 11:23:39]

I doubt that the cast to bool is required here, normally the type of result of
operator== is bool.

+  }
+  CATCH_AND_THROW(env)
+}
+
 static jboolean JNICALL JniIsListed(JNIEnv* env, jclass clazz, jlong ptr)
 {
   try
@@ -92,6 +110,8 @@ static jboolean JNICALL JniIsAcceptableAds(JNIEnv* env, jclass clazz, jlong ptr)
 
 static JNINativeMethod methods[] =
 {
+  { (char*)"isDisabled", (char*)"(J)Z", (void*)JniIsDisabled },
+  { (char*)"setDisabled", (char*)"(JZ)V", (void*)JniSetDisabled },
   { (char*)"isListed", (char*)"(J)Z", (void*)JniIsListed },
   { (char*)"addToList", (char*)"(J)V", (void*)JniAddToList },
   { (char*)"removeFromList", (char*)"(J)V", (void*)JniRemoveFromList },