diff options
author | Mikael Peltier <mikaelpeltier@google.com> | 2017-07-28 11:36:56 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2017-07-28 11:36:56 +0000 |
commit | dabd313004429af7249c93872cdbb05d2c6461a2 (patch) | |
tree | 50ec3efed6780fa3c2a2dfdb343129f09209b379 | |
parent | ec3280803c446819aef4c088ccf1e56fe413ffbb (diff) | |
parent | fa101f9d35e4c6ac86f4f8ebfc2bd7257559dd54 (diff) | |
download | r8-dabd313004429af7249c93872cdbb05d2c6461a2.tar.gz |
Merge "Fix potential issues related to CFG printing"
-rw-r--r-- | src/main/java/com/android/tools/r8/D8.java | 11 | ||||
-rw-r--r-- | src/main/java/com/android/tools/r8/R8.java | 11 |
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; |