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

Unified Diff: jni/Utils.cpp

Issue 5697499218051072: Usage of new API, cleanups (reduced) (Closed)
Patch Set: Even more review issues fixed. Created April 28, 2014, 10:18 a.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
« no previous file with comments | « jni/Utils.h ('k') | src/com/github/rjeschke/neetutils/Objects.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: jni/Utils.cpp
diff --git a/jni/Utils.cpp b/jni/Utils.cpp
index 84e2b531c4d9708f04c293db439479e27520b994..4815a8eec5e9e09693358e058bbd924738fef745 100644
--- a/jni/Utils.cpp
+++ b/jni/Utils.cpp
@@ -18,26 +18,6 @@
#include <string>
#include "Utils.h"
-#include "Debug.h"
-
-const std::string GetString(JNIEnv *pEnv, jstring str)
-{
- D(D_WARN, "getString()");
-
- if (str == NULL)
- return std::string();
-
- jboolean iscopy;
-
- const char *s = pEnv->GetStringUTFChars(str, &iscopy);
- jsize len = pEnv->GetStringUTFLength(str);
-
- const std::string value(s, len);
-
- pEnv->ReleaseStringUTFChars(str, s);
-
- return value;
-}
std::string JniJavaToStdString(JNIEnv* env, jstring str)
{
« no previous file with comments | « jni/Utils.h ('k') | src/com/github/rjeschke/neetutils/Objects.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld