apache · patacongo · Jan 2, 2020 · Jan 2, 2020
diff --git a/builtin/exec_builtin.c b/builtin/exec_builtin.c
@@ -217,15 +217,15 @@ int exec_builtin(FAR const char *appname, FAR char * const *argv,
    * be set appropriately).
    */
 
-  (void)posix_spawn_file_actions_destroy(&file_actions);
-  (void)posix_spawnattr_destroy(&attr);
+  posix_spawn_file_actions_destroy(&file_actions);
+  posix_spawnattr_destroy(&attr);
   return pid;
 
 errout_with_actions:
-  (void)posix_spawn_file_actions_destroy(&file_actions);
+  posix_spawn_file_actions_destroy(&file_actions);
 
 errout_with_attrs:
-  (void)posix_spawnattr_destroy(&attr);
+  posix_spawnattr_destroy(&attr);
 
 errout_with_errno:
   set_errno(ret);

diff --git a/examples/ajoystick/ajoy_main.c b/examples/ajoystick/ajoy_main.c
@@ -203,8 +203,8 @@ static int ajoy_wait(int fd, FAR const struct timespec *timeout)
 
   /* Wait for a signal */
 
-  (void)sigemptyset(&set);
-  (void)sigaddset(&set, CONFIG_EXAMPLES_AJOYSTICK_SIGNO);
+  sigemptyset(&set);
+  sigaddset(&set, CONFIG_EXAMPLES_AJOYSTICK_SIGNO);
   ret = sigtimedwait(&set, &value, timeout);
   if (ret < 0)
     {

diff --git a/examples/alarm/alarm_main.c b/examples/alarm/alarm_main.c
@@ -93,8 +93,8 @@ static int alarm_daemon(int argc, FAR char *argv[])
 
   /* Make sure that the alarm signal is unmasked */
 
-  (void)sigemptyset(&set);
-  (void)sigaddset(&set, CONFIG_EXAMPLES_ALARM_SIGNO);
+  sigemptyset(&set);
+  sigaddset(&set, CONFIG_EXAMPLES_ALARM_SIGNO);
   ret = sigprocmask(SIG_UNBLOCK, &set, NULL);
   if (ret != OK)
     {
@@ -109,8 +109,8 @@ static int alarm_daemon(int argc, FAR char *argv[])
   act.sa_sigaction = alarm_handler;
   act.sa_flags     = SA_SIGINFO;
 
-  (void)sigfillset(&act.sa_mask);
-  (void)sigdelset(&act.sa_mask, CONFIG_EXAMPLES_ALARM_SIGNO);
+  sigfillset(&act.sa_mask);
+  sigdelset(&act.sa_mask, CONFIG_EXAMPLES_ALARM_SIGNO);
 
   ret = sigaction(CONFIG_EXAMPLES_ALARM_SIGNO, &act, NULL);
   if (ret < 0)
@@ -325,7 +325,7 @@ int main(int argc, FAR char *argv[])
           fprintf(stderr, "ERROR: RTC_RD_ALARM ioctl failed: %d\n",
                   errcode);
 
-          (void)close(fd);
+          close(fd);
           return EXIT_FAILURE;
         }
 
@@ -380,7 +380,7 @@ int main(int argc, FAR char *argv[])
           fprintf(stderr, "ERROR: RTC_CANCEL_ALARM ioctl failed: %d\n",
                   errcode);
 
-          (void)close(fd);
+          close(fd);
           return EXIT_FAILURE;
         }
 
@@ -409,13 +409,13 @@ int main(int argc, FAR char *argv[])
           fprintf(stderr, "ERROR: RTC_SET_RELATIVE ioctl failed: %d\n",
                   errcode);
 
-          (void)close(fd);
+          close(fd);
           return EXIT_FAILURE;
         }
 
       printf("Alarm %d set in %lu seconds\n", alarmid, seconds);
     }
 
-  (void)close(fd);
+  close(fd);
   return EXIT_SUCCESS;
 }
diff --git a/examples/buttons/buttons_main.c b/examples/buttons/buttons_main.c
@@ -249,8 +249,8 @@ static int button_daemon(int argc, char *argv[])
 #ifdef CONFIG_EXAMPLES_BUTTONS_SIGNAL
       /* Wait for a signal */
 
-      (void)sigemptyset(&set);
-      (void)sigaddset(&set, CONFIG_EXAMPLES_BUTTONS_SIGNO);
+      sigemptyset(&set);
+      sigaddset(&set, CONFIG_EXAMPLES_BUTTONS_SIGNO);
       ret = sigwaitinfo(&set, &value);
       if (ret < 0)
         {
@@ -366,7 +366,7 @@ static int button_daemon(int argc, char *argv[])
     }
 
 errout_with_fd:
