dup() -> dup2()
authorFRIGN <dev@frign.de>
Tue, 22 Sep 2015 11:13:25 +0000 (13:13 +0200)
committerFRIGN <dev@frign.de>
Tue, 22 Sep 2015 12:15:24 +0000 (14:15 +0200)
gcc would warn about an unused result. We know it is 0 and dup()
can't fail in these circumstances, as we closed fd0 previously.
Using dup2() to do the same saves one line and shuts gcc up, bringing
us a clean build back.

st.c

diff --git a/st.c b/st.c
index bd8b815..bcf74b3 100644 (file)
--- a/st.c
+++ b/st.c
@@ -1430,8 +1430,7 @@ ttynew(void)
        if (opt_line) {
                if ((cmdfd = open(opt_line, O_RDWR)) < 0)
                        die("open line failed: %s\n", strerror(errno));
-               close(0);
-               dup(cmdfd);
+               dup2(cmdfd, 0);
                stty();
                return;
        }