tools/record: use a pointer to the signalfd poll entry
authorPeter Hutterer <peter.hutterer@who-t.net>
Mon, 25 Jan 2021 07:05:00 +0000 (17:05 +1000)
committerPeter Hutterer <peter.hutterer@who-t.net>
Thu, 28 Jan 2021 21:31:46 +0000 (07:31 +1000)
No functional changes, this makes the code slightly more readable, especially
once we start adding more "special" fds.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
tools/libinput-record.c

index 735675084f9801753007aa3b3f4d53c7ddb9cd50..e64f56d56264dbfda89b8350fbbe5cffc71b15fc 100644 (file)
@@ -2115,6 +2115,7 @@ mainloop(struct record_context *ctx)
 {
        bool autorestart = (ctx->timeout > 0);
        struct pollfd fds[ctx->ndevices + 2];
+       struct pollfd *signal_fd = &fds[0];
        unsigned int nfds = 0;
        struct record_device *d = NULL;
        sigset_t mask;
@@ -2135,8 +2136,8 @@ mainloop(struct record_context *ctx)
                };
        }
 
-       fds[0].fd = signalfd(-1, &mask, SFD_NONBLOCK);
-       assert(fds[0].fd != -1);
+       signal_fd->fd = signalfd(-1, &mask, SFD_NONBLOCK);
+       assert(signal_fd->fd != -1);
        nfds++;
 
        if (ctx->libinput) {
@@ -2218,7 +2219,7 @@ mainloop(struct record_context *ctx)
 
                        }
 
-                       if (fds[0].revents != 0) { /* signal */
+                       if (signal_fd->revents != 0) { /* signal */
                                autorestart = false;
                                break;
                        }
@@ -2295,7 +2296,7 @@ mainloop(struct record_context *ctx)
                ctx->output_file = NULL;
        } while (autorestart);
 
-       close(fds[0].fd);
+       close(signal_fd->fd);
 
        sigprocmask(SIG_UNBLOCK, &mask, NULL);