replace __arraycount with ARRAY_SIZE

void-linux · Sep 1, 2023 · 5bd8d35 · 5bd8d35
1 parent 349f153
commit 5bd8d35
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 12 deletions.
diff --git a/include/macro.h b/include/macro.h
@@ -37,4 +37,6 @@
 # define HIDDEN
 #endif
 
+#define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]))
+
 #endif /*!XBPS_MACRO_H*/
diff --git a/include/xbps_api_impl.h b/include/xbps_api_impl.h
@@ -39,10 +39,6 @@
 			ARCHIVE_EXTRACT_UNLINK
 #define FEXTRACT_FLAGS	ARCHIVE_EXTRACT_OWNER | EXTRACT_FLAGS
 
-#ifndef __arraycount
-#define __arraycount(x) (sizeof(x) / sizeof(*x))
-#endif
-
 /**
  * @private
  */

diff --git a/lib/conf.c b/lib/conf.c
@@ -42,6 +42,7 @@
 #include <string.h>
 #include <strings.h>
 
+#include "macro.h"
 #include "xbps_api_impl.h"
 
 /**
@@ -212,7 +213,7 @@ parse_option(char *line, size_t linelen, char **valp, size_t *vallen)
 	if (*p != '=')
 		return KEY_ERROR;
 
-	result = bsearch(&needle, keys, __arraycount(keys), sizeof(struct key), cmpkey);
+	result = bsearch(&needle, keys, ARRAY_SIZE(keys), sizeof(struct key), cmpkey);
 	if (result == NULL)
 		return KEY_ERROR;
 

diff --git a/lib/pkgdb_conversion.c b/lib/pkgdb_conversion.c
@@ -23,14 +23,16 @@
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#include <stdio.h>
+#include <sys/stat.h>
+
+#include <errno.h>
+#include <fcntl.h>
 #include <stdbool.h>
+#include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
-#include <errno.h>
-#include <sys/stat.h>
-#include <fcntl.h>
 
+#include "macro.h"
 #include "xbps_api_impl.h"
 
 static int
@@ -115,7 +117,7 @@ pkgdb038(struct xbps_handle *xhp, const char *opkgdb_plist)
 
 			key = xbps_dictionary_keysym_cstring_nocopy(obj2);
 			curobj = xbps_dictionary_get_keysym(pkgmetad, obj2);
-			for (uint8_t i = 0; i < __arraycount(excluded); i++) {
+			for (uint8_t i = 0; i < ARRAY_SIZE(excluded); i++) {
 				if (strcmp(excluded[i], key) == 0) {
 					skip = true;
 					break;

diff --git a/lib/transaction_files.c b/lib/transaction_files.c
@@ -32,8 +32,9 @@
 #include <stdlib.h>
 #include <string.h>
 
-#include "xbps_api_impl.h"
+#include "macro.h"
 #include "uthash.h"
+#include "xbps_api_impl.h"
 
 enum type {
 	TYPE_LINK = 1,
@@ -296,7 +297,7 @@ collect_obsoletes(struct xbps_handle *xhp)
 		/*
 		 * Make sure to not remove any symlink of root directory.
 		 */
-		for (uint8_t x = 0; x < __arraycount(basesymlinks); x++) {
+		for (uint8_t x = 0; x < ARRAY_SIZE(basesymlinks); x++) {
 			if (strcmp(item->file+1, basesymlinks[x]) == 0) {
 				found = true;
 				break;