-  (void)close(fd);
+  close(fd);
 
 errout:
   g_button_daemon_started = false;

diff --git a/examples/calib_udelay/calib_udelay_main.c b/examples/calib_udelay/calib_udelay_main.c
@@ -78,7 +78,7 @@ static uint64_t gettime_nsecs(void)
   struct timespec ts;
   uint64_t nsecs;
 
-  (void)clock_gettime(CLOCK_MONOTONIC, &ts);
+  clock_gettime(CLOCK_MONOTONIC, &ts);
 
   nsecs  = ts.tv_sec;
   nsecs *= 1000 * 1000 * 1000;

diff --git a/examples/canard/canard_main.c b/examples/canard/canard_main.c
@@ -233,8 +233,8 @@ static void onTransferReceived(CanardInstance *ins,
                                (uint16_t) total_size);
       if (resp_res <= 0)
         {
-          (void)fprintf(stderr, "Could not respond to GetNodeInfo; error %d\n",
-                        resp_res);
+          fprintf(stderr, "Could not respond to GetNodeInfo; error %d\n",
+                  resp_res);
         }
     }
 }
@@ -257,8 +257,6 @@ static bool shouldAcceptTransfer(const CanardInstance * ins,
                                  CanardTransferType transfer_type,
                                  uint8_t source_node_id)
 {
-  (void)source_node_id;
-
   if (canardGetLocalNodeID(ins) == CANARD_BROADCAST_NODE_ID)
     {
       /* If we're in the process of allocation of dynamic node ID, accept
@@ -335,8 +333,8 @@ void process1HzTasks(uint64_t timestamp_usec)
                       buffer, UAVCAN_NODE_STATUS_MESSAGE_SIZE);
     if (bc_res <= 0)
       {
-        (void)fprintf(stderr, "Could not broadcast node status; error %d\n",
-                      bc_res);
+        fprintf(stderr, "Could not broadcast node status; error %d\n",
+                bc_res);
       }
   }
 
@@ -352,8 +350,8 @@ void process1HzTasks(uint64_t timestamp_usec)
                              payload, 0);
     if (resp_res <= 0)
       {
-        (void)fprintf(stderr, "Could not request GetNodeInfo; error %d\n",
-                      resp_res);
+        fprintf(stderr, "Could not request GetNodeInfo; error %d\n",
+                resp_res);
       }
   }
 
@@ -380,9 +378,9 @@ void processTxRxOnce(CanardNuttXInstance * nuttxcan, int timeout_msec)
       if (tx_res < 0)           /* Failure - drop the frame and report */
         {
           canardPopTxQueue(&canard);
-          (void)fprintf(stderr,
-                        "Transmit error %d, frame dropped, errno '%s'\n",
-                        tx_res, strerror(errno));
+          fprintf(stderr,
+                  "Transmit error %d, frame dropped, errno '%s'\n",
+                  tx_res, strerror(errno));
         }
       else if (tx_res > 0)      /* Success - just drop the frame */
         {
@@ -402,8 +400,8 @@ void processTxRxOnce(CanardNuttXInstance * nuttxcan, int timeout_msec)
 
   if (rx_res < 0)               /* Failure - report */
     {
-      (void)fprintf(stderr, "Receive error %d, errno '%s'\n", rx_res,
-                    strerror(errno));
+      fprintf(stderr, "Receive error %d, errno '%s'\n", rx_res,
+              strerror(errno));
     }
   else if (rx_res > 0)          /* Success - process the frame */
     {

diff --git a/examples/chrono/chrono_main.c b/examples/chrono/chrono_main.c
@@ -186,8 +186,8 @@ static int button_daemon(int argc, char *argv[])
 
       /* Wait for a signal */
 
-      (void)sigemptyset(&set);
-      (void)sigaddset(&set, BUTTON_SIGNO);
+      sigemptyset(&set);
+      sigaddset(&set, BUTTON_SIGNO);
       ret = sigwaitinfo(&set, &value);
       if (ret < 0)
         {
@@ -198,7 +198,7 @@ static int button_daemon(int argc, char *argv[])
 
       if (priv->state == CHRONO_STOPPED)
         {
-          (void)clock_gettime(CLOCK_MONOTONIC, &priv->ts_start);
+          clock_gettime(CLOCK_MONOTONIC, &priv->ts_start);
           priv->state = CHRONO_RUNNING;
         }
       else
@@ -214,7 +214,7 @@ static int button_daemon(int argc, char *argv[])
     }
 
 errout_with_fd:
-  (void)close(fd);
+  close(fd);
 
 errout:
 
@@ -284,7 +284,7 @@ static void slcd_putc(FAR struct lib_outstream_s *stream, int ch)
 
   if (stream->nput >= CONFIG_EXAMPLES_SLCD_BUFSIZE)
     {
-      (void)slcd_flush(stream);
+      slcd_flush(stream);
     }
 }
 
@@ -427,7 +427,7 @@ int main(int argc, FAR char *argv[])
         {
           /* Get the current time */
 
-          (void)clock_gettime(CLOCK_MONOTONIC, &priv->ts_end);
+          clock_gettime(CLOCK_MONOTONIC, &priv->ts_end);
 
           /* How many seconds passed from initial time? */
 

diff --git a/examples/configdata/configdata_main.c b/examples/configdata/configdata_main.c
@@ -179,7 +179,7 @@ static void configdata_loopmemusage(void)
 #ifdef CONFIG_CAN_PASS_STRUCTS
   g_mmafter = mallinfo();
 #else
-  (void)mallinfo(&g_mmafter);
+  mallinfo(&g_mmafter);
 #endif
 
   /* Show the change from the previous loop */
@@ -206,7 +206,7 @@ static void configdata_endmemusage(void)
 #ifdef CONFIG_CAN_PASS_STRUCTS
   g_mmafter = mallinfo();
 #else
-  (void)mallinfo(&g_mmafter);
+  mallinfo(&g_mmafter);
 #endif
   printf("\nFinal memory usage:\n");
   configdata_showmemusage(&g_mmbefore, &g_mmafter);
@@ -682,7 +682,7 @@ int main(int argc, FAR char *argv[])
 #ifdef CONFIG_CAN_PASS_STRUCTS
   g_mmbefore = mallinfo();
 #else
-  (void)mallinfo(&g_mmbefore);
+  mallinfo(&g_mmbefore);
 #endif
 
   /* Loop seveal times ... create some config data items, delete them
@@ -706,7 +706,7 @@ int main(int argc, FAR char *argv[])
 #ifndef CONFIG_EXAMPLES_CONFIGDATA_SILENT
       printf("\n=== FILLING %u =============================\n", i);
 #endif
-      (void)configdata_fillconfig();
+      configdata_fillconfig();
 #ifndef CONFIG_EXAMPLES_CONFIGDATA_SILENT
       printf("Filled /dev/config\n");
       printf("  Number of entries: %d\n", g_nentries);

diff --git a/examples/discover/discover_main.c b/examples/discover/discover_main.c
@@ -153,7 +153,7 @@ int main(int argc, FAR char *argv[])
   if (handle)
     {
       struct dhcpc_state ds;
-      (void)dhcpc_request(handle, &ds);
+      dhcpc_request(handle, &ds);
       netlib_set_ipv4addr("eth0", &ds.ipaddr);
 
       if (ds.netmask.s_addr != 0)

diff --git a/examples/djoystick/djoy_main.c b/examples/djoystick/djoy_main.c
@@ -201,8 +201,8 @@ int main(int argc, FAR char *argv[])
 
       /* Wait for a signal */
 
-      (void)sigemptyset(&set);
-      (void)sigaddset(&set, CONFIG_EXAMPLES_DJOYSTICK_SIGNO);
+      sigemptyset(&set);
+      sigaddset(&set, CONFIG_EXAMPLES_DJOYSTICK_SIGNO);
       ret = sigwaitinfo(&set, &value);
       if (ret < 0)
         {

diff --git a/examples/elf/elf_main.c b/examples/elf/elf_main.c
@@ -182,7 +182,7 @@ static void mm_update(FAR unsigned int *previous, FAR const char *msg)
 #ifdef CONFIG_CAN_PASS_STRUCTS
   mmcurrent = mallinfo();
 #else
-  (void)mallinfo(&mmcurrent);
+  mallinfo(&mmcurrent);
 #endif
 
   /* Show the change from the previous time */
@@ -207,7 +207,7 @@ static void mm_initmonitor(void)
 #ifdef CONFIG_CAN_PASS_STRUCTS
   mmcurrent = mallinfo();
 #else
-  (void)mallinfo(&mmcurrent);
+  mallinfo(&mmcurrent);
 #endif
 
   g_mminitial = mmcurrent.uordblks;
@@ -362,7 +362,7 @@ int main(int argc, FAR char *argv[])
    * the ROMFS mountpoint.
    */
 
-  (void)setenv("PATH", MOUNTPT, 1);
+  setenv("PATH", MOUNTPT, 1);
 #endif
 
   /* Now exercise every program in the ROMFS file system */

diff --git a/examples/elf/tests/signal/signal.c b/examples/elf/tests/signal/signal.c
@@ -139,7 +139,7 @@ int main(int argc, char **argv)
   act.sa_sigaction = siguser_action;
   act.sa_flags     = SA_SIGINFO;
 
-  (void)sigemptyset(&act.sa_mask);
+  sigemptyset(&act.sa_mask);
 
   status = sigaction(SIGUSR1, &act, &oact1);
   if (status != 0)