From: Scott Worley <scottworley@scottworley.com>
Date: Sun, 7 May 2017 18:27:58 +0000 (-0700)
Subject: Pull out make_temporary_file
X-Git-Url: http://git.scottworley.com/overonion/commitdiff_plain/2377b838ff45bc3407e0c65028eac858c47c9d4e?hp=a3915755b9992783e291b8921497ab7075557011

Pull out make_temporary_file
---

diff --git a/Makefile b/Makefile
index fe978d3..6e2eacb 100644
--- a/Makefile
+++ b/Makefile
@@ -17,8 +17,8 @@ test: reverse_test
 
 .PHONY: clean distclean depend test
 
-reverse: reverse.o reverse_lib.o
-reverse_test: reverse_test.o reverse_lib.o
+reverse: reverse.o reverse_lib.o temp_file.o
+reverse_test: reverse_test.o reverse_lib.o temp_file.o
 	$(CC) -o $@ $^ $(CFLAGS)
 
 %.o: %.c
diff --git a/reverse_lib.c b/reverse_lib.c
index 20837ee..f217b1c 100644
--- a/reverse_lib.c
+++ b/reverse_lib.c
@@ -1,6 +1,7 @@
-#define _GNU_SOURCE
 #define _FILE_OFFSET_BITS 64
 
+#include "temp_file.h"
+
 #include <err.h>
 #include <fcntl.h>
 #include <stdio.h>
@@ -32,23 +33,6 @@ void reverse_file(const char* input_filename, FILE* output_stream) {
   if (close(fd) == -1) err(EX_IOERR, "Could not close input");
 }
 
-/* Create a temporary file in $TMPDIR, or /tmp if TMPDIR is not set.
- * Caller must free temp_filename and fclose temp_file.  Succeeds or terminates
- * the process. */
-static void make_temporary_file(char** temp_filename, FILE** temp_file) {
-  char* TMPDIR = getenv("TMPDIR");
-  if (TMPDIR == NULL) {
-    TMPDIR = "/tmp";
-  }
-  if (asprintf(temp_filename, "%s/reverse.XXXXXX", TMPDIR) == -1) {
-    errx(EX_OSERR, "Could not assemble temporary filename");
-  }
-  int fd = mkstemp(*temp_filename);
-  if (fd == -1) err(EX_IOERR, "Could not make a temporary file");
-  *temp_file = fdopen(fd, "w");
-  if (*temp_file == NULL) err(EX_IOERR, "Could not open temporary file");
-}
-
 /* Copy data from input to output until EOF is reached. */
 static void copy(FILE* input, FILE* output) {
   for (;;) {
diff --git a/temp_file.c b/temp_file.c
new file mode 100644
index 0000000..68bfb88
--- /dev/null
+++ b/temp_file.c
@@ -0,0 +1,22 @@
+#define _GNU_SOURCE
+
+#include "temp_file.h"
+
+#include <err.h>
+#include <stdlib.h>
+#include <stdio.h>
+#include <sysexits.h>
+
+void make_temporary_file(char** temp_filename, FILE** temp_file) {
+  char* TMPDIR = getenv("TMPDIR");
+  if (TMPDIR == NULL) {
+    TMPDIR = "/tmp";
+  }
+  if (asprintf(temp_filename, "%s/reverse.XXXXXX", TMPDIR) == -1) {
+    errx(EX_OSERR, "Could not assemble temporary filename");
+  }
+  int fd = mkstemp(*temp_filename);
+  if (fd == -1) err(EX_IOERR, "Could not make a temporary file");
+  *temp_file = fdopen(fd, "w");
+  if (*temp_file == NULL) err(EX_IOERR, "Could not open temporary file");
+}
diff --git a/temp_file.h b/temp_file.h
new file mode 100644
index 0000000..d4fe41e
--- /dev/null
+++ b/temp_file.h
@@ -0,0 +1,11 @@
+#ifndef _OVERONION_TEMP_FILE_H
+#define _OVERONION_TEMP_FILE_H
+
+#include <stdio.h>
+
+/* Create a temporary file in $TMPDIR, or /tmp if TMPDIR is not set.
+ * Caller must free temp_filename and fclose temp_file.  Succeeds or terminates
+ * the process. */
+void make_temporary_file(char** temp_filename, FILE** temp_file);
+
+#endif /* _OVERONION_TEMP_FILE_H */