protocol, status, title);
+ snprintf(fixed_type, sizeof(fixed_type), type, CONFIG_THTTPD_CHARSET);
+ snprintf(buf, sizeof(buf), "%.20s %d %s\r\n", hc->protocol, status, title);
add_response(hc, buf);
- (void)snprintf(buf, sizeof(buf), "Server: %s\r\n", "thttpd");
+ snprintf(buf, sizeof(buf), "Server: %s\r\n", "thttpd");
add_response(hc, buf);
- (void)snprintf(buf, sizeof(buf), "Content-Type: %s\r\n", fixed_type);
+ snprintf(buf, sizeof(buf), "Content-Type: %s\r\n", fixed_type);
add_response(hc, buf);
- (void)strftime(tmbuf, sizeof(tmbuf), rfc1123fmt, gmtime(&now.tv_sec));
- (void)snprintf(buf, sizeof(buf), "Date: %s\r\n", tmbuf);
+ strftime(tmbuf, sizeof(tmbuf), rfc1123fmt, gmtime(&now.tv_sec));
+ snprintf(buf, sizeof(buf), "Date: %s\r\n", tmbuf);
add_response(hc, buf);
- (void)strftime(tmbuf, sizeof(tmbuf), rfc1123fmt, gmtime(&mod));
- (void)snprintf(buf, sizeof(buf), "Last-Modified: %s\r\n", tmbuf);
+ strftime(tmbuf, sizeof(tmbuf), rfc1123fmt, gmtime(&mod));
+ snprintf(buf, sizeof(buf), "Last-Modified: %s\r\n", tmbuf);
add_response(hc, buf);
add_response(hc, "Accept-Ranges: bytes\r\n");
add_response(hc, "Connection: close\r\n");
@@ -366,42 +366,42 @@ static void send_mime(httpd_conn *hc, int status, const char *title, const char
s100 = status / 100;
if (s100 != 2 && s100 != 3)
{
- (void)snprintf(buf, sizeof(buf), "Cache-Control: no-cache,no-store\r\n");
+ snprintf(buf, sizeof(buf), "Cache-Control: no-cache,no-store\r\n");
add_response(hc, buf);
}
if (encodings[0] != '\0')
{
- (void)snprintf(buf, sizeof(buf), "Content-Encoding: %s\r\n", encodings);
+ snprintf(buf, sizeof(buf), "Content-Encoding: %s\r\n", encodings);
add_response(hc, buf);
}
if (partial_content)
{
- (void)snprintf(buf, sizeof(buf),"Content-Range: bytes %ld-%ld/%ld\r\n",
- (long)hc->range_start, (long)hc->range_end, (long)length);
+ snprintf(buf, sizeof(buf),"Content-Range: bytes %ld-%ld/%ld\r\n",
+ (long)hc->range_start, (long)hc->range_end, (long)length);
add_response(hc, buf);
- (void)snprintf(buf, sizeof(buf),"Content-Length: %ld\r\n",
- (long)(hc->range_end - hc->range_start + 1));
+ snprintf(buf, sizeof(buf),"Content-Length: %ld\r\n",
+ (long)(hc->range_end - hc->range_start + 1));
add_response(hc, buf);
}
else if (length >= 0)
{
- (void)snprintf(buf, sizeof(buf), "Content-Length: %ld\r\n", (long)length);
+ snprintf(buf, sizeof(buf), "Content-Length: %ld\r\n", (long)length);
add_response(hc, buf);
}
#ifdef CONFIG_THTTPD_P3P
- (void)snprintf(buf, sizeof(buf), "P3P: %s\r\n", CONFIG_THTTPD_P3P);
+ snprintf(buf, sizeof(buf), "P3P: %s\r\n", CONFIG_THTTPD_P3P);
add_response(hc, buf);
#endif
#ifdef CONFIG_THTTPD_MAXAGE
expires = now + CONFIG_THTTPD_MAXAGE;
- (void)strftime(expbuf, sizeof(expbuf), rfc1123fmt, gmtime(&expires));
- (void)snprintf(buf, sizeof(buf),
- "Cache-Control: max-age=%d\r\nExpires: %s\r\n",
- CONFIG_THTTPD_MAXAGE, expbuf);
+ strftime(expbuf, sizeof(expbuf), rfc1123fmt, gmtime(&expires));
+ snprintf(buf, sizeof(buf),
+ "Cache-Control: max-age=%d\r\nExpires: %s\r\n",
+ CONFIG_THTTPD_MAXAGE, expbuf);
add_response(hc, buf);
#endif
@@ -424,7 +424,7 @@ static void send_response(httpd_conn *hc, int status, const char *title, const c
send_mime(hc, status, title, "", extraheads, "text/html; charset=%s", (off_t)-1, (time_t)0);
add_response(hc, html_html);
add_response(hc, html_hdtitle);
- (void)snprintf(buf, sizeof(buf), "%d %s", status, title);
+ snprintf(buf, sizeof(buf), "%d %s", status, title);
add_response(hc, buf);
add_response(hc, html_titlehd);
add_response(hc, html_body);
@@ -433,7 +433,7 @@ static void send_response(httpd_conn *hc, int status, const char *title, const c
add_response(hc, html_endhdr2);
defang(arg, defanged, sizeof(defanged));
- (void)snprintf(buf, sizeof(buf), form, defanged);
+ snprintf(buf, sizeof(buf), form, defanged);
add_response(hc, buf);
if (match("**MSIE**", hc->useragent))
@@ -514,7 +514,7 @@ static int send_err_file(httpd_conn *hc, int status, char *title, char *extrahea
buf[nread] = '\0';
add_response(hc, buf);
}
- (void)fclose(fp);
+ fclose(fp);
#ifdef ERR_APPEND_SERVER_INFO
send_response_tail(hc);
@@ -532,7 +532,7 @@ static void send_authenticate(httpd_conn *hc, char *realm)
static char headstr[] = "WWW-Authenticate: Basic realm=\"";
httpd_realloc_str(&header, &maxheader, sizeof(headstr) + strlen(realm) + 3);
- (void)snprintf(header, maxheader, "%s%s\"\r\n", headstr, realm);
+ snprintf(header, maxheader, "%s%s\"\r\n", headstr, realm);
httpd_send_err(hc, 401, err401title, header, err401form, hc->encodedurl);
/* If the request was a POST then there might still be data to be read, so
@@ -698,7 +698,7 @@ static int auth_check2(httpd_conn *hc, char *dirname)
httpd_realloc_str(&authpath, &maxauthpath,
strlen(dirname) + 1 + sizeof(CONFIG_THTTPD_AUTH_FILE));
- (void)snprintf(authpath, maxauthpath, "%s/%s", dirname, CONFIG_THTTPD_AUTH_FILE);
+ snprintf(authpath, maxauthpath, "%s/%s", dirname, CONFIG_THTTPD_AUTH_FILE);
/* Does this directory have an auth file? */
@@ -759,7 +759,7 @@ static int auth_check2(httpd_conn *hc, char *dirname)
httpd_realloc_str(&hc->remoteuser, &hc->maxremoteuser,
strlen(authinfo));
- (void)strcpy(hc->remoteuser, authinfo);
+ strcpy(hc->remoteuser, authinfo);
return 1;
}
else
@@ -816,7 +816,7 @@ static int auth_check2(httpd_conn *hc, char *dirname)
{
/* Yes. */
- (void)fclose(fp);
+ fclose(fp);
/* So is the password right? */
@@ -825,17 +825,17 @@ static int auth_check2(httpd_conn *hc, char *dirname)
/* Ok! */
httpd_realloc_str(&hc->remoteuser, &hc->maxremoteuser, strlen(line));
- (void)strcpy(hc->remoteuser, line);
+ strcpy(hc->remoteuser, line);
/* And cache this user's info for next time. */
httpd_realloc_str(&prevauthpath, &maxprevauthpath, strlen(authpath));
- (void)strcpy(prevauthpath, authpath);
+ strcpy(prevauthpath, authpath);
prevmtime = sb.st_mtime;
httpd_realloc_str(&prevuser, &maxprevuser, strlen(authinfo));
- (void)strcpy(prevuser, authinfo);
+ strcpy(prevuser, authinfo);
httpd_realloc_str(&prevcryp, &maxprevcryp, strlen(cryp));
- (void)strcpy(prevcryp, cryp);
+ strcpy(prevcryp, cryp);
return 1;
}
else
@@ -850,7 +850,7 @@ static int auth_check2(httpd_conn *hc, char *dirname)
/* Didn't find that user. Access denied. */
- (void)fclose(fp);
+ fclose(fp);
send_authenticate(hc, dirname);
return -1;
}
@@ -873,16 +873,16 @@ static void send_dirredirect(httpd_conn *hc)
}
httpd_realloc_str(&location, &maxlocation, strlen(hc->encodedurl) + 2 + strlen(hc->query));
- (void)snprintf(location, maxlocation, "%s/?%s", hc->encodedurl, hc->query);
+ snprintf(location, maxlocation, "%s/?%s", hc->encodedurl, hc->query);
}
else
{
httpd_realloc_str(&location, &maxlocation, strlen(hc->encodedurl) + 1);
- (void)snprintf(location, maxlocation, "%s/", hc->encodedurl);
+ snprintf(location, maxlocation, "%s/", hc->encodedurl);
}
httpd_realloc_str(&header, &maxheader, sizeof(headstr) + strlen(location));
- (void)snprintf(header, maxheader, "%s%s\r\n", headstr, location);
+ snprintf(header, maxheader, "%s%s\r\n", headstr, location);
send_response(hc, 302, err302title, header, err302form, location);
}
@@ -898,17 +898,17 @@ static int httpd_tilde_map1(httpd_conn *hc)
len = strlen(hc->expnfilename) - 1;
httpd_realloc_str(&temp, &maxtemp, len);
- (void)strcpy(temp, &hc->expnfilename[1]);
+ strcpy(temp, &hc->expnfilename[1]);
httpd_realloc_str(&hc->expnfilename, &hc->maxexpnfilename, strlen(prefix) + 1 + len);
- (void)strcpy(hc->expnfilename, prefix);
+ strcpy(hc->expnfilename, prefix);
if (prefix[0] != '\0')
{
- (void)strcat(hc->expnfilename, "/");
+ strcat(hc->expnfilename, "/");
}
- (void)strcat(hc->expnfilename, temp);
+ strcat(hc->expnfilename, temp);
return 1;
}
#endif /* CONFIG_THTTPD_TILDE_MAP1 */
@@ -929,7 +929,7 @@ static int httpd_tilde_map2(httpd_conn *hc)
/* Get the username. */
httpd_realloc_str(&temp, &maxtemp, strlen(hc->expnfilename) - 1);
- (void)strcpy(temp, &hc->expnfilename[1]);
+ strcpy(temp, &hc->expnfilename[1]);
cp = strchr(temp, '/');
if (cp)
@@ -952,11 +952,11 @@ static int httpd_tilde_map2(httpd_conn *hc)
/* Set up altdir. */
httpd_realloc_str(&hc->altdir, &hc->maxaltdir, strlen(pw->pw_dir) + 1 + strlen(postfix));
- (void)strcpy(hc->altdir, pw->pw_dir);
+ strcpy(hc->altdir, pw->pw_dir);
if (postfix[0] != '\0')
{
- (void)strcat(hc->altdir, "/");
- (void)strcat(hc->altdir, postfix);
+ strcat(hc->altdir, "/");
+ strcat(hc->altdir, postfix);
}
alt = expand_filename(hc->altdir, &rest, true);
@@ -966,12 +966,12 @@ static int httpd_tilde_map2(httpd_conn *hc)
}
httpd_realloc_str(&hc->altdir, &hc->maxaltdir, strlen(alt));
- (void)strcpy(hc->altdir, alt);
+ strcpy(hc->altdir, alt);
/* And the filename becomes altdir plus the post-~ part of the original. */
httpd_realloc_str(&hc->expnfilename, &hc->maxexpnfilename, strlen(hc->altdir) + 1 + strlen(cp));
- (void)snprintf(hc->expnfilename, hc->maxexpnfilename, "%s/%s", hc->altdir, cp);
+ snprintf(hc->expnfilename, hc->maxexpnfilename, "%s/%s", hc->altdir, cp);
/* For this type of tilde mapping, we want to defeat vhost mapping. */
@@ -1075,12 +1075,12 @@ static int vhost_map(httpd_conn *hc)
*cp2++ = '/';
}
- (void)strcpy(cp2, hc->vhostname);
+ strcpy(cp2, hc->vhostname);
#else /* VHOST_DIRLEVELS */
httpd_realloc_str(&hc->hostdir, &hc->maxhostdir, strlen(hc->vhostname));
- (void)strcpy(hc->hostdir, hc->vhostname);
+ strcpy(hc->hostdir, hc->vhostname);
#endif /* VHOST_DIRLEVELS */
@@ -1088,11 +1088,11 @@ static int vhost_map(httpd_conn *hc)
len = strlen(hc->expnfilename);
httpd_realloc_str(&tempfilename, &maxtempfilename, len);
- (void)strcpy(tempfilename, hc->expnfilename);
+ strcpy(tempfilename, hc->expnfilename);
httpd_realloc_str(&hc->expnfilename, &hc->maxexpnfilename, strlen(hc->hostdir) + 1 + len);
- (void)strcpy(hc->expnfilename, hc->hostdir);
- (void)strcat(hc->expnfilename, "/");
- (void)strcat(hc->expnfilename, tempfilename);
+ strcpy(hc->expnfilename, hc->hostdir);
+ strcat(hc->expnfilename, "/");
+ strcat(hc->expnfilename, tempfilename);
return 1;
}
#endif
@@ -1129,7 +1129,7 @@ static char *expand_filename(char *path, char **restP, bool tildemapped)
{
checkedlen = strlen(path);
httpd_realloc_str(&checked, &maxchecked, checkedlen);
- (void)strcpy(checked, path);
+ strcpy(checked, path);
/* Trim trailing slashes. */
@@ -1186,7 +1186,7 @@ static char *expand_filename(char *path, char **restP, bool tildemapped)
restlen = strlen(path);
httpd_realloc_str(&rest, &maxrest, restlen+1);
- (void)strcpy(rest, path);
+ strcpy(rest, path);
/* trim trailing slash */
@@ -1212,7 +1212,7 @@ static char *expand_filename(char *path, char **restP, bool tildemapped)
/* Special case for absolute paths. */
httpd_realloc_str(&checked, &maxchecked, checkedlen + 1);
- (void)strncpy(&checked[checkedlen], r, 1);
+ strncpy(&checked[checkedlen], r, 1);
checkedlen += 1;
}
else if (strncmp(r, "..", MAX(i, 2)) == 0)
@@ -1244,7 +1244,7 @@ static char *expand_filename(char *path, char **restP, bool tildemapped)
checked[checkedlen++] = '/';
}
- (void)strncpy(&checked[checkedlen], r, i);
+ strncpy(&checked[checkedlen], r, i);
checkedlen += i;
}
@@ -1287,7 +1287,7 @@ static char *expand_filename(char *path, char **restP, bool tildemapped)
checked[checkedlen++] = '/';
}
- (void)strcpy(&checked[checkedlen], r);
+ strcpy(&checked[checkedlen], r);
checkedlen += restlen;
}
@@ -1301,7 +1301,7 @@ static char *expand_filename(char *path, char **restP, bool tildemapped)
*restP = r;
if (checked[0] == '\0')
{
- (void)strcpy(checked, httpd_root);
+ strcpy(checked, httpd_root);
}
ninfo("checked: \"%s\"\n", checked);
@@ -1347,19 +1347,19 @@ static void de_dotdot(char *file)
continue;
}
- (void)strcpy(cp + 1, cp2);
+ strcpy(cp + 1, cp2);
}
/* Remove leading ./ and any /./ sequences. */
while (strncmp(file, "./", 2) == 0)
{
- (void)strcpy(file, file + 2);
+ strcpy(file, file + 2);
}
while ((cp = strstr(file, "/./")) != NULL)
{
- (void)strcpy(cp, cp + 2);
+ strcpy(cp, cp + 2);
}
/* Alternate between removing leading ../ and removing xxx/../ */
@@ -1368,7 +1368,7 @@ static void de_dotdot(char *file)
{
while (strncmp(file, "../", 3) == 0)
{
- (void)strcpy(file, file + 3);
+ strcpy(file, file + 3);
}
cp = strstr(file, "/../");
@@ -1382,7 +1382,7 @@ static void de_dotdot(char *file)
continue;
}
- (void)strcpy(cp2 + 1, cp + 4);
+ strcpy(cp2 + 1, cp + 4);
}
/* Also elide any xxx/.. at the end. */
@@ -1528,11 +1528,11 @@ done:
encodings_len + enc_tab[me_indexes[i]].val_len + 1);
if (hc->encodings[0] != '\0')
{
- (void)strcpy(&hc->encodings[encodings_len], ",");
+ strcpy(&hc->encodings[encodings_len], ",");
++encodings_len;
}
- (void)strcpy(&hc->encodings[encodings_len], enc_tab[me_indexes[i]].val);
+ strcpy(&hc->encodings[encodings_len], enc_tab[me_indexes[i]].val);
encodings_len += enc_tab[me_indexes[i]].val_len;
}
}
@@ -1600,11 +1600,11 @@ static void ls_child(int argc, char **argv)
fputs(html_html, fp);
fputs(html_hdtitle, fp);
- (void)fprintf(fp, "Index of %s", hc->encodedurl, hc->encodedurl);
+ fprintf(fp, "Index of %s", hc->encodedurl, hc->encodedurl);
fputs(html_titlehd, fp);
fputs(html_body, fp);
fputs(html_hdr2, fp);
- (void)fprintf(fp, "Index of %s", hc->encodedurl, hc->encodedurl);
+ fprintf(fp, "Index of %s", hc->encodedurl, hc->encodedurl);
fputs(html_endhdr2, fp);
fputs(html_crlf, fp);
fputs("\r\nmode links bytes last-changed name\r\n
", fp);
@@ -1643,7 +1643,7 @@ static void ls_child(int argc, char **argv)
}
namlen = NAMLEN(de);
- (void)strncpy(nameptrs[nnames], de->d_name, namlen);
+ strncpy(nameptrs[nnames], de->d_name, namlen);
nameptrs[nnames][namlen] = '\0';
++nnames;
}
@@ -1667,19 +1667,19 @@ static void ls_child(int argc, char **argv)
if (hc->expnfilename[0] == '\0' || strcmp(hc->expnfilename, ".") == 0)
{
- (void)strcpy(name, nameptrs[i]);
- (void)strcpy(rname, nameptrs[i]);
+ strcpy(name, nameptrs[i]);
+ strcpy(rname, nameptrs[i]);
}
else
{
- (void)snprintf(name, maxname, "%s/%s", hc->expnfilename, nameptrs[i]);
+ snprintf(name, maxname, "%s/%s", hc->expnfilename, nameptrs[i]);
if (strcmp(hc->origfilename, ".") == 0)
{
- (void)snprintf(rname, maxrname, "%s", nameptrs[i]);
+ snprintf(rname, maxrname, "%s", nameptrs[i]);
}
else
{
- (void)snprintf(rname, maxrname, "%s%s", hc->origfilename, nameptrs[i]);
+ snprintf(rname, maxrname, "%s%s", hc->origfilename, nameptrs[i]);
}
}
@@ -1792,17 +1792,17 @@ static void ls_child(int argc, char **argv)
/* And print. */
- (void)fprintf(fp,
- "%s %3ld %10lld %s %s%s%s%s\n",
- modestr, 0, (int16_t)sb.st_size, timestr, encrname,
- S_ISDIR(sb.st_mode) ? "/" : "", nameptrs[i], linkprefix,
- link, fileclass);
+ fprintf(fp,
+ "%s %3ld %10lld %s %s%s%s%s\n",
+ modestr, 0, (int16_t)sb.st_size, timestr, encrname,
+ S_ISDIR(sb.st_mode) ? "/" : "", nameptrs[i], linkprefix,
+ link, fileclass);
}
fputs("
", fp);
fputs(html_endbody, fp);
fputs(html_endhtml, fp);
- (void)fclose(fp);
+ fclose(fp);
exit(0);
}
@@ -2142,7 +2142,7 @@ void httpd_unlisten(httpd_server * hs)
{
if (hs->listen_fd != -1)
{
- (void)close(hs->listen_fd);
+ close(hs->listen_fd);
hs->listen_fd = -1;
}
}
@@ -2162,7 +2162,7 @@ void httpd_write_response(httpd_conn *hc)
if (hc->buflen > 0)
{
- (void)httpd_write(hc->conn_fd, hc->buffer, hc->buflen);
+ httpd_write(hc->conn_fd, hc->buffer, hc->buflen);
hc->buflen = 0;
}
}
@@ -2178,7 +2178,7 @@ void httpd_set_ndelay(int fd)
{
newflags = flags | (int)O_NDELAY;
if (newflags != flags)
- (void)fcntl(fd, F_SETFL, newflags);
+ fcntl(fd, F_SETFL, newflags);
}
}
@@ -2194,7 +2194,7 @@ void httpd_clear_ndelay(int fd)
newflags = flags & ~(int)O_NDELAY;
if (newflags != flags)
{
- (void)fcntl(fd, F_SETFL, newflags);
+ fcntl(fd, F_SETFL, newflags);
}
}
}
@@ -2212,8 +2212,8 @@ void httpd_send_err(httpd_conn *hc, int status, const char *title, const char *e
#ifdef CONFIG_THTTPD_VHOST
if (hc->hostdir[0] != '\0')
{
- (void)snprintf(filename, sizeof(filename),
- "%s/%s/err%d.html", hc->hostdir, CONFIG_THTTPD_ERROR_DIRECTORY, status);
+ snprintf(filename, sizeof(filename),
+ "%s/%s/err%d.html", hc->hostdir, CONFIG_THTTPD_ERROR_DIRECTORY, status);
if (send_err_file(hc, status, title, extraheads, filename))
{
ninfo("Sent VHOST error file\n");
@@ -2224,7 +2224,7 @@ void httpd_send_err(httpd_conn *hc, int status, const char *title, const char *e
/* Try server-wide error page. */
- (void)snprintf(filename, sizeof(filename), "%s/err%d.html", CONFIG_THTTPD_ERROR_DIRECTORY, status);
+ snprintf(filename, sizeof(filename), "%s/err%d.html", CONFIG_THTTPD_ERROR_DIRECTORY, status);
if (send_err_file(hc, status, title, extraheads, filename))
{
ninfo("Sent server-wide error page\n");
@@ -2321,8 +2321,8 @@ int httpd_get_conn(httpd_server *hs, int listen_fd, httpd_conn *hc)
#endif
hc->hs = hs;
- (void)memset(&hc->client_addr, 0, sizeof(hc->client_addr));
- (void)memmove(&hc->client_addr, &sa, sockaddr_len(&sa));
+ memset(&hc->client_addr, 0, sizeof(hc->client_addr));
+ memmove(&hc->client_addr, &sa, sockaddr_len(&sa));
hc->read_idx = 0;
hc->checked_idx = 0;
hc->checked_state = CHST_FIRSTWORD;
@@ -2671,7 +2671,7 @@ int httpd_parse_request(httpd_conn *hc)
}
httpd_realloc_str(&hc->reqhost, &hc->maxreqhost, strlen(reqhost));
- (void)strcpy(hc->reqhost, reqhost);
+ strcpy(hc->reqhost, reqhost);
*url = '/';
}
@@ -2706,13 +2706,13 @@ int httpd_parse_request(httpd_conn *hc)
httpd_strdecode(hc->decodedurl, hc->encodedurl);
httpd_realloc_str(&hc->origfilename, &hc->maxorigfilename, strlen(hc->decodedurl));
- (void)strcpy(hc->origfilename, &hc->decodedurl[1]);
+ strcpy(hc->origfilename, &hc->decodedurl[1]);
/* Special case for top-level URL. */
if (hc->origfilename[0] == '\0')
{
- (void)strcpy(hc->origfilename, ".");
+ strcpy(hc->origfilename, ".");
}
/* Extract query string from encoded URL. */
@@ -2722,7 +2722,7 @@ int httpd_parse_request(httpd_conn *hc)
{
++cp;
httpd_realloc_str(&hc->query, &hc->maxquery, strlen(cp));
- (void)strcpy(hc->query, cp);
+ strcpy(hc->query, cp);
/* Remove query from (decoded) origfilename. */
@@ -2796,13 +2796,13 @@ int httpd_parse_request(httpd_conn *hc)
continue;
}
httpd_realloc_str(&hc->accept, &hc->maxaccept, strlen(hc->accept) + 2 + strlen(cp));
- (void)strcat(hc->accept, ", ");
+ strcat(hc->accept, ", ");
}
else
{
httpd_realloc_str(&hc->accept, &hc->maxaccept, strlen(cp));
}
- (void)strcat(hc->accept, cp);
+ strcat(hc->accept, cp);
}
else if (strncasecmp(buf, "Accept-Encoding:", 16) == 0)
{
@@ -2817,13 +2817,13 @@ int httpd_parse_request(httpd_conn *hc)
continue;
}
httpd_realloc_str(&hc->accepte, &hc->maxaccepte, strlen(hc->accepte) + 2 + strlen(cp));
- (void)strcat(hc->accepte, ", ");
+ strcat(hc->accepte, ", ");
}
else
{
httpd_realloc_str(&hc->accepte, &hc->maxaccepte, strlen(cp));
}
- (void)strcpy(hc->accepte, cp);
+ strcpy(hc->accepte, cp);
}
else if (strncasecmp(buf, "Accept-Language:", 16) == 0)
{
@@ -2983,7 +2983,7 @@ int httpd_parse_request(httpd_conn *hc)
httpd_realloc_str(&hc->expnfilename, &hc->maxexpnfilename,
strlen(hc->origfilename));
- (void)strcpy(hc->expnfilename, hc->origfilename);
+ strcpy(hc->expnfilename, hc->origfilename);
/* Tilde mapping. */
@@ -3027,9 +3027,9 @@ int httpd_parse_request(httpd_conn *hc)
}
httpd_realloc_str(&hc->expnfilename, &hc->maxexpnfilename, strlen(cp));
- (void)strcpy(hc->expnfilename, cp);
+ strcpy(hc->expnfilename, cp);
httpd_realloc_str(&hc->pathinfo, &hc->maxpathinfo, strlen(pi));
- (void)strcpy(hc->pathinfo, pi);
+ strcpy(hc->pathinfo, pi);
ninfo("expnfilename: \"%s\" pathinfo: \"%s\"\n", hc->expnfilename, hc->pathinfo);
/* Remove pathinfo stuff from the original filename too. */
@@ -3080,13 +3080,13 @@ void httpd_close_conn(httpd_conn *hc)
{
if (hc->file_fd >= 0)
{
- (void)close(hc->file_fd);
+ close(hc->file_fd);
hc->file_fd = -1;
}
if (hc->conn_fd >= 0)
{
- (void)close(hc->conn_fd);
+ close(hc->conn_fd);
hc->conn_fd = -1;
}
}
@@ -3197,11 +3197,11 @@ int httpd_start_request(httpd_conn *hc, struct timeval *nowP)
{
httpd_realloc_str(&indexname, &maxindexname,
expnlen + 1 + strlen(index_names[i]));
- (void)strcpy(indexname, hc->expnfilename);
+ strcpy(indexname, hc->expnfilename);
indxlen = strlen(indexname);
if (indxlen == 0 || indexname[indxlen - 1] != '/')
{
- (void)strcat(indexname, "/");
+ strcat(indexname, "/");
}
if (strcmp(indexname, "./") == 0)
@@ -3209,7 +3209,7 @@ int httpd_start_request(httpd_conn *hc, struct timeval *nowP)
indexname[0] = '\0';
}
- (void)strcat(indexname, index_names[i]);
+ strcat(indexname, index_names[i]);
if (stat(indexname, &hc->sb) >= 0)
{
goto got_one;
@@ -3278,7 +3278,7 @@ int httpd_start_request(httpd_conn *hc, struct timeval *nowP)
expnlen = strlen(cp);
httpd_realloc_str(&hc->expnfilename, &hc->maxexpnfilename, expnlen);
- (void)strcpy(hc->expnfilename, cp);
+ strcpy(hc->expnfilename, cp);
/* Now, is the index version world-readable or world-executable? */
@@ -3298,11 +3298,11 @@ int httpd_start_request(httpd_conn *hc, struct timeval *nowP)
#ifdef CONFIG_THTTPD_AUTH_FILE
httpd_realloc_str(&dirname, &maxdirname, expnlen);
- (void)strcpy(dirname, hc->expnfilename);
+ strcpy(dirname, hc->expnfilename);
cp = strrchr(dirname, '/');
if (!cp)
{
- (void)strcpy(dirname, httpd_root);
+ strcpy(dirname, httpd_root);
}
else
{
@@ -3439,7 +3439,7 @@ char *httpd_ntoa(httpd_sockaddr *saP)
{
/* Elide IPv6ish prefix for IPv4 addresses. */
- (void)strcpy(str, &str[7]);
+ strcpy(str, &str[7]);
}
return str;
diff --git a/netutils/thttpd/tdate_parse.c b/netutils/thttpd/tdate_parse.c
index 6287cbd6f..b0cc837a4 100644
--- a/netutils/thttpd/tdate_parse.c
+++ b/netutils/thttpd/tdate_parse.c
@@ -139,8 +139,8 @@ static int scan_wday(char *str_wday, long *tm_wdayP)
if (!sorted)
{
- (void)qsort(wday_tab, sizeof(wday_tab) / sizeof(struct strlong),
- sizeof(struct strlong), strlong_compare);
+ qsort(wday_tab, sizeof(wday_tab) / sizeof(struct strlong),
+ sizeof(struct strlong), strlong_compare);
sorted = 1;
}
pound_case(str_wday);
@@ -170,8 +170,8 @@ static int scan_mon(char *str_mon, long *tm_monP)
if (!sorted)
{
- (void)qsort(mon_tab, sizeof(mon_tab) / sizeof(struct strlong),
- sizeof(struct strlong), strlong_compare);
+ qsort(mon_tab, sizeof(mon_tab) / sizeof(struct strlong),
+ sizeof(struct strlong), strlong_compare);
sorted = 1;
}
pound_case(str_mon);
@@ -204,7 +204,7 @@ time_t tdate_parse(char *str)
/* Initialize. */
- (void)memset((char *)&tm, 0, sizeof(struct tm));
+ memset((char *)&tm, 0, sizeof(struct tm));
/* Skip initial whitespace ourselves - sscanf is clumsy at this. */
diff --git a/netutils/thttpd/thttpd.c b/netutils/thttpd/thttpd.c
index aa9d6782d..924a7a9c7 100644
--- a/netutils/thttpd/thttpd.c
+++ b/netutils/thttpd/thttpd.c
@@ -759,7 +759,7 @@ int thttpd_main(int argc, char **argv)
/* Main loop */
ninfo("Entering the main loop\n");
- (void)gettimeofday(&tv, NULL);
+ gettimeofday(&tv, NULL);
for (;;)
{
/* Do the fd watch */
@@ -778,7 +778,7 @@ int thttpd_main(int argc, char **argv)
exit(1);
}
- (void)gettimeofday(&tv, NULL);
+ gettimeofday(&tv, NULL);
if (num_ready == 0)
{
diff --git a/netutils/thttpd/thttpd_alloc.c b/netutils/thttpd/thttpd_alloc.c
index bb9501bf2..6c9369fe2 100644
--- a/netutils/thttpd/thttpd_alloc.c
+++ b/netutils/thttpd/thttpd_alloc.c
@@ -90,7 +90,7 @@ void httpd_memstats(void)
#ifdef CONFIG_CAN_PASS_STRUCTS
mm = mallinfo();
#else
- (void)mallinfo(&mm);
+ mallinfo(&mm);
#endif
ninfo("arena: %08x ordblks: %08x mxordblk: %08x uordblks: %08x fordblks: %08x\n",
mm.arena, mm.ordblks, mm.mxordblk, mm.uordblks, mm.fordblks);
diff --git a/netutils/thttpd/thttpd_cgi.c b/netutils/thttpd/thttpd_cgi.c
index fd9335ed2..4aa3730e8 100644
--- a/netutils/thttpd/thttpd_cgi.c
+++ b/netutils/thttpd/thttpd_cgi.c
@@ -199,7 +199,7 @@ static void create_environment(httpd_conn *hc)
setenv("GATEWAY_INTERFACE", "CGI/1.1", TRUE);
setenv("SERVER_PROTOCOL", hc->protocol, TRUE);
- (void)snprintf(buf, sizeof(buf), "%d", (int)CONFIG_THTTPD_PORT);
+ snprintf(buf, sizeof(buf), "%d", (int)CONFIG_THTTPD_PORT);
setenv("SERVER_PORT", buf, TRUE);
setenv("REQUEST_METHOD", httpd_method_str(hc->method), TRUE);
@@ -209,19 +209,19 @@ static void create_environment(httpd_conn *hc)
char *cp2;
size_t l;
- (void)snprintf(buf, sizeof(buf), "/%s", hc->pathinfo);
+ snprintf(buf, sizeof(buf), "/%s", hc->pathinfo);
setenv("PATH_INFO", buf, TRUE);
l = strlen(httpd_root) + strlen(hc->pathinfo) + 1;
cp2 = NEW(char, l);
if (cp2)
{
- (void)snprintf(cp2, l, "%s%s", httpd_root, hc->pathinfo);
+ snprintf(cp2, l, "%s%s", httpd_root, hc->pathinfo);
setenv("PATH_TRANSLATED", cp2, TRUE);
}
}
- (void)snprintf(buf, sizeof(buf), "/%s",strcmp(hc->origfilename, ".") == 0 ? "" : hc->origfilename);
+ snprintf(buf, sizeof(buf), "/%s",strcmp(hc->origfilename, ".") == 0 ? "" : hc->origfilename);
setenv("SCRIPT_NAME", buf, TRUE);
if (hc->query[0] != '\0')
@@ -272,7 +272,7 @@ static void create_environment(httpd_conn *hc)
if (hc->contentlength != -1)
{
- (void)snprintf(buf, sizeof(buf), "%lu", (unsigned long)hc->contentlength);
+ snprintf(buf, sizeof(buf), "%lu", (unsigned long)hc->contentlength);
setenv("CONTENT_LENGTH", buf, TRUE);
}
@@ -417,7 +417,7 @@ static inline int cgi_interpose_input(struct cgi_conn_s *cc)
/* And read up to 2 bytes. */
- (void)read(cc->connfd, cc->inbuf.buffer, CONFIG_THTTPD_CGIINBUFFERSIZE);
+ read(cc->connfd, cc->inbuf.buffer, CONFIG_THTTPD_CGIINBUFFERSIZE);
return 1;
}
return 0;
@@ -494,7 +494,7 @@ static inline int cgi_interpose_output(struct cgi_conn_s *cc)
/* Accumulate more header data */
httpd_realloc_str(&cc->outbuf.buffer, &cc->outbuf.size, cc->outbuf.len + nbytes_read);
- (void)memcpy(&(cc->outbuf.buffer[cc->outbuf.len]), cc->inbuf.buffer, nbytes_read);
+ memcpy(&(cc->outbuf.buffer[cc->outbuf.len]), cc->inbuf.buffer, nbytes_read);
cc->outbuf.len += nbytes_read;
cc->outbuf.buffer[cc->outbuf.len] = '\0';
ninfo("Header bytes accumulated: %d\n", cc->outbuf.len);
@@ -616,12 +616,12 @@ static inline int cgi_interpose_output(struct cgi_conn_s *cc)
break;
}
- (void)snprintf(cc->inbuf.buffer, CONFIG_THTTPD_CGIINBUFFERSIZE, "HTTP/1.0 %d %s\r\n", status, title);
- (void)httpd_write(cc->connfd, cc->inbuf.buffer, strlen(cc->inbuf.buffer));
+ snprintf(cc->inbuf.buffer, CONFIG_THTTPD_CGIINBUFFERSIZE, "HTTP/1.0 %d %s\r\n", status, title);
+ httpd_write(cc->connfd, cc->inbuf.buffer, strlen(cc->inbuf.buffer));
/* Write the saved cc->outbuf.buffer to the client. */
- (void)httpd_write(cc->connfd, cc->outbuf.buffer, cc->outbuf.len);
+ httpd_write(cc->connfd, cc->outbuf.buffer, cc->outbuf.len);
}
/* Then set up to read the data following the header from the CGI program and
@@ -675,7 +675,7 @@ static inline int cgi_interpose_output(struct cgi_conn_s *cc)
{
/* Forward the data from the CGI program to the client */
- (void)httpd_write(cc->connfd, cc->inbuf.buffer, nbytes_read);
+ httpd_write(cc->connfd, cc->inbuf.buffer, nbytes_read);
}
}
break;
@@ -831,7 +831,7 @@ static int cgi_child(int argc, char **argv)
directory = dirname(dupname);
if (directory)
{
- (void)chdir(directory); /* ignore errors */
+ chdir(directory); /* ignore errors */
}
httpd_free(dupname);
@@ -913,7 +913,7 @@ static int cgi_child(int argc, char **argv)
cgi_semgive(); /* Not safe to reference hc after this point */
do
{
- (void)fdwatch(fw, 1000);
+ fdwatch(fw, 1000);
/* Check for incoming data from the remote client to the CGI task */
diff --git a/netutils/thttpd/thttpd_strings.c b/netutils/thttpd/thttpd_strings.c
index c5835852a..ee3a6f67f 100644
--- a/netutils/thttpd/thttpd_strings.c
+++ b/netutils/thttpd/thttpd_strings.c
@@ -168,7 +168,7 @@ void httpd_strencode(char *to, int tosize, char *from)
}
else
{
- (void)sprintf(to, "%%%02x", (int)*from & 0xff);
+ sprintf(to, "%%%02x", (int)*from & 0xff);
to += 3;
tolen += 3;
}
diff --git a/netutils/thttpd/timers.c b/netutils/thttpd/timers.c
index cd97cb3f8..beda4affe 100644
--- a/netutils/thttpd/timers.c
+++ b/netutils/thttpd/timers.c
@@ -217,7 +217,7 @@ Timer *tmr_create(struct timeval *now, TimerProc *timer_proc,
}
else
{
- (void)gettimeofday(&tmr->time, NULL);
+ gettimeofday(&tmr->time, NULL);
}
tmr->time.tv_sec += msecs / 1000L;
diff --git a/netutils/webclient/webclient.c b/netutils/webclient/webclient.c
index 18626b6c6..a7c085092 100644
--- a/netutils/webclient/webclient.c
+++ b/netutils/webclient/webclient.c
@@ -347,9 +347,9 @@ static inline int wget_parseheaders(struct wget_s *ws)
* retain the current location.
*/
- (void)netlib_parsehttpurl(ws->line + strlen(g_httplocation), &ws->port,
- ws->hostname, CONFIG_WEBCLIENT_MAXHOSTNAME,
- ws->filename, CONFIG_WEBCLIENT_MAXFILENAME);
+ netlib_parsehttpurl(ws->line + strlen(g_httplocation), &ws->port,
+ ws->hostname, CONFIG_WEBCLIENT_MAXHOSTNAME,
+ ws->filename, CONFIG_WEBCLIENT_MAXFILENAME);
ninfo("New hostname='%s' filename='%s'\n", ws->hostname, ws->filename);
}
}
@@ -503,10 +503,10 @@ static int wget_base(FAR const char *url, FAR char *buffer, int buflen,
tv.tv_sec = CONFIG_WEBCLIENT_TIMEOUT;
tv.tv_usec = 0;
- (void)setsockopt(sockfd, SOL_SOCKET, SO_RCVTIMEO, (FAR const void *)&tv,
- sizeof(struct timeval));
- (void)setsockopt(sockfd, SOL_SOCKET, SO_SNDTIMEO, (FAR const void *)&tv,
- sizeof(struct timeval));
+ setsockopt(sockfd, SOL_SOCKET, SO_RCVTIMEO, (FAR const void *)&tv,
+ sizeof(struct timeval));
+ setsockopt(sockfd, SOL_SOCKET, SO_SNDTIMEO, (FAR const void *)&tv,
+ sizeof(struct timeval));
/* Get the server address from the host name */
diff --git a/netutils/webserver/httpd.c b/netutils/webserver/httpd.c
index 2ff85fd35..6d36d5136 100644
--- a/netutils/webserver/httpd.c
+++ b/netutils/webserver/httpd.c
@@ -167,9 +167,9 @@ static int httpd_openindex(struct httpd_state *pstate)
# if defined(CONFIG_NETUTILS_HTTPD_INDEX)
if (ret == ERROR && errno == EISDIR)
{
- (void) snprintf(pstate->ht_filename + z,
- sizeof pstate->ht_filename - z, "/%s",
- CONFIG_NETUTILS_HTTPD_INDEX);
+ snprintf(pstate->ht_filename + z,
+ sizeof pstate->ht_filename - z, "/%s",
+ CONFIG_NETUTILS_HTTPD_INDEX);
ret = httpd_open(pstate->ht_filename, &pstate->ht_file);
}
@@ -341,7 +341,7 @@ static int handle_script(struct httpd_state *pstate)
DEBUGASSERT(status >= 0);
UNUSED(status);
- (void)httpd_close(&pstate->ht_file);
+ httpd_close(&pstate->ht_file);
}
else
{
@@ -521,8 +521,8 @@ static int send_headers(struct httpd_state *pstate, int status, int len)
if (len >= 0)
{
- (void)snprintf(contentlen, HTTPD_MAX_CONTENTLEN,
- "Content-Length: %d\r\n", len);
+ snprintf(contentlen, HTTPD_MAX_CONTENTLEN,
+ "Content-Length: %d\r\n", len);
}
else
{
@@ -534,8 +534,8 @@ static int send_headers(struct httpd_state *pstate, int status, int len)
#if defined(CONFIG_NETUTILS_HTTPD_ENABLE_CHUNKED_ENCODING)
/* Turn on chunked encoding */
- (void)snprintf(contentlen, HTTPD_MAX_CONTENTLEN,
- "Transfer-Encoding: chunked\r\n");
+ snprintf(contentlen, HTTPD_MAX_CONTENTLEN,
+ "Transfer-Encoding: chunked\r\n");
pstate->ht_chunked = true;
#endif
}
@@ -593,9 +593,8 @@ static int httpd_senderror(struct httpd_state *pstate, int status)
}
#endif
- (void) snprintf(pstate->ht_filename, sizeof pstate->ht_filename,
- "%s/%d.html",
- CONFIG_NETUTILS_HTTPD_ERRPATH, status);
+ snprintf(pstate->ht_filename, sizeof pstate->ht_filename,
+ "%s/%d.html", CONFIG_NETUTILS_HTTPD_ERRPATH, status);
ret = httpd_openindex(pstate);
@@ -607,7 +606,7 @@ static int httpd_senderror(struct httpd_state *pstate, int status)
if (ret != OK)
{
- (void) snprintf(msg, sizeof msg, "Error %d\n", status);
+ snprintf(msg, sizeof msg, "Error %d\n", status);
ret = send_chunk(pstate, msg, sizeof msg - 1);
}
@@ -621,7 +620,7 @@ static int httpd_senderror(struct httpd_state *pstate, int status)
#endif
#endif
- (void)httpd_close(&pstate->ht_file);
+ httpd_close(&pstate->ht_file);
}
return ret;
@@ -702,7 +701,7 @@ static int httpd_sendfile(struct httpd_state *pstate)
#endif
done:
- (void)httpd_close(&pstate->ht_file);
+ httpd_close(&pstate->ht_file);
return ret;
}
@@ -812,7 +811,7 @@ static inline int httpd_parse(struct httpd_state *pstate)
}
*v = '\0';
- (void) strcpy(pstate->ht_filename, start);
+ strcpy(pstate->ht_filename, start);
state = STATE_HEADER;
break;
@@ -919,11 +918,11 @@ static void *httpd_handler(void *arg)
status = httpd_parse(pstate);
if (status >= 400)
{
- (void)httpd_senderror(pstate, status);
+ httpd_senderror(pstate, status);
}
else
{
- (void) httpd_sendfile(pstate);
+ httpd_sendfile(pstate);
}
#ifndef CONFIG_NETUTILS_HTTPD_KEEPALIVE_DISABLE
@@ -1009,7 +1008,7 @@ static void single_server(uint16_t portno, pthread_startroutine_t handler,
/* Handle the request. This blocks until complete. */
- (void)httpd_handler((FAR void *)acceptsd);
+ httpd_handler((FAR void *)acceptsd);
}
/* Close the sockets */
diff --git a/netutils/webserver/httpd_mmap.c b/netutils/webserver/httpd_mmap.c
index 818c3e643..35de60325 100644
--- a/netutils/webserver/httpd_mmap.c
+++ b/netutils/webserver/httpd_mmap.c
@@ -125,7 +125,7 @@ int httpd_mmap_open(const char *name, struct httpd_fs_file *file)
file->data = mmap(NULL, st.st_size, PROT_READ, MAP_SHARED | MAP_FILE, file->fd, 0);
if (file->data == MAP_FAILED)
{
- (void) close(file->fd);
+ close(file->fd);
return ERROR;
}
diff --git a/nshlib/nsh_altconsole.c b/nshlib/nsh_altconsole.c
index 66c5ffeeb..0d5d6286b 100644
--- a/nshlib/nsh_altconsole.c
+++ b/nshlib/nsh_altconsole.c
@@ -78,11 +78,11 @@ static int nsh_clone_console(FAR struct console_stdio_s *pstate)
/* Close stderr: note we only close stderr if we opened the alternative one */
- (void)fclose(stderr);
+ fclose(stderr);
/* Associate the new opened file descriptor to stderr */
- (void)dup2(fd, 2);
+ dup2(fd, 2);
/* Close the console device that we just opened */
@@ -101,11 +101,11 @@ static int nsh_clone_console(FAR struct console_stdio_s *pstate)
/* Close stdout: note we only close stdout if we opened the alternative one */
- (void)fclose(stdout);
+ fclose(stdout);
/* Associate the new opened file descriptor to stdout */
- (void)dup2(fd, 1);
+ dup2(fd, 1);
/* Close the console device that we just opened */
@@ -202,7 +202,7 @@ static int nsh_wait_inputdev(FAR struct console_stdio_s *pstate,
/* Close stdin: note we only closed stdin if we opened the alternative one */
- (void)fclose(stdin);
+ fclose(stdin);
/* Okay.. we have successfully opened the input device. Did
* we just re-open fd 0?
@@ -212,7 +212,7 @@ static int nsh_wait_inputdev(FAR struct console_stdio_s *pstate,
{
/* No.. Dup the fd to create standard fd 0. stdin should not know. */
- (void)dup2(fd, 0);
+ dup2(fd, 0);
/* Setup the input console */
@@ -281,13 +281,13 @@ int nsh_consolemain(int argc, char *argv[])
/* Execute the one-time start-up script. Any output will go to /dev/console. */
#ifdef CONFIG_NSH_ROMFSETC
- (void)nsh_initscript(&pstate->cn_vtbl);
+ nsh_initscript(&pstate->cn_vtbl);
#endif
#if defined(CONFIG_NSH_ARCHINIT) && defined(CONFIG_BOARDCTL_FINALINIT)
/* Perform architecture-specific final-initialization (if configured) */
- (void)boardctl(BOARDIOC_FINALINIT, 0);
+ boardctl(BOARDIOC_FINALINIT, 0);
#endif
/* First map stderr and stdout to alternative devices */
@@ -315,7 +315,7 @@ int nsh_consolemain(int argc, char *argv[])
/* Execute the session */
- (void)nsh_session(pstate);
+ nsh_session(pstate);
/* We lost the connection. Wait for the keyboard to
* be re-connected.
diff --git a/nshlib/nsh_builtin.c b/nshlib/nsh_builtin.c
index e32838620..1e7d64e71 100644
--- a/nshlib/nsh_builtin.c
+++ b/nshlib/nsh_builtin.c
@@ -140,7 +140,7 @@ int nsh_builtin(FAR struct nsh_vtbl_s *vtbl, FAR const char *cmd,
* SIGINT.
*/
- (void)ioctl(stdout->fs_fd, TIOCSCTTY, ret);
+ ioctl(stdout->fs_fd, TIOCSCTTY, ret);
/* Wait for the application to exit. We did lock the scheduler
* above, but that does not guarantee that the application did not
@@ -201,7 +201,7 @@ int nsh_builtin(FAR struct nsh_vtbl_s *vtbl, FAR const char *cmd,
*/
}
- (void)ioctl(stdout->fs_fd, TIOCSCTTY, -1);
+ ioctl(stdout->fs_fd, TIOCSCTTY, -1);
}
# ifndef CONFIG_NSH_DISABLEBG
else
diff --git a/nshlib/nsh_console.c b/nshlib/nsh_console.c
index 0c24d4c0e..ec9f5f912 100644
--- a/nshlib/nsh_console.c
+++ b/nshlib/nsh_console.c
@@ -313,7 +313,7 @@ static void nsh_consolerelease(FAR struct nsh_vtbl_s *vtbl)
/* Close the console stream */
#ifdef CONFIG_NSH_ALTCONDEV
- (void)fclose(pstate->cn_constream);
+ fclose(pstate->cn_constream);
#endif
#ifdef CONFIG_NSH_VARS
diff --git a/nshlib/nsh_consolemain.c b/nshlib/nsh_consolemain.c
index 052157304..9ed118f5f 100644
--- a/nshlib/nsh_consolemain.c
+++ b/nshlib/nsh_consolemain.c
@@ -87,7 +87,7 @@ int nsh_consolemain(int argc, char *argv[])
#ifdef CONFIG_NSH_ROMFSETC
/* Execute the start-up script */
- (void)nsh_initscript(&pstate->cn_vtbl);
+ nsh_initscript(&pstate->cn_vtbl);
#ifndef CONFIG_NSH_DISABLESCRIPT
/* Reset the option flags */
@@ -105,7 +105,7 @@ int nsh_consolemain(int argc, char *argv[])
#if defined(CONFIG_NSH_ARCHINIT) && defined(CONFIG_BOARDCTL_FINALINIT)
/* Perform architecture-specific final-initialization (if configured) */
- (void)boardctl(BOARDIOC_FINALINIT, 0);
+ boardctl(BOARDIOC_FINALINIT, 0);
#endif
/* Execute the session */
diff --git a/nshlib/nsh_dbgcmds.c b/nshlib/nsh_dbgcmds.c
index 17817a66c..eada7d151 100644
--- a/nshlib/nsh_dbgcmds.c
+++ b/nshlib/nsh_dbgcmds.c
@@ -392,7 +392,7 @@ int cmd_hexdump(FAR struct nsh_vtbl_s *vtbl, int argc, FAR char **argv)
buffer = (FAR uint8_t *)malloc(IOBUFFERSIZE);
if(buffer == NULL)
{
- (void)close(fd);
+ close(fd);
nsh_error(vtbl, g_fmtcmdfailed, "hexdump", "malloc", NSH_ERRNO);
return ERROR;
}
@@ -491,7 +491,7 @@ int cmd_hexdump(FAR struct nsh_vtbl_s *vtbl, int argc, FAR char **argv)
}
}
- (void)close(fd);
+ close(fd);
free(buffer);
return ret;
}
diff --git a/nshlib/nsh_ddcmd.c b/nshlib/nsh_ddcmd.c
index af0f1b3b6..a1a27c970 100644
--- a/nshlib/nsh_ddcmd.c
+++ b/nshlib/nsh_ddcmd.c
@@ -314,9 +314,9 @@ int cmd_dd(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
#ifdef CONFIG_NSH_CMDOPT_DD_STATS
#ifdef CONFIG_CLOCK_MONOTONIC
- (void)clock_gettime(CLOCK_MONOTONIC, &ts0);
+ clock_gettime(CLOCK_MONOTONIC, &ts0);
#else
- (void)clock_gettime(CLOCK_REALTIME, &ts0);
+ clock_gettime(CLOCK_REALTIME, &ts0);
#endif
#endif
@@ -367,9 +367,9 @@ int cmd_dd(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
#ifdef CONFIG_NSH_CMDOPT_DD_STATS
#ifdef CONFIG_CLOCK_MONOTONIC
- (void)clock_gettime(CLOCK_MONOTONIC, &ts1);
+ clock_gettime(CLOCK_MONOTONIC, &ts1);
#else
- (void)clock_gettime(CLOCK_REALTIME, &ts1);
+ clock_gettime(CLOCK_REALTIME, &ts1);
#endif
elapsed = (((uint64_t)ts1.tv_sec * NSEC_PER_SEC) + ts1.tv_nsec);
@@ -385,10 +385,10 @@ int cmd_dd(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
#endif
errout_with_outf:
- (void)close(dd.outfd);
+ close(dd.outfd);
errout_with_inf:
- (void)close(dd.infd);
+ close(dd.infd);
free(dd.buffer);
errout_with_paths:
diff --git a/nshlib/nsh_fileapps.c b/nshlib/nsh_fileapps.c
index ec8b796b8..44b4e8f74 100644
--- a/nshlib/nsh_fileapps.c
+++ b/nshlib/nsh_fileapps.c
@@ -175,7 +175,7 @@ int nsh_fileapp(FAR struct nsh_vtbl_s *vtbl, FAR const char *cmd,
* SIGINT.
*/
- (void)ioctl(stdout->fs_fd, TIOCSCTTY, pid);
+ ioctl(stdout->fs_fd, TIOCSCTTY, pid);
/* Wait for the application to exit. We did lock the scheduler
* above, but that does not guarantee that the application did not
@@ -228,7 +228,7 @@ int nsh_fileapp(FAR struct nsh_vtbl_s *vtbl, FAR const char *cmd,
*/
}
- (void)ioctl(stdout->fs_fd, TIOCSCTTY, -1);
+ ioctl(stdout->fs_fd, TIOCSCTTY, -1);
}
# ifndef CONFIG_NSH_DISABLEBG
else
@@ -270,10 +270,10 @@ int nsh_fileapp(FAR struct nsh_vtbl_s *vtbl, FAR const char *cmd,
*/
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:
/* Most posix_spawn interfaces return a positive errno value on failure
diff --git a/nshlib/nsh_fscmds.c b/nshlib/nsh_fscmds.c
index 190ecdbd5..89db3f08b 100644
--- a/nshlib/nsh_fscmds.c
+++ b/nshlib/nsh_fscmds.c
@@ -819,7 +819,7 @@ int cmd_losetup(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
/* Free resources */
errout_with_fd:
- (void)close(fd);
+ close(fd);
errout_with_paths:
if (loopdev)
@@ -986,7 +986,7 @@ int cmd_losmart(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
/* Free resources */
errout_with_fd:
- (void)close(fd);
+ close(fd);
errout_with_paths:
if (loopdev)
diff --git a/nshlib/nsh_fsutils.c b/nshlib/nsh_fsutils.c
index a8df61f71..931630c82 100644
--- a/nshlib/nsh_fsutils.c
+++ b/nshlib/nsh_fsutils.c
@@ -91,7 +91,7 @@ int nsh_catfile(FAR struct nsh_vtbl_s *vtbl, FAR const char *cmd,
buffer = (FAR char *)malloc(IOBUFFERSIZE);
if(buffer == NULL)
{
- (void)close(fd);
+ close(fd);
nsh_error(vtbl, g_fmtcmdfailed, cmd, "malloc", NSH_ERRNO);
return ERROR;
}
@@ -175,7 +175,7 @@ int nsh_catfile(FAR struct nsh_vtbl_s *vtbl, FAR const char *cmd,
/* Close the input file and return the result */
- (void)close(fd);
+ close(fd);
free(buffer);
return ret;
}
diff --git a/nshlib/nsh_init.c b/nshlib/nsh_init.c
index 4c61adc77..ce6d4097a 100644
--- a/nshlib/nsh_init.c
+++ b/nshlib/nsh_init.c
@@ -85,12 +85,12 @@ void nsh_initialize(void)
#if defined(CONFIG_NSH_READLINE) && defined(CONFIG_READLINE_TABCOMPLETION)
/* Configure the NSH prompt */
- (void)readline_prompt(g_nshprompt);
+ readline_prompt(g_nshprompt);
#ifdef CONFIG_READLINE_HAVE_EXTMATCH
/* Set up for tab completion on NSH commands */
- (void)readline_extmatch(&g_nsh_extmatch);
+ readline_extmatch(&g_nsh_extmatch);
#endif
#endif
@@ -101,12 +101,12 @@ void nsh_initialize(void)
#ifdef CONFIG_NSH_ARCHINIT
/* Perform architecture-specific initialization (if configured) */
- (void)boardctl(BOARDIOC_INIT, 0);
+ boardctl(BOARDIOC_INIT, 0);
#endif
#ifdef CONFIG_NSH_NETINIT
/* Bring up the network */
- (void)netinit_bringup();
+ netinit_bringup();
#endif
}
diff --git a/nshlib/nsh_modcmds.c b/nshlib/nsh_modcmds.c
index 58b395099..671d916d1 100644
--- a/nshlib/nsh_modcmds.c
+++ b/nshlib/nsh_modcmds.c
@@ -181,7 +181,7 @@ int cmd_lsmod(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
datasize ? datasize : "");
}
- (void)fclose(stream);
+ fclose(stream);
return OK;
}
#endif /* CONFIG_FS_PROCFS && !CONFIG_FS_PROCFS_EXCLUDE_MODULE */
diff --git a/nshlib/nsh_netcmds.c b/nshlib/nsh_netcmds.c
index 6a3a254f0..1bf2309c5 100644
--- a/nshlib/nsh_netcmds.c
+++ b/nshlib/nsh_netcmds.c
@@ -187,7 +187,7 @@ typedef int (*nsh_netdev_callback_t)(FAR struct nsh_vtbl_s *vtbl,
!defined(CONFIG_NSH_DISABLE_IFCONFIG)
static inline void net_statistics(FAR struct nsh_vtbl_s *vtbl)
{
- (void)nsh_catfile(vtbl, "ifconfig", CONFIG_NSH_PROC_MOUNTPOINT "/net/stat");
+ nsh_catfile(vtbl, "ifconfig", CONFIG_NSH_PROC_MOUNTPOINT "/net/stat");
}
#else
# define net_statistics(vtbl)
@@ -207,7 +207,7 @@ static int ifconfig_callback(FAR struct nsh_vtbl_s *vtbl, FAR char *devname)
/* Construct the full path to the /proc/net entry for this device */
snprintf(buffer, IFNAMSIZ + 12, CONFIG_NSH_PROC_MOUNTPOINT "/net/%s", devname);
- (void)nsh_catfile(vtbl, "ifconfig", buffer);
+ nsh_catfile(vtbl, "ifconfig", buffer);
return OK;
}
@@ -359,7 +359,7 @@ errout:
static void wget_callback(FAR char **buffer, int offset, int datend,
FAR int *buflen, FAR void *arg)
{
- (void)write((int)((intptr_t)arg), &((*buffer)[offset]), datend - offset);
+ write((int)((intptr_t)arg), &((*buffer)[offset]), datend - offset);
}
#endif
#endif
@@ -411,7 +411,7 @@ static int nsh_foreach_netdev(nsh_netdev_callback_t callback,
/* Close the directory */
- (void)closedir(dir);
+ closedir(dir);
return ret;
}
#endif
@@ -925,7 +925,7 @@ int cmd_ifconfig(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
{
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/nshlib/nsh_parse.c b/nshlib/nsh_parse.c
index 7447f0692..f86c15d94 100644
--- a/nshlib/nsh_parse.c
+++ b/nshlib/nsh_parse.c
@@ -357,7 +357,7 @@ static void nsh_releaseargs(struct cmdarg_s *arg)
if (vtbl->np.np_redirect)
{
- (void)close(arg->fd);
+ close(arg->fd);
}
#endif
@@ -639,7 +639,7 @@ static int nsh_execute(FAR struct nsh_vtbl_s *vtbl,
if (vtbl->np.np_redirect)
{
- (void)nsh_redirect(bkgvtbl, fd, NULL);
+ nsh_redirect(bkgvtbl, fd, NULL);
}
#endif
@@ -681,9 +681,9 @@ static int nsh_execute(FAR struct nsh_vtbl_s *vtbl,
/* Set up the thread attributes */
- (void)pthread_attr_init(&attr);
- (void)pthread_attr_setschedpolicy(&attr, SCHED_NSH);
- (void)pthread_attr_setschedparam(&attr, ¶m);
+ pthread_attr_init(&attr);
+ pthread_attr_setschedpolicy(&attr, SCHED_NSH);
+ pthread_attr_setschedparam(&attr, ¶m);
/* Execute the command as a separate thread at the appropriate priority */
@@ -700,7 +700,7 @@ static int nsh_execute(FAR struct nsh_vtbl_s *vtbl,
* Otherwise, we would have a memory leak.
*/
- (void)pthread_detach(thread);
+ pthread_detach(thread);
nsh_output(vtbl, "%s [%d:%d]\n", argv[0], thread, param.sched_priority);
}
diff --git a/nshlib/nsh_session.c b/nshlib/nsh_session.c
index d06a1c28c..6b5ac5566 100644
--- a/nshlib/nsh_session.c
+++ b/nshlib/nsh_session.c
@@ -120,7 +120,7 @@ int nsh_session(FAR struct console_stdio_s *pstate)
/* Execute the login script */
#ifdef CONFIG_NSH_ROMFSRC
- (void)nsh_loginscript(vtbl);
+ nsh_loginscript(vtbl);
#endif
/* Then enter the command line parsing loop */
@@ -178,7 +178,7 @@ int nsh_session(FAR struct console_stdio_s *pstate)
/* Parse process the command */
- (void)nsh_parse(vtbl, pstate->cn_line);
+ nsh_parse(vtbl, pstate->cn_line);
fflush(pstate->cn_outstream);
}
diff --git a/nshlib/nsh_stdsession.c b/nshlib/nsh_stdsession.c
index ed376a407..64353d189 100644
--- a/nshlib/nsh_stdsession.c
+++ b/nshlib/nsh_stdsession.c
@@ -162,7 +162,7 @@ int nsh_session(FAR struct console_stdio_s *pstate)
/* Parse process the command */
- (void)nsh_parse(vtbl, pstate->cn_line);
+ nsh_parse(vtbl, pstate->cn_line);
}
/* We do not get here, but this is necessary to keep some compilers happy.
diff --git a/nshlib/nsh_syscmds.c b/nshlib/nsh_syscmds.c
index 8d19c4dde..1d00faf59 100644
--- a/nshlib/nsh_syscmds.c
+++ b/nshlib/nsh_syscmds.c
@@ -127,7 +127,7 @@ int cmd_shutdown(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
* reset the board due to some constraints.
*/
- (void)boardctl(BOARDIOC_RESET, EXIT_SUCCESS);
+ boardctl(BOARDIOC_RESET, EXIT_SUCCESS);
}
else
{
@@ -136,7 +136,7 @@ int cmd_shutdown(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
* to power-off the* board due to some constraints.
*/
- (void)boardctl(BOARDIOC_POWEROFF, EXIT_SUCCESS);
+ boardctl(BOARDIOC_POWEROFF, EXIT_SUCCESS);
}
#elif defined(CONFIG_BOARDCTL_RESET)
@@ -157,7 +157,7 @@ int cmd_shutdown(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
* reset the board due to some constraints.
*/
- (void)boardctl(BOARDIOC_RESET, EXIT_SUCCESS);
+ boardctl(BOARDIOC_RESET, EXIT_SUCCESS);
#else
/* Only the reset behavior is supported and we already know that there is
@@ -169,7 +169,7 @@ int cmd_shutdown(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
* off the board due to some constraints.
*/
- (void)boardctl(BOARDIOC_POWEROFF, EXIT_SUCCESS);
+ boardctl(BOARDIOC_POWEROFF, EXIT_SUCCESS);
#endif
/* boarctl() will not return in any case. It if does, it means that
@@ -288,11 +288,11 @@ int cmd_poweroff(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
if (argc > 1)
{
- (void)boardctl(BOARDIOC_POWEROFF, atoi(argv[1]));
+ boardctl(BOARDIOC_POWEROFF, atoi(argv[1]));
}
else
{
- (void)boardctl(BOARDIOC_POWEROFF, EXIT_SUCCESS);
+ boardctl(BOARDIOC_POWEROFF, EXIT_SUCCESS);
}
/* boarctl() will not return in any case. It if does, it means that
@@ -318,11 +318,11 @@ int cmd_reboot(FAR struct nsh_vtbl_s *vtbl, int argc, char **argv)
if (argc > 1)
{
- (void)boardctl(BOARDIOC_RESET, atoi(argv[1]));
+ boardctl(BOARDIOC_RESET, atoi(argv[1]));
}
else
{
- (void)boardctl(BOARDIOC_RESET, EXIT_SUCCESS);
+ boardctl(BOARDIOC_RESET, EXIT_SUCCESS);
}
/* boarctl() will not return in this case. It if does, it means that
diff --git a/nshlib/nsh_system.c b/nshlib/nsh_system.c
index d78438f64..1549bdcdd 100644
--- a/nshlib/nsh_system.c
+++ b/nshlib/nsh_system.c
@@ -83,7 +83,7 @@ int nsh_system(int argc, char *argv[])
/* Parse process the command */
- (void)nsh_parse(vtbl, argv[1]);
+ nsh_parse(vtbl, argv[1]);
#if CONFIG_NFILE_STREAMS > 0
fflush(pstate->cn_outstream);
#endif
diff --git a/nshlib/nsh_telnetd.c b/nshlib/nsh_telnetd.c
index 5311790de..e82f6adf6 100644
--- a/nshlib/nsh_telnetd.c
+++ b/nshlib/nsh_telnetd.c
@@ -133,13 +133,13 @@ static int nsh_telnetmain(int argc, char *argv[])
*/
#if defined(CONFIG_NSH_ROMFSETC) && !defined(CONFIG_NSH_CONSOLE)
- (void)nsh_initscript(vtbl);
+ nsh_initscript(vtbl);
#endif
/* Execute the login script */
#ifdef CONFIG_NSH_ROMFSRC
- (void)nsh_loginscript(vtbl);
+ nsh_loginscript(vtbl);
#endif
/* Then enter the command line parsing loop */
@@ -202,7 +202,7 @@ static int nsh_telnetmain(int argc, char *argv[])
/* Parse process the received Telnet command */
- (void)nsh_parse(vtbl, pstate->cn_line);
+ nsh_parse(vtbl, pstate->cn_line);
fflush(pstate->cn_outstream);
}
diff --git a/nshlib/nsh_usbconsole.c b/nshlib/nsh_usbconsole.c
index 4d15578dc..c21fc2f38 100644
--- a/nshlib/nsh_usbconsole.c
+++ b/nshlib/nsh_usbconsole.c
@@ -77,15 +77,15 @@ static void nsh_configstdio(int fd)
{
/* Make sure the stdin, stdout, and stderr are closed */
- (void)fclose(stdin);
- (void)fclose(stdout);
- (void)fclose(stderr);
+ fclose(stdin);
+ fclose(stdout);
+ fclose(stderr);
/* Dup the fd to create standard fd 0-2 */
- (void)dup2(fd, 0);
- (void)dup2(fd, 1);
- (void)dup2(fd, 2);
+ dup2(fd, 0);
+ dup2(fd, 1);
+ dup2(fd, 2);
/* fdopen to get the stdin, stdout and stderr streams. The following logic depends
* on the fact that the library layer will allocate FILEs in order. And since
@@ -96,9 +96,9 @@ static void nsh_configstdio(int fd)
* fd = 2 is stderr (write-only, append)
*/
- (void)fdopen(0, "r");
- (void)fdopen(1, "a");
- (void)fdopen(2, "a");
+ fdopen(0, "r");
+ fdopen(1, "a");
+ fdopen(2, "a");
}
/****************************************************************************
@@ -217,7 +217,7 @@ restart:
if (nbytes <= 0)
{
- (void)close(fd);
+ close(fd);
goto restart;
}
}
@@ -308,19 +308,19 @@ int nsh_consolemain(int argc, char *argv[])
/* Configure to use /dev/null if we do not have a valid console. */
#ifndef CONFIG_DEV_CONSOLE
- (void)nsh_nullstdio();
+ nsh_nullstdio();
#endif
/* Execute the one-time start-up script (output may go to /dev/null) */
#ifdef CONFIG_NSH_ROMFSETC
- (void)nsh_initscript(&pstate->cn_vtbl);
+ nsh_initscript(&pstate->cn_vtbl);
#endif
#if defined(CONFIG_NSH_ARCHINIT) && defined(CONFIG_BOARDCTL_FINALINIT)
/* Perform architecture-specific final-initialization (if configured) */
- (void)boardctl(BOARDIOC_FINALINIT, 0);
+ boardctl(BOARDIOC_FINALINIT, 0);
#endif
/* Now loop, executing creating a session for each USB connection */
@@ -332,19 +332,18 @@ int nsh_consolemain(int argc, char *argv[])
*/
ret = nsh_waitusbready();
-
- (void)ret; /* Eliminate warning if not used */
+ UNUSED(ret); /* Eliminate warning if not used */
DEBUGASSERT(ret == OK);
/* Execute the session */
- (void)nsh_session(pstate);
+ nsh_session(pstate);
/* Switch to /dev/null because we probably no longer have a
* valid console device.
*/
- (void)nsh_nullstdio();
+ nsh_nullstdio();
}
}
diff --git a/nshlib/nsh_usbtrace.c b/nshlib/nsh_usbtrace.c
index 463732f62..f0ab035f5 100644
--- a/nshlib/nsh_usbtrace.c
+++ b/nshlib/nsh_usbtrace.c
@@ -129,7 +129,7 @@ static int nsh_tracecallback(struct usbtrace_s *trace, void *arg)
void nsh_usbtrace(void)
{
- (void)usbtrace_enumerate(nsh_tracecallback, NULL);
+ usbtrace_enumerate(nsh_tracecallback, NULL);
}
#endif /* CONFIG_NSH_USBDEV_TRACE */
diff --git a/nshlib/nsh_vars.c b/nshlib/nsh_vars.c
index 991647edf..986143ad6 100644
--- a/nshlib/nsh_vars.c
+++ b/nshlib/nsh_vars.c
@@ -255,7 +255,7 @@ int nsh_setvar(FAR struct nsh_vtbl_s *vtbl, FAR const char *name,
* will be added again below.
*/
- (void)nsh_removevar(pstate, pair);
+ nsh_removevar(pstate, pair);
}
/* Get the size of the new name=value string. The +2 is for the '=' and for
@@ -322,7 +322,7 @@ int nsh_unsetvar(FAR struct nsh_vtbl_s *vtbl, FAR const char *name)
{
/* It does! Remove the name=value pair from the NSH variables. */
- (void)nsh_removevar(pstate, pair);
+ nsh_removevar(pstate, pair);
/* Reallocate the new NSH variable buffer */
diff --git a/system/cdcacm/cdcacm_main.c b/system/cdcacm/cdcacm_main.c
index 50b97642f..862656c90 100644
--- a/system/cdcacm/cdcacm_main.c
+++ b/system/cdcacm/cdcacm_main.c
@@ -146,7 +146,7 @@ int serdis_main(int argc, char *argv[])
ctrl.instance = CONFIG_SYSTEM_CDCACM_DEVMINOR;
ctrl.handle = &g_cdcacm.handle;
- (void)boardctl(BOARDIOC_USBDEV_CONTROL, (uintptr_t)&ctrl);
+ boardctl(BOARDIOC_USBDEV_CONTROL, (uintptr_t)&ctrl);
g_cdcacm.handle = NULL;
printf("serdis: Disconnected\n");
return EXIT_SUCCESS;
diff --git a/system/cfgdata/cfgdata_main.c b/system/cfgdata/cfgdata_main.c
index c10d456d5..7e52c87f4 100644
--- a/system/cfgdata/cfgdata_main.c
+++ b/system/cfgdata/cfgdata_main.c
@@ -248,7 +248,6 @@ static void cfgdatacmd_parse_byte_array(struct config_data_s *cfg,
static void cfgdatacmd_set(int argc, char *argv[])
{
- int x;
int ret;
int fd;
struct config_data_s cfg;
@@ -259,9 +258,10 @@ static void cfgdatacmd_set(int argc, char *argv[])
/* Copy the name to the cfg struct */
strncpy(cfg.name, argv[2], CONFIG_MTD_CONFIG_NAME_LEN);
- (void) x;
#else
+ int x;
+
/* Parse the id and instance */
cfg.id = atoi(argv[2]);
@@ -380,7 +380,6 @@ static void cfgdatacmd_set(int argc, char *argv[])
static void cfgdatacmd_unset(int argc, char *argv[])
{
- int x;
int ret;
int fd;
struct config_data_s cfg;
@@ -389,9 +388,10 @@ static void cfgdatacmd_unset(int argc, char *argv[])
/* Copy the name to the cfg struct */
strncpy(cfg.name, argv[2], CONFIG_MTD_CONFIG_NAME_LEN);
- (void) x;
#else
+ int x;
+
/* Parse the id and instance */
cfg.id = atoi(argv[2]);
@@ -444,7 +444,7 @@ static void cfgdatacmd_unset(int argc, char *argv[])
static void cfgdatacmd_print(int argc, char *argv[])
{
struct config_data_s cfg;
- int x, ret;
+ int ret;
int fd;
bool isstring;
@@ -453,9 +453,10 @@ static void cfgdatacmd_print(int argc, char *argv[])
/* Copy the name to the cfg struct */
strncpy(cfg.name, argv[2], CONFIG_MTD_CONFIG_NAME_LEN);
- (void)x;
#else
+ int x;
+
/* Parse the id and instance */
cfg.id = atoi(argv[2]);
diff --git a/system/composite/composite_main.c b/system/composite/composite_main.c
index 3cbfc9547..08864f239 100644
--- a/system/composite/composite_main.c
+++ b/system/composite/composite_main.c
@@ -127,7 +127,7 @@ static void check_test_memory_usage(FAR const char *msg)
#ifdef CONFIG_CAN_PASS_STRUCTS
g_composite.mmcurrent = mallinfo();
#else
- (void)mallinfo(&g_composite.mmcurrent);
+ mallinfo(&g_composite.mmcurrent);
#endif
/* Show the change from the previous time */
@@ -159,7 +159,7 @@ static void final_memory_usage(FAR const char *msg)
#ifdef CONFIG_CAN_PASS_STRUCTS
g_composite.mmcurrent = mallinfo();
#else
- (void)mallinfo(&g_composite.mmcurrent);
+ mallinfo(&g_composite.mmcurrent);
#endif
/* Show the change from the previous time */
@@ -447,7 +447,7 @@ int main(int argc, FAR char *argv[])
g_composite.mmstart = mallinfo();
g_composite.mmprevious = g_composite.mmstart;
# else
- (void)mallinfo(&g_composite.mmstart);
+ mallinfo(&g_composite.mmstart);
memcpy(&g_composite.mmprevious, &g_composite.mmstart, sizeof(struct mallinfo));
# endif
#endif
@@ -566,7 +566,7 @@ int disconn_main(int argc, char *argv[])
ctrl.config = config;
ctrl.handle = &g_composite.cmphandle;
- (void)boardctl(BOARDIOC_USBDEV_CONTROL, (uintptr_t)&ctrl);
+ boardctl(BOARDIOC_USBDEV_CONTROL, (uintptr_t)&ctrl);
g_composite.cmphandle = NULL;
printf("disconn_main: Disconnected\n");
diff --git a/system/dhcpc/renew_main.c b/system/dhcpc/renew_main.c
index 33c8250a5..872a54635 100644
--- a/system/dhcpc/renew_main.c
+++ b/system/dhcpc/renew_main.c
@@ -105,7 +105,7 @@ int main(int argc, FAR char *argv[])
ret = dhcpc_request(handle, &ds);
if (ret < 0)
{
- (void)dhcpc_close(handle);
+ dhcpc_close(handle);
fprintf(stderr, "ERROR: dhcpc_request() failed\n");
return EXIT_FAILURE;
}
diff --git a/system/i2c/i2c_dev.c b/system/i2c/i2c_dev.c
index 22f66907e..3ded08de0 100644
--- a/system/i2c/i2c_dev.c
+++ b/system/i2c/i2c_dev.c
@@ -182,7 +182,7 @@ int i2ccmd_dev(FAR struct i2ctool_s *i2ctool, int argc, char **argv)
i2ctool_flush(i2ctool);
}
- (void)close(fd);
+ close(fd);
errout:
diff --git a/system/i2c/i2c_dump.c b/system/i2c/i2c_dump.c
index f63657afe..330f76fcb 100644
--- a/system/i2c/i2c_dump.c
+++ b/system/i2c/i2c_dump.c
@@ -227,7 +227,7 @@ int i2ccmd_dump(FAR struct i2ctool_s *i2ctool, int argc, FAR char **argv)
free(buf);
errout_with_fildes:
- (void)close(fd);
+ close(fd);
return ret;
}
diff --git a/system/i2c/i2c_get.c b/system/i2c/i2c_get.c
index 75dce4a42..3f1f12383 100644
--- a/system/i2c/i2c_get.c
+++ b/system/i2c/i2c_get.c
@@ -161,7 +161,7 @@ int i2ccmd_get(FAR struct i2ctool_s *i2ctool, int argc, FAR char **argv)
}
}
- (void)close(fd);
+ close(fd);
return ret;
}
diff --git a/system/i2c/i2c_main.c b/system/i2c/i2c_main.c
index d0db768eb..11b6c497c 100644
--- a/system/i2c/i2c_main.c
+++ b/system/i2c/i2c_main.c
@@ -388,7 +388,7 @@ int main(int argc, FAR char *argv[])
/* Parse and process the command line */
i2c_setup(&g_i2ctool);
- (void)i2c_parse(&g_i2ctool, argc, argv);
+ i2c_parse(&g_i2ctool, argc, argv);
i2ctool_flush(&g_i2ctool);
i2c_teardown(&g_i2ctool);
diff --git a/system/i2c/i2c_set.c b/system/i2c/i2c_set.c
index 4a1bcd687..c107cfcbf 100644
--- a/system/i2c/i2c_set.c
+++ b/system/i2c/i2c_set.c
@@ -188,7 +188,7 @@ int i2ccmd_set(FAR struct i2ctool_s *i2ctool, int argc, FAR char **argv)
}
}
- (void)close(fd);
+ close(fd);
return ret;
}
diff --git a/system/i2c/i2c_verf.c b/system/i2c/i2c_verf.c
index 48d5d6e2f..fa55b25af 100644
--- a/system/i2c/i2c_verf.c
+++ b/system/i2c/i2c_verf.c
@@ -215,6 +215,6 @@ int i2ccmd_verf(FAR struct i2ctool_s *i2ctool, int argc, FAR char **argv)
}
}
- (void)close(fd);
+ close(fd);
return ret;
}
diff --git a/system/lzf/lzf_main.c b/system/lzf/lzf_main.c
index 19097e26f..75af6876c 100644
--- a/system/lzf/lzf_main.c
+++ b/system/lzf/lzf_main.c
@@ -82,7 +82,7 @@ static uint8_t g_buf2[MAX_BLOCKSIZE + LZF_MAX_HDR_SIZE + 16];
static void lzf_exit(int exitcode) noreturn_function;
static void lzf_exit(int exitcode)
{
- (void)sem_post(&g_exclsem);
+ sem_post(&g_exclsem);
exit(exitcode);
}
#else
diff --git a/system/mdio/mdio_main.c b/system/mdio/mdio_main.c
index 011dfa59e..d95e8dcf0 100644
--- a/system/mdio/mdio_main.c
+++ b/system/mdio/mdio_main.c
@@ -218,7 +218,7 @@ int main(int argc, FAR char *argv[])
for (i = 0; i < 32; i++)
{
- (void)get_phy_reg(phy_id, i, &val_out);
+ get_phy_reg(phy_id, i, &val_out);
printf("phy[%d][%d] = 0x%4x\n", phy_id, i, val_out);
}
}
diff --git a/system/nsh/nsh_main.c b/system/nsh/nsh_main.c
index 14b908cb2..c888dbe11 100644
--- a/system/nsh/nsh_main.c
+++ b/system/nsh/nsh_main.c
@@ -158,7 +158,7 @@ static int nsh_task(void)
symdesc.symtab = (FAR struct symtab_s *)CONFIG_SYSTEM_NSH_SYMTAB_ARRAYNAME; /* Discard 'const' */
symdesc.nsymbols = CONFIG_SYSTEM_NSH_SYMTAB_COUNTNAME;
- (void)boardctl(BOARDIOC_APP_SYMTAB, (uintptr_t)&symdesc);
+ boardctl(BOARDIOC_APP_SYMTAB, (uintptr_t)&symdesc);
#endif
/* Initialize the NSH library */
@@ -215,13 +215,13 @@ int main(int argc, FAR char *argv[])
/* Check the task priority that we were started with */
- (void)sched_getparam(0, ¶m);
+ sched_getparam(0, ¶m);
if (param.sched_priority != CONFIG_SYSTEM_NSH_PRIORITY)
{
/* If not then set the priority to the configured priority */
param.sched_priority = CONFIG_SYSTEM_NSH_PRIORITY;
- (void)sched_setparam(0, ¶m);
+ sched_setparam(0, ¶m);
}
/* There are two modes that NSH can be executed in:
diff --git a/system/nxplayer/nxplayer.c b/system/nxplayer/nxplayer.c
index 64fd40100..af73105ff 100644
--- a/system/nxplayer/nxplayer.c
+++ b/system/nxplayer/nxplayer.c
@@ -208,10 +208,10 @@ static int _open_with_http(const char *fullurl)
tv.tv_sec = 10; /* TODO */
tv.tv_usec = 0;
- (void)setsockopt(s, SOL_SOCKET, SO_RCVTIMEO, (FAR const void *)&tv,
- sizeof(struct timeval));
- (void)setsockopt(s, SOL_SOCKET, SO_SNDTIMEO, (FAR const void *)&tv,
- sizeof(struct timeval));
+ setsockopt(s, SOL_SOCKET, SO_RCVTIMEO, (FAR const void *)&tv,
+ sizeof(struct timeval));
+ setsockopt(s, SOL_SOCKET, SO_SNDTIMEO, (FAR const void *)&tv,
+ sizeof(struct timeval));
struct sockaddr_in server;
server.sin_family = AF_INET;
@@ -975,7 +975,7 @@ static void *nxplayer_playthread(pthread_addr_t pvarg)
*/
#ifndef CONFIG_AUDIO_EXCLUDE_VOLUME
- (void)nxplayer_setvolume(pplayer, pplayer->volume);
+ nxplayer_setvolume(pplayer, pplayer->volume);
#ifndef CONFIG_AUDIO_EXCLUDE_BALANCE
nxplayer_setbalance(pplayer, pplayer->balance);
#endif
@@ -1998,9 +1998,9 @@ static int nxplayer_playinternal(FAR struct nxplayer_s *pplayer,
pthread_attr_init(&tattr);
sparam.sched_priority = sched_get_priority_max(SCHED_FIFO) - 9;
- (void)pthread_attr_setschedparam(&tattr, &sparam);
- (void)pthread_attr_setstacksize(&tattr,
- CONFIG_NXPLAYER_PLAYTHREAD_STACKSIZE);
+ pthread_attr_setschedparam(&tattr, &sparam);
+ pthread_attr_setstacksize(&tattr,
+ CONFIG_NXPLAYER_PLAYTHREAD_STACKSIZE);
/* Add a reference count to the player for the thread and start the
* thread. We increment for the thread to avoid thread start-up
diff --git a/system/nxrecorder/nxrecorder.c b/system/nxrecorder/nxrecorder.c
index 677f4d098..7d72e5654 100644
--- a/system/nxrecorder/nxrecorder.c
+++ b/system/nxrecorder/nxrecorder.c
@@ -928,9 +928,9 @@ int nxrecorder_recordraw(FAR struct nxrecorder_s *precorder,
pthread_attr_init(&tattr);
sparam.sched_priority = sched_get_priority_max(SCHED_FIFO) - 9;
- (void)pthread_attr_setschedparam(&tattr, &sparam);
- (void)pthread_attr_setstacksize(&tattr,
- CONFIG_NXRECORDER_RECORDTHREAD_STACKSIZE);
+ pthread_attr_setschedparam(&tattr, &sparam);
+ pthread_attr_setstacksize(&tattr,
+ CONFIG_NXRECORDER_RECORDTHREAD_STACKSIZE);
/* Add a reference count to the recorder for the thread and start the
* thread. We increment for the thread to avoid thread start-up
diff --git a/system/popen/popen.c b/system/popen/popen.c
index e70064b0e..dfc201ddb 100644
--- a/system/popen/popen.c
+++ b/system/popen/popen.c
@@ -280,14 +280,14 @@ FILE *popen(FAR const char *command, FAR const char *mode)
* the interface.
*/
- (void)close(newfd);
+ close(newfd);
/* Free attributes and file actions. Ignoring return values in the case
* of an error.
*/
- (void)posix_spawn_file_actions_destroy(&file_actions);
- (void)posix_spawnattr_destroy(&attr);
+ posix_spawn_file_actions_destroy(&file_actions);
+ posix_spawnattr_destroy(&attr);
/* Finale and return input input/output stream */
@@ -295,17 +295,17 @@ FILE *popen(FAR const char *command, FAR const char *mode)
return &container->copy;
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_stream:
- (void)fclose(container->original);
+ fclose(container->original);
errout_with_pipe:
- (void)close(fd[0]);
- (void)close(fd[1]);
+ close(fd[0]);
+ close(fd[1]);
errout_with_container:
free(container);
@@ -380,7 +380,7 @@ int pclose(FILE *stream)
* process)
*/
- (void)fclose(original);
+ fclose(original);
shell = container->shell;
free(container);
diff --git a/system/readline/readline.c b/system/readline/readline.c
index a223d0f75..34b3b37a5 100644
--- a/system/readline/readline.c
+++ b/system/readline/readline.c
@@ -181,7 +181,7 @@ static void readline_write(FAR struct rl_common_s *vtbl,
FAR struct readline_s *priv = (FAR struct readline_s *)vtbl;
DEBUGASSERT(priv && buffer && buflen > 0);
- (void)write(priv->outfd, buffer, buflen);
+ write(priv->outfd, buffer, buflen);
}
#endif
diff --git a/system/sched_note/note_main.c b/system/sched_note/note_main.c
index 089906b85..4c251c1dc 100644
--- a/system/sched_note/note_main.c
+++ b/system/sched_note/note_main.c
@@ -624,7 +624,7 @@ static int note_daemon(int argc, char *argv[])
usleep(CONFIG_SYSTEM_NOTE_DELAY * 1000L);
}
- (void)close(fd);
+ close(fd);
errout:
g_note_daemon_started = false;
diff --git a/system/spi/spi_exch.c b/system/spi/spi_exch.c
index bd9677586..6bdee9e98 100644
--- a/system/spi/spi_exch.c
+++ b/system/spi/spi_exch.c
@@ -161,7 +161,7 @@ int spicmd_exch(FAR struct spitool_s *spitool, int argc, FAR char **argv)
ret = spidev_transfer(fd, &seq);
- (void)close(fd);
+ close(fd);
if (ret)
{
diff --git a/system/spi/spi_main.c b/system/spi/spi_main.c
index 8e8de73dc..d444c330e 100644
--- a/system/spi/spi_main.c
+++ b/system/spi/spi_main.c
@@ -403,7 +403,7 @@ int main(int argc, FAR char *argv[])
/* Parse and process the command line */
spi_setup(&g_spitool);
- (void)spi_parse(&g_spitool, argc, argv);
+ spi_parse(&g_spitool, argc, argv);
spitool_flush(&g_spitool);
spi_teardown(&g_spitool);
diff --git a/system/system/system.c b/system/system/system.c
index 82dbdde7a..2573269e2 100644
--- a/system/system/system.c
+++ b/system/system/system.c
@@ -170,11 +170,11 @@ int system(FAR const char *cmd)
rc = ERROR;
}
- (void)posix_spawnattr_destroy(&attr);
+ posix_spawnattr_destroy(&attr);
return rc;
errout_with_attrs:
- (void)posix_spawnattr_destroy(&attr);
+ posix_spawnattr_destroy(&attr);
errout:
set_errno(errcode);
diff --git a/system/taskset/taskset.c b/system/taskset/taskset.c
index b7307549a..4decd979b 100644
--- a/system/taskset/taskset.c
+++ b/system/taskset/taskset.c
@@ -186,7 +186,7 @@ int main(int argc, FAR char *argv[])
CONFIG_SYSTEM_TASKSET_STACKSIZE, nsh_system,
(FAR char * const *)nshargv);
- (void)waitpid(pid, &rc, 0);
+ waitpid(pid, &rc, 0);
}
}
diff --git a/system/tee/tee.c b/system/tee/tee.c
index 474a689c0..5a1b377ec 100644
--- a/system/tee/tee.c
+++ b/system/tee/tee.c
@@ -228,7 +228,7 @@ errout_with_list:
for (curr = head; curr; curr = next)
{
next = curr->next;
- (void)close(curr->fd);
+ close(curr->fd);
free(curr);
}
diff --git a/system/usbmsc/usbmsc_main.c b/system/usbmsc/usbmsc_main.c
index 55662c829..d4f01b209 100644
--- a/system/usbmsc/usbmsc_main.c
+++ b/system/usbmsc/usbmsc_main.c
@@ -152,7 +152,7 @@ static void check_test_memory_usage(FAR const char *msg)
#ifdef CONFIG_CAN_PASS_STRUCTS
g_usbmsc.mmcurrent = mallinfo();
#else
- (void)mallinfo(&g_usbmsc.mmcurrent);
+ mallinfo(&g_usbmsc.mmcurrent);
#endif
/* Show the change from the previous time */
@@ -184,7 +184,7 @@ static void final_memory_usage(FAR const char *msg)
#ifdef CONFIG_CAN_PASS_STRUCTS
g_usbmsc.mmcurrent = mallinfo();
#else
- (void)mallinfo(&g_usbmsc.mmcurrent);
+ mallinfo(&g_usbmsc.mmcurrent);
#endif
/* Show the change from the previous time */
@@ -414,7 +414,7 @@ static void usbmsc_disconnect(FAR void *handle)
ctrl.instance = 0;
ctrl.handle = &handle;
- (void)boardctl(BOARDIOC_USBDEV_CONTROL, (uintptr_t)&ctrl);
+ boardctl(BOARDIOC_USBDEV_CONTROL, (uintptr_t)&ctrl);
}
/****************************************************************************
@@ -458,7 +458,7 @@ int main(int argc, FAR char *argv[])
g_usbmsc.mmstart = mallinfo();
g_usbmsc.mmprevious = g_usbmsc.mmstart;
# else
- (void)mallinfo(&g_usbmsc.mmstart);
+ mallinfo(&g_usbmsc.mmstart);
memcpy(&g_usbmsc.mmprevious, &g_usbmsc.mmstart, sizeof(struct mallinfo));
# endif
#endif
diff --git a/system/vi/vi.c b/system/vi/vi.c
index 5853059c9..ebf3a012d 100644
--- a/system/vi/vi.c
+++ b/system/vi/vi.c
@@ -1338,7 +1338,7 @@ static bool vi_insertfile(FAR struct vi_s *vi, off_t pos,
}
vi->fullredraw = true;
- (void)fclose(stream);
+ fclose(stream);
return ret;
}
@@ -1379,11 +1379,11 @@ static bool vi_savetext(FAR struct vi_s *vi, FAR const char *filename,
/* Report the error (or partial write). EINTR is not handled. */
vi_error(vi, g_fmtcmdfail, "fwrite", errno);
- (void)fclose(stream);
+ fclose(stream);
return false;
}
- (void)fclose(stream);
+ fclose(stream);
len = sprintf(vi->scratch, "%dC written", nwritten);
vi_write(vi, vi->scratch, len);
@@ -3703,7 +3703,7 @@ static void vi_cmd_mode(FAR struct vi_s *vi)
case KEY_CMDMODE_BOTTOM: /* Move to bottom of screen */
{
- (void)vi_gotoscreenbottom(vi, 0);
+ vi_gotoscreenbottom(vi, 0);
}
break;
@@ -4927,11 +4927,11 @@ static void vi_parsefind(FAR struct vi_s *vi, bool revfind)
vi->revfind = revfind;
if (revfind)
{
- (void)vi_revfindstring(vi);
+ vi_revfindstring(vi);
}
else
{
- (void)vi_findstring(vi);
+ vi_findstring(vi);
}
/* Exit the sub-mode and revert to command mode */
@@ -5879,7 +5879,7 @@ int main(int argc, FAR char *argv[])
/* Load the file into memory */
- (void)vi_insertfile(vi, 0, vi->filename);
+ vi_insertfile(vi, 0, vi->filename);
vi->modified = false;
/* Skip over the filename argument. There should nothing after this */
diff --git a/system/zmodem/rz_main.c b/system/zmodem/rz_main.c
index 3597c7a1b..7ad268bf7 100644
--- a/system/zmodem/rz_main.c
+++ b/system/zmodem/rz_main.c
@@ -172,7 +172,7 @@ int main(int argc, FAR char *argv[])
exitcode = EXIT_SUCCESS;
errout_with_zmodem:
- (void)zmr_release(handle);
+ zmr_release(handle);
errout_with_device:
#ifdef CONFIG_SERIAL_TERMIOS
@@ -187,7 +187,7 @@ errout_with_device:
tcsetattr(fd, TCSANOW, &saveterm);
#endif
- (void)close(fd);
+ close(fd);
errout:
return exitcode;
diff --git a/system/zmodem/sz_main.c b/system/zmodem/sz_main.c
index f02c54e3b..c2ca20194 100644
--- a/system/zmodem/sz_main.c
+++ b/system/zmodem/sz_main.c
@@ -271,7 +271,7 @@ int main(int argc, FAR char *argv[])
exitcode = EXIT_SUCCESS;
errout_with_zmodem:
- (void)zms_release(handle);
+ zms_release(handle);
errout_with_device:
#ifdef CONFIG_SERIAL_TERMIOS
@@ -286,7 +286,7 @@ errout_with_device:
tcsetattr(fd, TCSANOW, &saveterm);
#endif
- (void)close(fd);
+ close(fd);
errout:
return exitcode;
diff --git a/system/zmodem/zm_receive.c b/system/zmodem/zm_receive.c
index 7ad0a24e3..39244f027 100644
--- a/system/zmodem/zm_receive.c
+++ b/system/zmodem/zm_receive.c
@@ -738,7 +738,7 @@ static int zmr_filedata(FAR struct zm_state_s *pzm)
/* Send the cancel string */
- (void)zm_remwrite(pzm->remfd, g_canistr, CANISTR_SIZE);
+ zm_remwrite(pzm->remfd, g_canistr, CANISTR_SIZE);
/* Enter PSTATE_DATA */
@@ -782,7 +782,7 @@ static int zmr_filedata(FAR struct zm_state_s *pzm)
pzm->state = ZMR_FINISH;
pzm->pstate = PSTATE_IDLE;
pzm->psubstate = PIDLE_ZPAD;
- (void)zmr_fileerror(pzmr, ZFERR, (uint32_t)errorcode);
+ zmr_fileerror(pzmr, ZFERR, (uint32_t)errorcode);
return -errorcode;
}
@@ -1697,7 +1697,7 @@ int zmr_release(ZMRHANDLE handle)
/* Release the timer resources */
- (void)zm_timerrelease(&pzmr->cmn);
+ zm_timerrelease(&pzmr->cmn);
/* Clean up any resouces that may be held from the last file transfer */
diff --git a/system/zmodem/zm_send.c b/system/zmodem/zm_send.c
index a2cbbeda2..6efbfbb8b 100644
--- a/system/zmodem/zm_send.c
+++ b/system/zmodem/zm_send.c
@@ -1622,7 +1622,7 @@ ZMSHANDLE zms_initialize(int remfd)
return (ZMSHANDLE)pzms;
errout_with_timer:
- (void)zm_timerrelease(&pzms->cmn);
+ zm_timerrelease(&pzms->cmn);
errout:
free(pzms);
return (ZMSHANDLE)NULL;
@@ -1714,7 +1714,7 @@ int zms_release(ZMSHANDLE handle)
/* Release the timer resources */
- (void)zm_timerrelease(&pzms->cmn);
+ zm_timerrelease(&pzms->cmn);
/* Make sure that the file is closed */
diff --git a/system/zmodem/zm_state.c b/system/zmodem/zm_state.c
index c67213e13..75844719a 100644
--- a/system/zmodem/zm_state.c
+++ b/system/zmodem/zm_state.c
@@ -890,7 +890,7 @@ int zm_datapump(FAR struct zm_state_s *pzm)
/* Stop the timer */
- (void)zm_timerstop(pzm);
+ zm_timerstop(pzm);
sched_unlock();
/* EOF from the remote peer can only mean that we lost the connection
diff --git a/testing/fstest/fstest_main.c b/testing/fstest/fstest_main.c
index ede130352..6cdb5f729 100644
--- a/testing/fstest/fstest_main.c
+++ b/testing/fstest/fstest_main.c
@@ -153,7 +153,7 @@ static void fstest_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 */
@@ -179,7 +179,7 @@ static void fstest_endmemusage(void)
#ifdef CONFIG_CAN_PASS_STRUCTS
g_mmafter = mallinfo();
#else
- (void)mallinfo(&g_mmafter);
+ mallinfo(&g_mmafter);
#endif
printf("\nFinal memory usage:\n");
fstest_showmemusage(&g_mmbefore, &g_mmafter);
@@ -967,7 +967,7 @@ int main(int argc, FAR char *argv[])
g_mmbefore = mallinfo();
g_mmprevious = g_mmbefore;
#else
- (void)mallinfo(&g_mmbefore);
+ mallinfo(&g_mmbefore);
memcpy(&g_mmprevious, &g_mmbefore, sizeof(struct mallinfo));
#endif
@@ -988,7 +988,7 @@ int main(int argc, FAR char *argv[])
*/
printf("\n=== FILLING %u =============================\n", i);
- (void)fstest_fillfs();
+ fstest_fillfs();
printf("Filled file system\n");
printf(" Number of files: %d\n", g_nfiles);
printf(" Number deleted: %d\n", g_ndeleted);
@@ -1084,7 +1084,7 @@ int main(int argc, FAR char *argv[])
/* Perform garbage collection, integrity checks */
- (void)fstest_gc(buf.f_bfree);
+ fstest_gc(buf.f_bfree);
/* Show memory usage */
diff --git a/testing/nxffs/nxffs_main.c b/testing/nxffs/nxffs_main.c
index e7ecdc9cf..7f40ba311 100644
--- a/testing/nxffs/nxffs_main.c
+++ b/testing/nxffs/nxffs_main.c
@@ -184,7 +184,7 @@ static void nxffs_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 */
@@ -210,7 +210,7 @@ static void nxffs_endmemusage(void)
#ifdef CONFIG_CAN_PASS_STRUCTS
g_mmafter = mallinfo();
#else
- (void)mallinfo(&g_mmafter);
+ mallinfo(&g_mmafter);
#endif
printf("\nFinal memory usage:\n");
nxffs_showmemusage(&g_mmbefore, &g_mmafter);
@@ -833,7 +833,7 @@ int main(int argc, FAR char *argv[])
g_mmbefore = mallinfo();
g_mmprevious = g_mmbefore;
#else
- (void)mallinfo(&g_mmbefore);
+ mallinfo(&g_mmbefore);
memcpy(&g_mmprevious, &g_mmbefore, sizeof(struct mallinfo));
#endif
@@ -854,7 +854,7 @@ int main(int argc, FAR char *argv[])
*/
printf("\n=== FILLING %u =============================\n", i);
- (void)nxffs_fillfs();
+ nxffs_fillfs();
printf("Filled file system\n");
printf(" Number of files: %d\n", g_nfiles);
printf(" Number deleted: %d\n", g_ndeleted);
diff --git a/testing/ostest/aio.c b/testing/ostest/aio.c
index 9515ed899..d55e7a870 100644
--- a/testing/ostest/aio.c
+++ b/testing/ostest/aio.c
@@ -304,7 +304,7 @@ void aio_test(void)
sigfillset(&set);
sigdelset(&set, SIGUSR1);
sigdelset(&set, SIGALRM);
- (void)sigprocmask(SIG_SETMASK, &set, &oset);
+ sigprocmask(SIG_SETMASK, &set, &oset);
/* Case 1: Poll for transfer complete */
@@ -627,7 +627,7 @@ void aio_test(void)
}
while (ret < 0);
- (void)sigprocmask(SIG_SETMASK, &oset, NULL);
+ sigprocmask(SIG_SETMASK, &oset, NULL);
printf("aio_test: Test completed successfully\n");
return;
@@ -635,7 +635,7 @@ errout_with_fildes:
close(g_fildes);
g_fildes = -1;
errout_with_procmask:
- (void)sigprocmask(SIG_SETMASK, &oset, NULL);
+ sigprocmask(SIG_SETMASK, &oset, NULL);
printf("aio_test: ERROR: Test aborted\n");
}
diff --git a/testing/ostest/barrier.c b/testing/ostest/barrier.c
index 859281184..6f4172c65 100644
--- a/testing/ostest/barrier.c
+++ b/testing/ostest/barrier.c
@@ -138,7 +138,7 @@ void barrier_test(void)
/* Create the barrier */
- (void)pthread_barrierattr_init(&barrierattr);
+ pthread_barrierattr_init(&barrierattr);
/* Start CONFIG_TESTING_OSTEST_NBARRIER_THREADS thread instances */
diff --git a/testing/ostest/fpu.c b/testing/ostest/fpu.c
index ea463339d..f4adcf781 100644
--- a/testing/ostest/fpu.c
+++ b/testing/ostest/fpu.c
@@ -336,8 +336,8 @@ void fpu_test(void)
/* Wait for each task to complete */
fflush(stdout);
- (void)waitpid(task1, &statloc, 0);
- (void)waitpid(task2, &statloc, 0);
+ waitpid(task1, &statloc, 0);
+ waitpid(task2, &statloc, 0);
#else
printf("fpu_test: ERROR: The FPU test is not properly configured\n");
diff --git a/testing/ostest/mutex.c b/testing/ostest/mutex.c
index 0a0494d77..b604ab462 100644
--- a/testing/ostest/mutex.c
+++ b/testing/ostest/mutex.c
@@ -130,7 +130,7 @@ void mutex_test(void)
printf("Starting thread 1\n");
#ifdef SDCC
- (void)pthread_attr_init(&attr);
+ pthread_attr_init(&attr);
status = pthread_create(&thread1, &attr, thread_func, (pthread_addr_t)1);
#else
status = pthread_create(&thread1, NULL, thread_func, (pthread_addr_t)1);
diff --git a/testing/ostest/ostest_main.c b/testing/ostest/ostest_main.c
index 57ad67f49..d35d4f2f1 100644
--- a/testing/ostest/ostest_main.c
+++ b/testing/ostest/ostest_main.c
@@ -142,7 +142,7 @@ static void check_test_memory_usage(void)
#ifdef CONFIG_CAN_PASS_STRUCTS
g_mmafter = mallinfo();
#else
- (void)mallinfo(&g_mmafter);
+ mallinfo(&g_mmafter);
#endif
/* Show the change from the previous time */
@@ -228,7 +228,7 @@ static int user_main(int argc, char *argv[])
g_mmbefore = mallinfo();
g_mmprevious = g_mmbefore;
#else
- (void)mallinfo(&g_mmbefore);
+ mallinfo(&g_mmbefore);
memcpy(&g_mmprevious, &g_mmbefore, sizeof(struct mallinfo));
#endif
@@ -556,7 +556,7 @@ static int user_main(int argc, char *argv[])
#ifdef CONFIG_CAN_PASS_STRUCTS
g_mmafter = mallinfo();
#else
- (void)mallinfo(&g_mmafter);
+ mallinfo(&g_mmafter);
#endif
printf("\nFinal memory usage:\n");
diff --git a/testing/ostest/posixtimer.c b/testing/ostest/posixtimer.c
index ffec5ff0c..a6bc356d1 100644
--- a/testing/ostest/posixtimer.c
+++ b/testing/ostest/posixtimer.c
@@ -118,7 +118,7 @@ static void timer_expiration(int signo, siginfo_t *info, void *ucontext)
/* Check sigprocmask */
- (void)sigfillset(&allsigs);
+ sigfillset(&allsigs);
status = sigprocmask(SIG_SETMASK, NULL, &oldset);
if (status != OK)
{
@@ -156,8 +156,8 @@ void timer_test(void)
printf("timer_test: Unmasking signal %d\n" , MY_TIMER_SIGNAL);
- (void)sigemptyset(&set);
- (void)sigaddset(&set, MY_TIMER_SIGNAL);
+ sigemptyset(&set);
+ sigaddset(&set, MY_TIMER_SIGNAL);
status = sigprocmask(SIG_UNBLOCK, &set, NULL);
if (status != OK)
{
@@ -169,8 +169,8 @@ void timer_test(void)
act.sa_sigaction = timer_expiration;
act.sa_flags = SA_SIGINFO;
- (void)sigfillset(&act.sa_mask);
- (void)sigdelset(&act.sa_mask, MY_TIMER_SIGNAL);
+ sigfillset(&act.sa_mask);
+ sigdelset(&act.sa_mask, MY_TIMER_SIGNAL);
status = sigaction(MY_TIMER_SIGNAL, &act, &oact);
if (status != OK)
diff --git a/testing/ostest/prioinherit.c b/testing/ostest/prioinherit.c
index db66d3b90..9e8ff13dc 100644
--- a/testing/ostest/prioinherit.c
+++ b/testing/ostest/prioinherit.c
@@ -552,20 +552,20 @@ void priority_inheritance(void)
{
printf("priority_inheritance: Waiting for highpri_thread-%d to complete\n", i+1);
FFLUSH();
- (void)pthread_join(highpri[i], &result);
+ pthread_join(highpri[i], &result);
dump_nfreeholders("priority_inheritance:");
}
printf("priority_inheritance: Waiting for medpri_thread to complete\n");
FFLUSH();
- (void)pthread_join(medpri, &result);
+ pthread_join(medpri, &result);
dump_nfreeholders("priority_inheritance:");
for (i = 0; i < NLOWPRI_THREADS; i++)
{
printf("priority_inheritance: Waiting for lowpri_thread-%d to complete\n", i+1);
FFLUSH();
- (void)pthread_join(lowpri[i], &result);
+ pthread_join(lowpri[i], &result);
dump_nfreeholders("priority_inheritance:");
}
diff --git a/testing/ostest/pthread_rwlock.c b/testing/ostest/pthread_rwlock.c
index 48449e85d..71fabee95 100644
--- a/testing/ostest/pthread_rwlock.c
+++ b/testing/ostest/pthread_rwlock.c
@@ -268,8 +268,8 @@ static void test_two_threads(void)
g_race_cond_thread_pos = 0;
status = pthread_create(&thread1, NULL, race_cond_thread1, &rc);
status = pthread_create(&thread2, NULL, race_cond_thread2, &rc);
- (void) pthread_join(thread1, NULL);
- (void) pthread_join(thread2, NULL);
+ pthread_join(thread1, NULL);
+ pthread_join(thread2, NULL);
}
static void * timeout_thread1(FAR void * data)
@@ -373,8 +373,8 @@ static void test_timeout(void)
status = pthread_create(&thread1, NULL, timeout_thread1, &rc);
status = pthread_create(&thread2, NULL, timeout_thread2, &rc);
- (void) pthread_join(thread1, NULL);
- (void) pthread_join(thread2, NULL);
+ pthread_join(thread1, NULL);
+ pthread_join(thread2, NULL);
}
/****************************************************************************
diff --git a/testing/ostest/pthread_rwlock_cancel.c b/testing/ostest/pthread_rwlock_cancel.c
index bd985fd27..538ea4f12 100644
--- a/testing/ostest/pthread_rwlock_cancel.c
+++ b/testing/ostest/pthread_rwlock_cancel.c
@@ -63,7 +63,7 @@ static void * timeout_thread1(FAR void * data)
while(1)
{
- (void)clock_gettime(CLOCK_REALTIME, &time);
+ clock_gettime(CLOCK_REALTIME, &time);
time.tv_sec += 1;
status = pthread_rwlock_timedrdlock(sync->write_lock, &time);
@@ -84,7 +84,7 @@ static void * timeout_thread2(FAR void * data)
while (1)
{
- (void)clock_gettime(CLOCK_REALTIME, &time);
+ clock_gettime(CLOCK_REALTIME, &time);
time.tv_sec += 1;
status = pthread_rwlock_timedrdlock(sync->read_lock, &time);
@@ -104,7 +104,7 @@ static void * timeout_thread2(FAR void * data)
}
}
- (void)clock_gettime(CLOCK_REALTIME, &time);
+ clock_gettime(CLOCK_REALTIME, &time);
time.tv_sec += 1;
status = pthread_rwlock_timedwrlock(sync->read_lock, &time);
@@ -186,8 +186,8 @@ static void test_timeout(void)
printf("pthread_rwlock_cancel: ERROR pthread_cancel, status=%d\n", status);
}
- (void) pthread_join(thread1, NULL);
- (void) pthread_join(thread2, NULL);
+ pthread_join(thread1, NULL);
+ pthread_join(thread2, NULL);
/* Do some operations on locks in order to check if they are still in
* usable state after deferred cancellation. */
diff --git a/testing/ostest/rmutex.c b/testing/ostest/rmutex.c
index 917e2d5f6..c1be76b24 100644
--- a/testing/ostest/rmutex.c
+++ b/testing/ostest/rmutex.c
@@ -161,7 +161,7 @@ void recursive_mutex_test(void)
{
printf("recursive_mutex_test: Starting thread %d\n", i+1);
#ifdef SDCC
- (void)pthread_attr_init(&attr);
+ pthread_attr_init(&attr);
status = pthread_create(&thread[i], &attr, thread_outer, (pthread_addr_t)((uintptr_t)i+1));
#else
status = pthread_create(&thread[i], NULL, thread_outer, (pthread_addr_t)((uintptr_t)i+1));
diff --git a/testing/ostest/sighand.c b/testing/ostest/sighand.c
index 71537b01b..6078562b4 100644
--- a/testing/ostest/sighand.c
+++ b/testing/ostest/sighand.c
@@ -136,7 +136,7 @@ static void wakeup_action(int signo, siginfo_t *info, void *ucontext)
/* Check sigprocmask */
- (void)sigfillset(&allsigs);
+ sigfillset(&allsigs);
status = sigprocmask(SIG_SETMASK, NULL, &oldset);
if (status != OK)
{
@@ -161,8 +161,8 @@ static int waiter_main(int argc, char *argv[])
printf("waiter_main: Waiter started\n" );
printf("waiter_main: Unmasking signal %d\n" , WAKEUP_SIGNAL);
- (void)sigemptyset(&set);
- (void)sigaddset(&set, WAKEUP_SIGNAL);
+ sigemptyset(&set);
+ sigaddset(&set, WAKEUP_SIGNAL);
status = sigprocmask(SIG_UNBLOCK, &set, NULL);
if (status != OK)
{
@@ -174,8 +174,8 @@ static int waiter_main(int argc, char *argv[])
act.sa_sigaction = wakeup_action;
act.sa_flags = SA_SIGINFO;
- (void)sigfillset(&act.sa_mask);
- (void)sigdelset(&act.sa_mask, WAKEUP_SIGNAL);
+ sigfillset(&act.sa_mask);
+ sigdelset(&act.sa_mask, WAKEUP_SIGNAL);
status = sigaction(WAKEUP_SIGNAL, &act, &oact);
if (status != OK)
@@ -214,7 +214,7 @@ static int waiter_main(int argc, char *argv[])
/* Detach the signal handler */
act.sa_handler = SIG_DFL;
- (void)sigaction(WAKEUP_SIGNAL, &act, &oact);
+ sigaction(WAKEUP_SIGNAL, &act, &oact);
printf("waiter_main: done\n" );
FFLUSH();
@@ -245,8 +245,8 @@ void sighand_test(void)
#ifdef CONFIG_SCHED_HAVE_PARENT
printf("sighand_test: Unmasking SIGCHLD\n");
- (void)sigemptyset(&set);
- (void)sigaddset(&set, SIGCHLD);
+ sigemptyset(&set);
+ sigaddset(&set, SIGCHLD);
status = sigprocmask(SIG_UNBLOCK, &set, NULL);
if (status != OK)
{
@@ -258,8 +258,8 @@ void sighand_test(void)
act.sa_sigaction = death_of_child;
act.sa_flags = SA_SIGINFO;
- (void)sigfillset(&act.sa_mask);
- (void)sigdelset(&act.sa_mask, SIGCHLD);
+ sigfillset(&act.sa_mask);
+ sigdelset(&act.sa_mask, SIGCHLD);
status = sigaction(SIGCHLD, &act, &oact);
if (status != OK)
@@ -332,7 +332,7 @@ void sighand_test(void)
#ifdef CONFIG_SCHED_HAVE_PARENT
act.sa_handler = SIG_DFL;
- (void)sigaction(SIGCHLD, &act, &oact);
+ sigaction(SIGCHLD, &act, &oact);
#endif
printf("sighand_test: done\n" );
diff --git a/testing/ostest/signest.c b/testing/ostest/signest.c
index 3bede84eb..ea720d83f 100644
--- a/testing/ostest/signest.c
+++ b/testing/ostest/signest.c
@@ -134,7 +134,7 @@ static int waiter_main(int argc, char *argv[])
printf("waiter_main: Waiter started\n" );
printf("waiter_main: Setting signal mask\n" );
- (void)sigemptyset(&set);
+ sigemptyset(&set);
ret = sigprocmask(SIG_SETMASK, &set, NULL);
if (ret < 0)
{
@@ -147,12 +147,12 @@ static int waiter_main(int argc, char *argv[])
act.sa_handler = waiter_action;
act.sa_flags = 0;
- (void)sigemptyset(&act.sa_mask);
+ sigemptyset(&act.sa_mask);
for (i = 1; i < MAX_SIGNO; i += 2)
{
if (signest_catchable(i))
{
- (void)sigaddset(&act.sa_mask, i);
+ sigaddset(&act.sa_mask, i);
}
}
@@ -196,7 +196,7 @@ static int interfere_main(int argc, char *argv[])
g_interferer_running = true;
while (!g_done)
{
- (void)sem_wait(&g_interferer_sem);
+ sem_wait(&g_interferer_sem);
}
/* Just exit, the system should clean up the signal handlers */
diff --git a/testing/ostest/specific.c b/testing/ostest/specific.c
index 2776fbd5a..e0c7dd9d2 100644
--- a/testing/ostest/specific.c
+++ b/testing/ostest/specific.c
@@ -129,7 +129,7 @@ void specific_test(void)
printf("Starting thread\n");
#ifdef SDCC
- (void)pthread_attr_init(&attr);
+ pthread_attr_init(&attr);
ret = pthread_create(&thread, &attr, thread_func, (pthread_addr_t)0);
#else
ret = pthread_create(&thread, NULL, thread_func, (pthread_addr_t)0);
diff --git a/testing/ostest/timedmutex.c b/testing/ostest/timedmutex.c
index 7593cc661..41088487d 100644
--- a/testing/ostest/timedmutex.c
+++ b/testing/ostest/timedmutex.c
@@ -119,7 +119,7 @@ static void *thread_func(FAR void *parameter)
/* Release the lock and wait a bit in case the main thread wants it. */
- (void)pthread_mutex_unlock(&g_mutex);
+ pthread_mutex_unlock(&g_mutex);
usleep(500*1000);
}
@@ -157,7 +157,7 @@ void timedmutex_test(void)
printf("mutex_test: Starting thread\n");
#ifdef SDCC
- (void)pthread_attr_init(&attr);
+ pthread_attr_init(&attr);
status = pthread_create(&thread, &attr, thread_func, (pthread_addr_t)0);
#else
status = pthread_create(&thread, NULL, thread_func, (pthread_addr_t)0);
@@ -175,7 +175,7 @@ void timedmutex_test(void)
/* Then unlock the mutex. This should wake up the pthread. */
printf("mutex_test: Unlocking\n");
- (void)pthread_mutex_unlock(&g_mutex);
+ pthread_mutex_unlock(&g_mutex);
/* Wait a bit to assure that the thread gets a chance to lock the mutex at
* least once (it may probably loop and retake the mutex several times)
@@ -227,7 +227,7 @@ void timedmutex_test(void)
#endif
errout_with_lock:
- (void)pthread_mutex_unlock(&g_mutex);
+ pthread_mutex_unlock(&g_mutex);
errout_with_mutex:
- (void)pthread_mutex_destroy(&g_mutex);
+ pthread_mutex_destroy(&g_mutex);
}
diff --git a/testing/smart/smart_main.c b/testing/smart/smart_main.c
index a6100ad3b..dfe9d85e3 100644
--- a/testing/smart/smart_main.c
+++ b/testing/smart/smart_main.c
@@ -187,7 +187,7 @@ static void smart_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 */
@@ -213,7 +213,7 @@ static void smart_endmemusage(void)
#ifdef CONFIG_CAN_PASS_STRUCTS
g_mmafter = mallinfo();
#else
- (void)mallinfo(&g_mmafter);
+ mallinfo(&g_mmafter);
#endif
printf("\nFinal memory usage:\n");
smart_showmemusage(&g_mmbefore, &g_mmafter);
@@ -829,9 +829,9 @@ int main(int argc, FAR char *argv[])
/* Create a SMARTFS filesystem */
#ifdef CONFIG_SMARTFS_MULTI_ROOT_DIRS
- (void)mksmartfs("/dev/smart1", 1024, 1);
+ mksmartfs("/dev/smart1", 1024, 1);
#else
- (void)mksmartfs("/dev/smart1", 1024);
+ mksmartfs("/dev/smart1", 1024);
#endif
/* Mount the file system */
@@ -850,7 +850,7 @@ int main(int argc, FAR char *argv[])
g_mmbefore = mallinfo();
g_mmprevious = g_mmbefore;
#else
- (void)mallinfo(&g_mmbefore);
+ mallinfo(&g_mmbefore);
memcpy(&g_mmprevious, &g_mmbefore, sizeof(struct mallinfo));
#endif
@@ -871,7 +871,7 @@ int main(int argc, FAR char *argv[])
*/
printf("\n=== FILLING %u =============================\n", i);
- (void)smart_fillfs();
+ smart_fillfs();
printf("Filled file system\n");
printf(" Number of files: %d\n", g_nfiles);
printf(" Number deleted: %d\n", g_ndeleted);
diff --git a/testing/smp/smp_main.c b/testing/smp/smp_main.c
index dea93b907..4c9089df9 100644
--- a/testing/smp/smp_main.c
+++ b/testing/smp/smp_main.c
@@ -148,7 +148,7 @@ static void hog_time(FAR const char *caller, int threadno)
/* Let other threads run */
- (void)pthread_yield();
+ pthread_yield();
show_cpu_conditional(caller, threadno);
}
}
@@ -265,7 +265,7 @@ int main(int argc, FAR char *argv[])
/* Create the barrier */
- (void)pthread_barrierattr_init(&barrierattr);
+ pthread_barrierattr_init(&barrierattr);
/* Start CONFIG_TESTING_SMP_NBARRIER_THREADS thread instances */
diff --git a/wireless/gs2200m/gs2200m_main.c b/wireless/gs2200m/gs2200m_main.c
index 39f32b36d..44fc3adbb 100644
--- a/wireless/gs2200m/gs2200m_main.c
+++ b/wireless/gs2200m/gs2200m_main.c
@@ -508,8 +508,8 @@ static int socket_request(int fd, FAR struct gs2200m_s *priv,
*/
usock = gs2200m_socket_get(priv, usockid);
- (void)usock_send_event(fd, priv, usock,
- USRSOCK_EVENT_SENDTO_READY);
+ usock_send_event(fd, priv, usock,
+ USRSOCK_EVENT_SENDTO_READY);
}
gs2200m_printf("%s: end \n", __func__);
@@ -553,7 +553,7 @@ static int close_request(int fd, FAR struct gs2200m_s *priv,
memset(&clmsg, 0, sizeof(clmsg));
clmsg.cid = cid;
- (void)ioctl(priv->gsfd, GS2200M_IOC_CLOSE, (unsigned long)&clmsg);
+ ioctl(priv->gsfd, GS2200M_IOC_CLOSE, (unsigned long)&clmsg);
errout:
@@ -1281,7 +1281,7 @@ static int ioctl_request(int fd, FAR struct gs2200m_s *priv,
case SIOCSIFDSTADDR:
case SIOCSIFNETMASK:
- (void)read(fd, &imsg.ifr, sizeof(imsg.ifr));
+ read(fd, &imsg.ifr, sizeof(imsg.ifr));
break;
default:
@@ -1314,7 +1314,7 @@ static int ioctl_request(int fd, FAR struct gs2200m_s *priv,
resp2.valuelen_nontrunc = sizeof(imsg.ifr);
resp2.valuelen = sizeof(imsg.ifr);
- (void)_write_to_usock(fd, &resp2, sizeof(resp2));
+ _write_to_usock(fd, &resp2, sizeof(resp2));
/* Return struct ifreq address */
diff --git a/wireless/ieee802154/i8sak/i8sak_assoc.c b/wireless/ieee802154/i8sak/i8sak_assoc.c
index 3c754045d..6c2c236a9 100644
--- a/wireless/ieee802154/i8sak/i8sak_assoc.c
+++ b/wireless/ieee802154/i8sak/i8sak_assoc.c
@@ -311,7 +311,7 @@ void i8sak_assoc_cmd(FAR struct i8sak_s *i8sak, int argc, FAR char *argv[])
/* Clean up and return */
- (void)i8sak_eventlistener_removereceiver(i8sak, assoc_eventcb);
+ i8sak_eventlistener_removereceiver(i8sak, assoc_eventcb);
i8sak_releasedaemon(i8sak);
close(fd);
}