diff options
author | Yohann Roussel <yroussel@google.com> | 2017-01-23 14:22:59 +0000 |
---|---|---|
committer | android-build-merger <android-build-merger@google.com> | 2017-01-23 14:22:59 +0000 |
commit | abad28102877b06278c37338ea3070d03f0d8a58 (patch) | |
tree | 9fda846e09accde83d80156315164a043ffffed1 | |
parent | d7fdff1d85e13e401e3665ae0e41b0fcfd4c3f5a (diff) | |
parent | d3f4428cfda7b5a232c041544c1315798af44bc5 (diff) | |
download | multidex-abad28102877b06278c37338ea3070d03f0d8a58.tar.gz |
Check extracted dex only once per usage am: fe10f9fd25 am: 8c46dd3b3f am: 221296887a
am: d3f4428cfd
Change-Id: Idc5e7387402151453f3a4ea75300e9c9c892e9b3
-rw-r--r-- | library/src/android/support/multidex/MultiDex.java | 28 | ||||
-rw-r--r-- | library/src/android/support/multidex/MultiDexExtractor.java | 2 |
2 files changed, 2 insertions, 28 deletions
diff --git a/library/src/android/support/multidex/MultiDex.java b/library/src/android/support/multidex/MultiDex.java index 142f125..fad50d7 100644 --- a/library/src/android/support/multidex/MultiDex.java +++ b/library/src/android/support/multidex/MultiDex.java @@ -156,20 +156,7 @@ public final class MultiDex { File dexDir = getDexDir(context, applicationInfo); List<File> files = MultiDexExtractor.load(context, applicationInfo, dexDir, false); - if (checkValidZipFiles(files)) { - installSecondaryDexes(loader, dexDir, files); - } else { - Log.w(TAG, "Files were not valid zip files. Forcing a reload."); - // Try again, but this time force a reload of the zip file. - files = MultiDexExtractor.load(context, applicationInfo, dexDir, true); - - if (checkValidZipFiles(files)) { - installSecondaryDexes(loader, dexDir, files); - } else { - // Second time didn't work, give up - throw new RuntimeException("Zip files were not valid."); - } - } + installSecondaryDexes(loader, dexDir, files); } } catch (Exception e) { @@ -245,19 +232,6 @@ public final class MultiDex { } /** - * Returns whether all files in the list are valid zip files. If {@code files} is empty, then - * returns true. - */ - private static boolean checkValidZipFiles(List<File> files) { - for (File file : files) { - if (!MultiDexExtractor.verifyZipFile(file)) { - return false; - } - } - return true; - } - - /** * Locates a given field anywhere in the class inheritance hierarchy. * * @param instance an object to search the field into. diff --git a/library/src/android/support/multidex/MultiDexExtractor.java b/library/src/android/support/multidex/MultiDexExtractor.java index 6d09245..998ccec 100644 --- a/library/src/android/support/multidex/MultiDexExtractor.java +++ b/library/src/android/support/multidex/MultiDexExtractor.java @@ -374,7 +374,7 @@ final class MultiDexExtractor { /** * Returns whether the file is a valid zip file. */ - static boolean verifyZipFile(File file) { + private static boolean verifyZipFile(File file) { try { ZipFile zipFile = new ZipFile(file); try { |