Fix return value of BeginDocument (fixes #103)

pdfRenderer->BeginDocument return true on success so the condition was wrongly inverted. This commit fixes that and uses a bit cleaner code.
rmtheis · May 11, 2017 · 00d093d · 00d093d
1 parent 3bda8b1
commit 00d093d
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/tess-two/jni/com_googlecode_tesseract_android/tessbaseapi.cpp b/tess-two/jni/com_googlecode_tesseract_android/tessbaseapi.cpp
@@ -647,15 +647,11 @@ jboolean Java_com_googlecode_tesseract_android_TessBaseAPI_nativeBeginDocument(J
   const char *c_title = env->GetStringUTFChars(title, NULL);
   tesseract::TessPDFRenderer* pdfRenderer = (tesseract::TessPDFRenderer*) jRenderer;
 
-  jboolean res = JNI_TRUE;
-
-  if (pdfRenderer->BeginDocument(c_title)) {
-    res = JNI_FALSE;
-  }
+  bool res = pdfRenderer->BeginDocument(c_title);
 
   env->ReleaseStringUTFChars(title, c_title);
 
-  return res;
+  return (jboolean) (res ? JNI_TRUE : JNI_FALSE);
 }
 
 jboolean Java_com_googlecode_tesseract_android_TessBaseAPI_nativeEndDocument(JNIEnv *env,