commit 8795f34e379f7d29be2ee6cc0b9377106a1d96d7
author Joachim Sauer <jsauer@google.com> Tue Mar 15 14:55:17 2016 +0000
committer Joachim Sauer <jsauer@google.com> Tue Mar 15 15:16:18 2016 +0000
tree e6fc342dfef84ba98c4c3e1dcda42578ea98a494
parent 1ab6e78b23a8cc063c29e67a1d69f487baa1f0dc

Make JarJar abort when it can't read a .class file.

This change avoids future situations where jarjar silently skips
class files that it can't read (as happened with files compiled for
Java 8)

Bug: 27637680
Change-Id: Ic64a087801e29f89c745b04581e42a4489e7b65b

src/main/com/tonicsystems/jarjar/util/JarTransformer.java

diff --git a/src/main/com/tonicsystems/jarjar/util/JarTransformer.java b/src/main/com/tonicsystems/jarjar/util/JarTransformer.java
index 95f1216..53c7de0 100644
--- a/src/main/com/tonicsystems/jarjar/util/JarTransformer.java
+++ b/src/main/com/tonicsystems/jarjar/util/JarTransformer.java
@@ -29,7 +29,9 @@
             try {
                 reader = new ClassReader(struct.data);
             } catch (Exception e) {
-                return true; // TODO?
+                // Android-changed: Made this failure fatal to highlight class version issues.
+                // http://b/27637680
+                throw new RuntimeException("Failed to load " + struct.name, e);
             }
             GetNameClassWriter w = new GetNameClassWriter(ClassWriter.COMPUTE_MAXS);
             reader.accept(transform(w), ClassReader.EXPAND_FRAMES);