fix: use createTempPath in C grammar

atom-community · Mar 22, 2021 · 49d62d3 · 49d62d3
1 parent ccea0bf
commit 49d62d3
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/lib/grammar-utils.js b/lib/grammar-utils.js
@@ -11,6 +11,13 @@ temp.track()
 const GrammarUtils = {
   tempFilesDir: path.join(os.tmpdir(), "atom_script_tempfiles"),
 
+  // Public: Create a temporary path
+  //
+  // Returns the {String} filepath of the new file
+  createTempPath(prefix = "", extension = "") {
+    return temp.path({ dir: this.tempFilesDir, prefix, suffix: extension })
+  },
+
   // Public: Create a temporary file with the provided code
   //
   // * `code`    A {String} containing some code

diff --git a/lib/grammars/c.js b/lib/grammars/c.js
@@ -11,13 +11,14 @@ const options = "-Wall -include stdio.h"
 
 // TODO add windows support
 function CArgs({ filepath }) {
+  const tempOutFile = GrammarUtils.createTempPath("c-", ".out")
   let cmdArgs = ""
   switch (os) {
     case "darwin":
-      cmdArgs = `xcrun clang ${options} -fcolor-diagnostics '${filepath}' -o /tmp/c.out && /tmp/c.out`
+      cmdArgs = `xcrun clang ${options} -fcolor-diagnostics '${filepath}' -o ${tempOutFile} && ${tempOutFile}`
       break
     case "linux":
-      cmdArgs = `cc ${options} '${filepath}' -o /tmp/c.out && /tmp/c.out`
+      cmdArgs = `cc ${options} '${filepath}' -o ${tempOutFile} && ${tempOutFile}`
       break
     default: {
       atom.notifications.addError(`Not support on ${os}`)