fixed Caused by java.lang.OutOfMemoryError

Failed to allocate a 59953368 byte allocation with 16777216 free bytes and 17MB until OOM java.lang.StringFactory.newStringFromChars (StringFactory.java:218) arrow_right com.jecelyin.editor.v2.io.FileReader.read (Unknown Source) com.jecelyin.editor.v2.editor.Document$ReadFileTask.doInBackground (Unknown Source) com.jecelyin.editor.v2.editor.Document$ReadFileTask.doInBackground (Unknown Source) android.os.AsyncTask$2.call (AsyncTask.java) arrow_drop_down java.lang.Thread.run (Thread.java)
tranleduy2000 · Jun 7, 2018 · 9f87398 · 9f87398
1 parent 6ff1cf7
commit 9f87398
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/lib-n-ide/src/main/java/com/jecelyin/editor/v2/editor/Document.java b/lib-n-ide/src/main/java/com/jecelyin/editor/v2/editor/Document.java
@@ -151,7 +151,7 @@ public void onStart() {
 
     @Override
     public SpannableStringBuilder onAsyncReaded(FileReader fileReader, boolean ok) {
-        Editable text = fileReader.getBuffer();
+        CharSequence text = fileReader.getBuffer();
         String firstLine = text.subSequence(0, Math.min(80, text.length())).toString();
 
         ModeProvider modeProvider = ModeProvider.getInstance();

diff --git a/lib-n-ide/src/main/java/com/jecelyin/editor/v2/io/FileReader.java b/lib-n-ide/src/main/java/com/jecelyin/editor/v2/io/FileReader.java
@@ -66,6 +66,8 @@ public boolean read() {
         } catch (Exception e) {
             DLog.e(e);
             return false;
+        } catch (OutOfMemoryError error) {
+            return false;
         }
     }