Few minor updates in Android app

TheMostDiligent · TheMostDiligent · commit 3c0abb2fb1ea · 2020-04-25T18:26:08.000-07:00
diff --git a/NativeApp/Android/ndk_helper/src/JNIHelper.cpp b/NativeApp/Android/ndk_helper/src/JNIHelper.cpp
@@ -485,7 +485,7 @@ std::string JNIHelper::GetIntentStringExtra(const std::string& extraDataKey) {
   jstring key = env->NewStringUTF(extraDataKey.c_str());
 
   jstring ret = (jstring)CallObjectMethod(
-      "GetIntentStringExtra", "(Ljava/lang/String;)Ljava/lang/String;", key);
+      "getIntentStringExtra", "(Ljava/lang/String;)Ljava/lang/String;", key);
 
   const char* stringExtra = env->GetStringUTFChars(ret, NULL);
   std::string s = std::string(stringExtra);
diff --git a/NativeApp/Android/ndk_helper/src/java/com/android/helper/NDKHelper.java b/NativeApp/Android/ndk_helper/src/java/com/android/helper/NDKHelper.java
@@ -318,7 +318,7 @@ public String getStringResource(String resourceName)
         return value;
     }
 
-    public String GetIntentStringExtra(String keyName)
+    public String getIntentStringExtra(String keyName)
     {
         String stringExtra = null;
         Intent intent = activity.getIntent();
diff --git a/NativeApp/include/Android/AndroidAppBase.hpp b/NativeApp/include/Android/AndroidAppBase.hpp
@@ -51,12 +51,12 @@ class AndroidAppBase : public AppBase
 
     bool CheckWindowSizeChanged()
     {
-        auto new_window_width_  = ANativeWindow_getWidth(app_->window);
-        auto new_window_height_ = ANativeWindow_getHeight(app_->window);
-        if (new_window_width_ != window_width_ || new_window_height_ != window_height_)
+        auto new_window_width  = ANativeWindow_getWidth(app_->window);
+        auto new_window_height = ANativeWindow_getHeight(app_->window);
+        if (new_window_width != window_width_ || new_window_height != window_height_)
         {
-            window_width_  = new_window_width_;
-            window_height_ = new_window_height_;
+            window_width_  = new_window_width;
+            window_height_ = new_window_height;
             return true;
         }
         else
diff --git a/NativeApp/src/Android/AndroidAppBase.cpp b/NativeApp/src/Android/AndroidAppBase.cpp
@@ -76,7 +76,7 @@ void AndroidAppBase::DrawFrame()
     // before the screen is actually resized. The only robust way
     // to detect window resize is to check it very frame
     if (CheckWindowSizeChanged())
-        WindowResize(0, 0);
+        WindowResize(window_width_, window_height_);
 
     float fFPS;
     if (monitor_.Update(fFPS))
@@ -160,7 +160,6 @@ void AndroidAppBase::HandleCmd(struct android_app* app, int32_t cmd)
             eng->SuspendSensors();
             // Also stop animating.
             eng->has_focus_ = false;
-            eng->DrawFrame();
             break;
 
         case APP_CMD_LOW_MEMORY:

Original file line number	Diff line number	Diff line change
`@@ -318,7 +318,7 @@ public String getStringResource(String resourceName)`
`318`	`318`	`return value;`
`319`	`319`	`}`
`320`	`320`
`321`		`- public String GetIntentStringExtra(String keyName)`
	`321`	`+ public String getIntentStringExtra(String keyName)`
`322`	`322`	`{`
`323`	`323`	`String stringExtra = null;`
`324`	`324`	`Intent intent = activity.getIntent();`
Original file line number	Diff line number	Diff line change
`@@ -51,12 +51,12 @@ class AndroidAppBase : public AppBase`
`51`	`51`
`52`	`52`	`bool CheckWindowSizeChanged()`
`53`	`53`	`{`
`54`		`- auto new_window_width_ = ANativeWindow_getWidth(app_->window);`
`55`		`- auto new_window_height_ = ANativeWindow_getHeight(app_->window);`
`56`		`- if (new_window_width_ != window_width_ \|\| new_window_height_ != window_height_)`
	`54`	`+ auto new_window_width = ANativeWindow_getWidth(app_->window);`
	`55`	`+ auto new_window_height = ANativeWindow_getHeight(app_->window);`
	`56`	`+ if (new_window_width != window_width_ \|\| new_window_height != window_height_)`
`57`	`57`	`{`
`58`		`- window_width_ = new_window_width_;`
`59`		`- window_height_ = new_window_height_;`
	`58`	`+ window_width_ = new_window_width;`
	`59`	`+ window_height_ = new_window_height;`
`60`	`60`	`return true;`
`61`	`61`	`}`
`62`	`62`	`else`