From ddad09c1f6ee04d9b6a0a5af36407c706b85e3c2 Mon Sep 17 00:00:00 2001
From: Ben Fry <fry@processing.org>
Date: Wed, 21 Jan 2015 18:50:18 -0500
Subject: [PATCH] fix for ^M added to files on Windows (fixes 3014) and
 bulletproofing for save with temp files (fixes 2923)

---
 app/src/processing/app/Base.java | 20 +++++++++++++++-----
 1 file changed, 15 insertions(+), 5 deletions(-)

diff --git a/app/src/processing/app/Base.java b/app/src/processing/app/Base.java
index 71ab1ab9b..3dd879615 100644
--- a/app/src/processing/app/Base.java
+++ b/app/src/processing/app/Base.java
@@ -2499,26 +2499,36 @@ public class Base {
    */
   static public void saveFile(String str, File file) throws IOException {
     File temp = File.createTempFile(file.getName(), null, file.getParentFile());
-    try{
+    try {
       // fix from cjwant to prevent symlinks from being destroyed.
       File canon = file.getCanonicalFile();
+      // assign the var as second step since previous line may throw exception   
       file = canon;
     } catch (IOException e) {
       throw new IOException("Could not resolve canonical representation of " +
                             file.getAbsolutePath());
     }
-    PApplet.saveStrings(temp, new String[] { str });
+    // Can't use saveStrings() here b/c Windows will add a ^M to the file
+    PrintWriter writer = PApplet.createWriter(temp);
+    writer.print(str);
+    boolean error = writer.checkError();  // calls flush()
+    writer.close();  // attempt to close regardless
+    if (error) {
+      throw new IOException("Error while trying to save " + file);
+    }
+    
+    // remove the old file before renaming the temp file
     if (file.exists()) {
       boolean result = file.delete();
       if (!result) {
         throw new IOException("Could not remove old version of " +
-                              file.getAbsolutePath());
+          file.getAbsolutePath());
       }
     }
     boolean result = temp.renameTo(file);
     if (!result) {
-      throw new IOException("Could not replace " +
-                            file.getAbsolutePath());
+      throw new IOException("Could not replace " + file.getAbsolutePath() + 
+                            " with " + temp.getAbsolutePath());
     }
   }