X-Git-Url: https://git.xinqibao.xyz/slstatus.git/blobdiff_plain/9750a3d731cd381e832bcacf1d03e48ddb46cc16..82eb6e3832dea63be4c22ac4415c3190add43f0b:/components/wifi.c

diff --git a/components/wifi.c b/components/wifi.c
index 33e09b7..24dca36 100644
--- a/components/wifi.c
+++ b/components/wifi.c
@@ -2,10 +2,10 @@
 #if defined(__linux__)
 	#include <errno.h>
 	#include <ifaddrs.h>
+	#include <limits.h>
 	#include <linux/wireless.h>
 	#include <sys/socket.h>
 	#include <stdio.h>
-	#include <limits.h>
 	#include <string.h>
 	#include <sys/ioctl.h>
 	#include <unistd.h>
@@ -16,7 +16,6 @@
 	wifi_perc(const char *iface)
 	{
 		int i, cur;
-		float perc;
 		int total = 70; /* the max of /proc/net/wireless */
 		char *p, *datastart;
 		char path[PATH_MAX];
@@ -25,8 +24,7 @@
 
 		snprintf(path, sizeof(path), "%s%s%s", "/sys/class/net/", iface,
 		         "/operstate");
-		fp = fopen(path, "r");
-		if (fp == NULL) {
+		if (!(fp = fopen(path, "r"))) {
 			fprintf(stderr, "fopen '%s': %s\n", path,
 			        strerror(errno));
 			return NULL;
@@ -37,8 +35,7 @@
 			return NULL;
 		}
 
-		fp = fopen("/proc/net/wireless", "r");
-		if (fp == NULL) {
+		if (!(fp = fopen("/proc/net/wireless", "r"))) {
 			fprintf(stderr, "fopen '/proc/net/wireless': %s\n",
 			        strerror(errno));
 			return NULL;
@@ -49,19 +46,19 @@
 				break;
 		}
 		fclose(fp);
-		if (i < 2 || !p)
+		if (i < 2 || !p) {
 			return NULL;
+		}
 
-		if ((datastart = strstr(buf, iface)) == NULL)
+		if (!(datastart = strstr(buf, iface))) {
 			return NULL;
+		}
 
 		datastart = (datastart+(strlen(iface)+1));
 		sscanf(datastart + 1, " %*d   %d  %*d  %*d\t\t  %*d\t   "
 		       "%*d\t\t%*d\t\t %*d\t  %*d\t\t %*d", &cur);
 
-		perc = (float)cur / total * 100.0;
-
-		return bprintf("%.0f", perc);
+		return bprintf("%d", (int)((float)cur / total * 100));
 	}
 
 	const char *
@@ -75,25 +72,25 @@
 		wreq.u.essid.length = IW_ESSID_MAX_SIZE+1;
 		snprintf(wreq.ifr_name, sizeof(wreq.ifr_name), "%s", iface);
 
-		if (sockfd == -1) {
+		if (sockfd < 0) {
 			fprintf(stderr, "socket 'AF_INET': %s\n",
 			        strerror(errno));
 			return NULL;
 		}
 		wreq.u.essid.pointer = id;
-		if (ioctl(sockfd,SIOCGIWESSID, &wreq) == -1) {
-			fprintf(stderr, "ioctl 'SIOCGIWESSID': %s\n",
-			        strerror(errno));
+		if (ioctl(sockfd,SIOCGIWESSID, &wreq) < 0) {
+			fprintf(stderr, "ioctl 'SIOCGIWESSID': %s\n", strerror(errno));
 			close(sockfd);
 			return NULL;
 		}
 
 		close(sockfd);
 
-		if (strcmp(id, "") == 0)
+		if (!strcmp(id, "")) {
 			return NULL;
-		else
-			return id;
+		}
+
+		return id;
 	}
 #elif defined(__OpenBSD__)
 	/* unimplemented */