Changeset 1349 for trunk

Timestamp:

05/06/19 10:11:20 (5 years ago)

Author:

bln4

Message:

Updated comment explaining how FrameIO.CreateFrameset can return null due to being unable to frameset directory.

File:

• trunk/src/org/expeditee/gui/FrameIO.java (modified) (1 diff)

trunk/src/org/expeditee/gui/FrameIO.java

@@ -1746 +1746 @@
                 // If the directory doesnt already exist then create it...
                 if (!dir.exists()) {
-                        // If the directory couldnt be created, then there is something
-                        // wrong... ie. The disk is full.
                         if (!dir.mkdirs()) {
+                                /*
+                                 * If the directory does not exist, but could not be created then there is something wrong.
+                                 * Prior to May 2019 the only known reason for this was because the disk could be full.
+                                 * Since then, we have discovered that null can occur when working with Google file stream.
+                                 * A directory can return false to an existence check, but then fail to create the directory
+                                 * due to it already existing because of sync issues.  While we have not confirmed, then may
+                                 * be the case with other network drives as well.
+                                 */
                                 return null;
                         }