New JNI bindings

jni/Utils.h

 /*
  * This file is part of Adblock Plus <http://adblockplus.org/>,
  * Copyright (C) 2006-2014 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 10 matching lines @@
 #include <string>
 #include <algorithm>
 #include <jni.h>
 #include <stdexcept>
 
 #include <AdblockPlus.h>
 #include <AdblockPlus/tr1_memory.h>
 
 #include "v8/v8stdint.h"
 
-#define PKG(x) "org/adblockplus/android/api/" x
+#define PKG(x) "org/adblockplus/libadblockplus/" x
 #define TYP(x) "L" PKG(x) ";"
 
 #define ABP_JNI_VERSION JNI_VERSION_1_6
 
 void JniThrowException(JNIEnv* env, const std::string& message);
 
 void JniThrowException(JNIEnv* env, const std::exception& e);
 
 void JniThrowException(JNIEnv* env);
 
 class JNIEnvAcquire
 {
 public:
-  JNIEnvAcquire(JavaVM* javaVM) :

[Felix Dahlke, 2014/03/28 11:27:18]

Colon on next line s'il vous plaît.

-      javaVM(javaVM), jniEnv(0), attachmentStatus(0)
-  {
-    attachmentStatus = javaVM->GetEnv((void **)&jniEnv, ABP_JNI_VERSION);

[Felix Dahlke, 2014/03/28 11:27:18]

Same as JniCallbacks.h, non-trivial non-template functions should be defined in
implementation files.

-    if (attachmentStatus == JNI_EDETACHED)
-    {
-      if (javaVM->AttachCurrentThread(&jniEnv, 0))
-      {
-        // This one is FATAL, we can't recover from this (because without a JVM we're dead), so
-        // throwing a runtime_exception in a ctor can be tolerated here IMHO
-        throw std::runtime_error("Failed to get JNI environment");
-      }
-    }
-  }
+  JNIEnvAcquire(JavaVM* javaVM);
+  ~JNIEnvAcquire();
 
-  ~JNIEnvAcquire()
-  {
-    if (attachmentStatus == JNI_EDETACHED)
-    {
-      javaVM->DetachCurrentThread();
-    }
-  }
-
-  inline JNIEnv* operator*()
+  JNIEnv* operator*()
   {
     return jniEnv;
   }
 
-  inline JNIEnv* operator->()
+  JNIEnv* operator->()
   {
     return jniEnv;
   }
 
 private:
   JavaVM* javaVM;
   JNIEnv* jniEnv;
   int attachmentStatus;
 };
 
 template<typename T>
 class JniGlobalReference
 {
 public:
   JniGlobalReference(JNIEnv* env, T reference)
   {
     env->GetJavaVM(&javaVM);
-    reference = static_cast<T>(env->NewGlobalRef(static_cast<jobject>(reference)));
+    this->reference = static_cast<T>(env->NewGlobalRef(static_cast<jobject>(reference)));
   }
 
   ~JniGlobalReference()
   {
     JNIEnvAcquire env(javaVM);
-
     env->DeleteGlobalRef(static_cast<jobject>(reference));
   }
 
   JniGlobalReference(const JniGlobalReference& other);
   JniGlobalReference& operator=(const JniGlobalReference& other);
 
-  inline T get()
+  T Get()
   {
     return reference;
   }
 
   typedef std::tr1::shared_ptr<JniGlobalReference<T> > Ptr;
 
 private:
   T reference;
   JavaVM* javaVM;
 };
@@ skipping 13 matching lines @@
 // dynamic_casts to shared_ptr<T: JsValue> ... also as the same inheritance is mirrored on the Java
 // side (and Java will throw a class cast exception on error) this shouldn't be an issue (TM)
 template<typename T>
 inline T* JniLongToTypePtr(jlong value)
 {
   return reinterpret_cast<T*>((size_t)value);
 }
 
 jobject NewJniArrayList(JNIEnv* env);
 
-std::string JniJava2StdString(JNIEnv* env, jstring str);
+std::string JniJavaToStdString(JNIEnv* env, jstring str);
 
 void JniAddObjectToList(JNIEnv* env, jobject list, jobject value);
 
 inline std::string JniGetStringField(JNIEnv* env, jclass clazz, jobject jObj, const char* name)
 {
-  return JniJava2StdString(env, reinterpret_cast<jstring>(env->GetObjectField(jObj, env->GetFieldID(clazz, name, "Ljava/lang/String;"))));
+  return JniJavaToStdString(env, reinterpret_cast<jstring>(env->GetObjectField(jObj, env->GetFieldID(clazz, name, "Ljava/lang/String;"))));
 }
 
 inline bool JniGetBooleanField(JNIEnv* env, jclass clazz, jobject jObj, const char* name)
 {
   return env->GetBooleanField(jObj, env->GetFieldID(clazz, name, "Z")) == JNI_TRUE;
 }
 
 inline int32_t JniGetIntField(JNIEnv* env, jclass clazz, jobject jObj, const char* name)
 {
   return (int32_t)env->GetBooleanField(jObj, env->GetFieldID(clazz, name, "I"));
@@ skipping 10 matching lines @@
   jmethodID method = env->GetMethodID(clazz, "<init>", "(J)V");
   return env->NewObject(clazz, method, JniPtrToLong(new AdblockPlus::FilterPtr(filter)));
 }
 
 inline jobject NewJniSubscription(JNIEnv* env, const AdblockPlus::SubscriptionPtr& subscription)
 {
   jclass clazz = env->FindClass(PKG("Subscription"));
   jmethodID method = env->GetMethodID(clazz, "<init>", "(J)V");
   return env->NewObject(clazz, method, JniPtrToLong(new AdblockPlus::SubscriptionPtr(subscription)));
 }
-
-#define TRY try
 
 #define CATCH_AND_THROW(jEnv) \
   catch (const std::exception& except) \
   { \
     JniThrowException(jEnv, except); \
   } \
   catch (...) \
   { \
     JniThrowException(jEnv); \
   }
@@ skipping 16 matching lines @@
 T TrimString(T text)
 {
   // Via http://stackoverflow.com/questions/216823/whats-the-best-way-to-trim-stdstring
   T trimmed(text);
   trimmed.erase(trimmed.begin(), std::find_if(trimmed.begin(), trimmed.end(), std::not1(std::ptr_fun<int, int>(std::isspace))));
   trimmed.erase(std::find_if(trimmed.rbegin(), trimmed.rend(), std::not1(std::ptr_fun<int, int>(std::isspace))).base(), trimmed.end());
   return trimmed;
 }
 
 #endif