aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMikael Peltier <mikaelpeltier@google.com>2017-07-28 11:36:56 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2017-07-28 11:36:56 +0000
commitdabd313004429af7249c93872cdbb05d2c6461a2 (patch)
tree50ec3efed6780fa3c2a2dfdb343129f09209b379
parentec3280803c446819aef4c088ccf1e56fe413ffbb (diff)
parentfa101f9d35e4c6ac86f4f8ebfc2bd7257559dd54 (diff)
downloadr8-dabd313004429af7249c93872cdbb05d2c6461a2.tar.gz
Merge "Fix potential issues related to CFG printing"
-rw-r--r--src/main/java/com/android/tools/r8/D8.java11
-rw-r--r--src/main/java/com/android/tools/r8/R8.java11
2 files changed, 16 insertions, 6 deletions
diff --git a/src/main/java/com/android/tools/r8/D8.java b/src/main/java/com/android/tools/r8/D8.java
index af67b4110..6988b0659 100644
--- a/src/main/java/com/android/tools/r8/D8.java
+++ b/src/main/java/com/android/tools/r8/D8.java
@@ -17,7 +17,10 @@ import com.android.tools.r8.utils.CfgPrinter;
import com.android.tools.r8.utils.InternalOptions;
import com.android.tools.r8.utils.ThreadUtils;
import com.android.tools.r8.utils.Timing;
+import java.io.FileOutputStream;
import java.io.IOException;
+import java.io.OutputStreamWriter;
+import java.nio.charset.StandardCharsets;
import java.nio.file.FileAlreadyExistsException;
import java.nio.file.NoSuchFileException;
import java.nio.file.Paths;
@@ -203,9 +206,11 @@ public final class D8 {
if (options.printCfgFile == null || options.printCfgFile.isEmpty()) {
System.out.print(printer.toString());
} else {
- java.io.FileWriter writer = new java.io.FileWriter(options.printCfgFile);
- writer.write(printer.toString());
- writer.close();
+ try (OutputStreamWriter writer = new OutputStreamWriter(
+ new FileOutputStream(options.printCfgFile),
+ StandardCharsets.UTF_8)) {
+ writer.write(printer.toString());
+ }
}
}
return application;
diff --git a/src/main/java/com/android/tools/r8/R8.java b/src/main/java/com/android/tools/r8/R8.java
index 4867b54fa..3f6fcd41f 100644
--- a/src/main/java/com/android/tools/r8/R8.java
+++ b/src/main/java/com/android/tools/r8/R8.java
@@ -45,9 +45,12 @@ import com.android.tools.r8.utils.Timing;
import com.google.common.io.ByteStreams;
import com.google.common.io.Closer;
import java.io.ByteArrayOutputStream;
+import java.io.FileOutputStream;
import java.io.IOException;
import java.io.OutputStream;
+import java.io.OutputStreamWriter;
import java.io.PrintStream;
+import java.nio.charset.StandardCharsets;
import java.nio.file.FileAlreadyExistsException;
import java.nio.file.Files;
import java.nio.file.NoSuchFileException;
@@ -125,9 +128,11 @@ public class R8 {
if (options.printCfgFile == null || options.printCfgFile.isEmpty()) {
System.out.print(printer.toString());
} else {
- java.io.FileWriter writer = new java.io.FileWriter(options.printCfgFile);
- writer.write(printer.toString());
- writer.close();
+ try (OutputStreamWriter writer = new OutputStreamWriter(
+ new FileOutputStream(options.printCfgFile),
+ StandardCharsets.UTF_8)) {
+ writer.write(printer.toString());
+ }
}
}
return application;