summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorYohann Roussel <yroussel@google.com>2015-07-30 15:14:15 +0200
committerYohann Roussel <yroussel@google.com>2015-08-04 15:41:27 +0200
commit375c7e4d054d04e641b63cfed7b64527a47bb8cb (patch)
treeca968e496be3d7caaf1dafc35a01d9dd31a3ba83
parentea5a352a9d235297960a7d17234658fc015b34a9 (diff)
downloadtoolchain_jack-375c7e4d054d04e641b63cfed7b64527a47bb8cb.zip
toolchain_jack-375c7e4d054d04e641b63cfed7b64527a47bb8cb.tar.gz
toolchain_jack-375c7e4d054d04e641b63cfed7b64527a47bb8cb.tar.bz2
Prevent Jack.check to leak a Config
(Cherry-pick from commit da68d3cf11ed2354878bc7cd97d1c2ef40ddb0bd) Change-Id: I783b3880829f3a5f8abd17d048432330ed782be8
-rw-r--r--jack/src/com/android/jack/Jack.java9
1 files changed, 6 insertions, 3 deletions
diff --git a/jack/src/com/android/jack/Jack.java b/jack/src/com/android/jack/Jack.java
index 554905c..04b172b 100644
--- a/jack/src/com/android/jack/Jack.java
+++ b/jack/src/com/android/jack/Jack.java
@@ -394,7 +394,6 @@ public abstract class Jack {
Config config = options.getConfig();
- ThreadConfig.setConfig(config);
boolean sanityChecks = config.get(Options.SANITY_CHECKS).booleanValue();
@@ -412,12 +411,14 @@ public abstract class Jack {
public static void run(@Nonnull Options options, @Nonnull RunnableHooks hooks)
throws JackUserException, ProcessException {
- Event event = TracerFactory.getTracer().start(JackEventType.JACK_RUN);
+ Event event = null;
try {
Config config = options.getConfig();
ThreadConfig.setConfig(config);
+ event = TracerFactory.getTracer().start(JackEventType.JACK_RUN);
+
ConfigPrinterFactory.getConfigPrinter().printConfig(config);
JSession session = getSession();
@@ -613,8 +614,10 @@ public abstract class Jack {
}
}
} finally {
+ if (event != null) {
+ event.end();
+ }
ThreadConfig.unsetConfig();
- event.end();
}
}