comments improved

Author: cgogolin

platform/android/AndroidManifest.xml

@@ -1,8 +1,8 @@
 <?xml version="1.0" encoding="utf-8"?>
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
 	  package="com.cgogolin.penandpdf"
-	  android:versionCode="61"
-	  android:versionName="1.3.3+"
+	  android:versionCode="62"
+	  android:versionName="1.3.4"
 	  android:installLocation="auto">
   <permission android:name="com.cgogolin.penandpdf.LAUNCH_PEN_AND_PDF_FILE_CHOOSER" />
   <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />

platform/android/src/com/cgogolin/penandpdf/PenAndPDFActivity.java

@@ -400,7 +400,7 @@ public void destroyAlertWaiter() {
     @Override
     public void onCreate(Bundle savedInstanceState)
         {
-                //See onSaveInstanceState() for why we do this
+                //Treat the bundle with the SaveInstanceStateManager before calling through to super
             savedInstanceState = SaveInstanceStateManager.recoverBundleIfNecessary(savedInstanceState, getClass().getClassLoader());
 
             super.onCreate(savedInstanceState);
@@ -1976,6 +1976,7 @@ protected void onSaveInstanceState(Bundle outState) { //Called when the app is d
         if(mDocView != null) outState.putParcelable("mDocView", mDocView.onSaveInstanceState());
         outState.putString("latestTextInSearchBox", latestTextInSearchBox);
 
+            //Treat the bundle with the SaveInstanceStateManager before saving it
         SaveInstanceStateManager.saveBundleIfNecessary(outState);
     }        
 

platform/android/src/com/cgogolin/penandpdf/SaveInstanceStateManager.java

@@ -2,13 +2,12 @@
 
 // Partially taken from https://stackoverflow.com/questions/14256809/save-bundle-to-file
 
-/* As the data returned by marshall can not be reliably read under a different
- * platform version we add the incremetal version to the file name. Restoring
- * then simply yields null.
- * see also: https://developer.android.com/reference/android/os/Parcel.html#marshall().
- If the bundle is too large, which can happen because it contains the drawing and drawing history
+/* If the bundle passed to onSaveInstanceState() is too large, which can happen because here it contains the drawing and drawing history,
  * the data is lost and under Android N the app hard crashes with a android.os.TransactionTooLargeException,
- * therefore we instead save the bundle to a file and restore from there in onResume()*/
+ * therefore through this handler we instead save the bundle to a file and restore from there in onResume()
+ * A warning: As the data returned by marshall can not be reliably read under a different
+ * platforms one shouldn't save a marshaled bundle to permanent storage. Here we are only saving to a temporary file wich is deleted when the VM exits and this should be OK.
+ * See also: https://developer.android.com/reference/android/os/Parcel.html#marshall(). */
 
 import android.content.Context;
 import android.os.Bundle;