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

Side by Side Diff: libadblockplus-android/jni/Utils.h

Issue 29424615: Issue 4231 - Fix unstable FilterEngineTest.testSetRemoveFilterChangeCallback (Closed)
Patch Set: using method from c++ utils Created April 28, 2017, 10:44 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-2017 eyeo GmbH 3 * Copyright (C) 2006-2017 eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 13 matching lines...) Expand all
24 #include <stdexcept> 24 #include <stdexcept>
25 #include <memory> 25 #include <memory>
26 26
27 #include <AdblockPlus.h> 27 #include <AdblockPlus.h>
28 28
29 #define PKG(x) "org/adblockplus/libadblockplus/" x 29 #define PKG(x) "org/adblockplus/libadblockplus/" x
30 #define TYP(x) "L" PKG(x) ";" 30 #define TYP(x) "L" PKG(x) ";"
31 31
32 #define ABP_JNI_VERSION JNI_VERSION_1_6 32 #define ABP_JNI_VERSION JNI_VERSION_1_6
33 33
34 namespace AdblockPlus
35 {
36 namespace Utils
37 {
38 std::string Slurp(std::istream& stream);
39 }
40 }
41
34 void JniUtils_OnLoad(JavaVM* vm, JNIEnv* env, void* reserved); 42 void JniUtils_OnLoad(JavaVM* vm, JNIEnv* env, void* reserved);
35 43
36 void JniUtils_OnUnload(JavaVM* vm, JNIEnv* env, void* reserved); 44 void JniUtils_OnUnload(JavaVM* vm, JNIEnv* env, void* reserved);
37 45
38 void JniThrowException(JNIEnv* env, const std::string& message); 46 void JniThrowException(JNIEnv* env, const std::string& message);
39 47
40 void JniThrowException(JNIEnv* env, const std::exception& e); 48 void JniThrowException(JNIEnv* env, const std::exception& e);
41 49
42 void JniThrowException(JNIEnv* env); 50 void JniThrowException(JNIEnv* env);
43 51
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
147 // side (and Java will throw a class cast exception on error) this shouldn't be an issue (TM) 155 // side (and Java will throw a class cast exception on error) this shouldn't be an issue (TM)
148 template<typename T> 156 template<typename T>
149 inline T* JniLongToTypePtr(jlong value) 157 inline T* JniLongToTypePtr(jlong value)
150 { 158 {
151 return reinterpret_cast<T*>((size_t)value); 159 return reinterpret_cast<T*>((size_t)value);
152 } 160 }
153 161
154 std::string JniJavaToStdString(JNIEnv* env, jstring str); 162 std::string JniJavaToStdString(JNIEnv* env, jstring str);
155 jstring JniStdStringToJava(JNIEnv* env, std::string str); 163 jstring JniStdStringToJava(JNIEnv* env, std::string str);
156 164
165
sergei 2017/05/22 12:09:07 This line seems unrelated.
anton 2017/05/22 13:04:38 Acknowledged.
157 jmethodID JniGetAddToListMethod(JNIEnv* env, jobject list); 166 jmethodID JniGetAddToListMethod(JNIEnv* env, jobject list);
158 167
159 void JniAddObjectToList(JNIEnv* env, jobject list, jmethodID addMethod, jobject value); 168 void JniAddObjectToList(JNIEnv* env, jobject list, jmethodID addMethod, jobject value);
160 169
161 void JniAddObjectToList(JNIEnv* env, jobject list, jobject value); 170 void JniAddObjectToList(JNIEnv* env, jobject list, jobject value);
162 171
163 inline std::string JniGetStringField(JNIEnv* env, jclass clazz, jobject jObj, co nst char* name) 172 inline std::string JniGetStringField(JNIEnv* env, jclass clazz, jobject jObj, co nst char* name)
164 { 173 {
165 return JniJavaToStdString(env, reinterpret_cast<jstring>(env->GetObjectField(j Obj, env->GetFieldID(clazz, name, "Ljava/lang/String;")))); 174 return JniJavaToStdString(env, reinterpret_cast<jstring>(env->GetObjectField(j Obj, env->GetFieldID(clazz, name, "Ljava/lang/String;"))));
166 } 175 }
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
211 JniThrowException(jEnv, except); \ 220 JniThrowException(jEnv, except); \
212 return retVal; \ 221 return retVal; \
213 } \ 222 } \
214 catch (...) \ 223 catch (...) \
215 { \ 224 { \
216 JniThrowException(jEnv); \ 225 JniThrowException(jEnv); \
217 return retVal; \ 226 return retVal; \
218 } 227 }
219 228
220 #endif 229 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld