logswan

commit be8404010e766954ea4129fd8b878ff19a989bc1
parent 3d6d9a665298f1fe0dad7740e1adaa70b8112a71
Author: Frederic Cambus <fred@statdns.com>
Date:   Fri,  8 Jan 2016 23:33:13 +0100

Using 'size_t' instead of 'int' for array indexes in for loops

Diffstat:
M
src/logswan.c
 | 
6
+++---
M
src/output.c
 | 
12
++++++------

2 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/src/logswan.c b/src/logswan.c
@@ -155,7 +155,7 @@ int main (int argc, char *argv[]) {
 			results.countries[countryId]++;
 
 			/* Increment continents array */
-			for (int loop = 0; loop<CONTINENTS; loop++) {
+			for (size_t loop = 0; loop<CONTINENTS; loop++) {
 				if (!strcmp(continentsId[loop], GeoIP_continent_by_id(countryId))) {
 					results.continents[loop] ++;
 					break;
@@ -189,7 +189,7 @@ int main (int argc, char *argv[]) {
 				parseRequest(&parsedRequest, parsedLine.request);
 
 				if (parsedRequest.method) {
-					for (int loop = 0; loop<METHODS; loop++) {
+					for (size_t loop = 0; loop<METHODS; loop++) {
 						if (!strcmp(methods[loop], parsedRequest.method)) {
 							results.methods[loop] ++;
 							break;
@@ -198,7 +198,7 @@ int main (int argc, char *argv[]) {
 				}
 
 				if (parsedRequest.protocol) {
-					for (int loop = 0; loop<PROTOCOLS; loop++) {
+					for (size_t loop = 0; loop<PROTOCOLS; loop++) {
 						if (!strcmp(protocols[loop], parsedRequest.protocol)) {
 							results.protocols[loop] ++;
 							break;
diff --git a/src/output.c b/src/output.c
@@ -29,37 +29,37 @@ char *output(Results results) {
 	json_t *methodsArray = json_array();
 	json_t *protocolsArray = json_array();
 
-	for (int loop=0; loop<CONTINENTS; loop++) {
+	for (size_t loop=0; loop<CONTINENTS; loop++) {
 		if (results.continents[loop]) {
 			json_array_append_new(continentsArray, json_pack("{s:s, s:s, s:i}", "data", continentsId[loop], "name", continentsNames[loop], "hits", results.continents[loop]));
 		}
 	}
 
-	for (int loop=0; loop<COUNTRIES; loop++) {
+	for (size_t loop=0; loop<COUNTRIES; loop++) {
 		if (results.countries[loop]) {
 			json_array_append_new(countriesArray, json_pack("{s:s, s:s, s:i}", "data", GeoIP_code_by_id(loop), "name", GeoIP_name_by_id(loop), "hits", results.countries[loop]));
 		}
 	}
 
-	for (int loop=0; loop<24; loop++) {
+	for (size_t loop=0; loop<24; loop++) {
 		if (results.hours[loop]) {
 			json_array_append_new(hoursArray, json_pack("{s:i, s:i}", "data", loop, "hits", results.hours[loop]));
 		}
 	}
 
-	for (int loop=0; loop<STATUS_CODE_MAX; loop++) {
+	for (size_t loop=0; loop<STATUS_CODE_MAX; loop++) {
 		if (results.status[loop]) {
 			json_array_append_new(httpStatusArray, json_pack("{s:i, s:i}", "data", loop, "hits", results.status[loop]));
 		}
 	}
 
-	for (int loop=0; loop<METHODS; loop++) {
+	for (size_t loop=0; loop<METHODS; loop++) {
 		if (results.methods[loop]) {
 			json_array_append_new(methodsArray, json_pack("{s:s, s:i}", "data", methods[loop], "hits", results.methods[loop]));
 		}
 	}
 
-	for (int loop=0; loop<PROTOCOLS; loop++) {
+	for (size_t loop=0; loop<PROTOCOLS; loop++) {
 		if (results.protocols[loop]) {
 			json_array_append_new(protocolsArray, json_pack("{s:s, s:i}", "data", protocols[loop], "hits", results.protocols[loop]));
 		}