From 3d98bea2898d48fa5fcdcdd483a297c42e8140a8 Mon Sep 17 00:00:00 2001 From: illiliti Date: Wed, 25 Aug 2021 01:38:33 +0300 Subject: [PATCH] *: code cleanup --- udev.c | 2 +- udev_device.c | 6 ++---- udev_enumerate.c | 7 +++---- udev_list.c | 2 +- udev_monitor.c | 2 +- 5 files changed, 8 insertions(+), 11 deletions(-) diff --git a/udev.c b/udev.c index e886f19..ab5d33a 100644 --- a/udev.c +++ b/udev.c @@ -27,7 +27,7 @@ struct udev *udev_new(void) { struct udev *udev; - udev = calloc(1, sizeof(struct udev)); + udev = calloc(1, sizeof(*udev)); if (!udev) { return NULL; diff --git a/udev_device.c b/udev_device.c index 7957bb2..47dd00f 100644 --- a/udev_device.c +++ b/udev_device.c @@ -26,8 +26,6 @@ #include "udev.h" #include "udev_list.h" -// sizeof(unsigned long) == 4 * 8 == 32 on 32-bit systems. -// sizeof(unsigned long) == 8 * 8 == 64 on 64-bit systems. #ifndef LONG_BIT #define LONG_BIT (sizeof(unsigned long) * 8) #endif @@ -547,7 +545,7 @@ struct udev_device *udev_device_new_from_syspath(struct udev *udev, const char * return NULL; } - udev_device = calloc(1, sizeof(struct udev_device)); + udev_device = calloc(1, sizeof(*udev_device)); if (!udev_device) { return NULL; @@ -650,7 +648,7 @@ struct udev_device *udev_device_new_from_file(struct udev *udev, const char *pat return NULL; } - udev_device = calloc(1, sizeof(struct udev_device)); + udev_device = calloc(1, sizeof(*udev_device)); if (!udev_device) { fclose(file); diff --git a/udev_enumerate.c b/udev_enumerate.c index 449da4f..2dd7aee 100644 --- a/udev_enumerate.c +++ b/udev_enumerate.c @@ -275,8 +275,8 @@ static int scan_devices(struct udev_enumerate *udev_enumerate, const char *path) { struct udev_enumerate_thread *thread; pthread_mutex_t mutex; + int i, cnt, ret = 1; struct dirent **de; - int ret, cnt, i; cnt = scandir(path, &de, filter_dot, NULL); @@ -284,8 +284,7 @@ static int scan_devices(struct udev_enumerate *udev_enumerate, const char *path) return 0; } - ret = 1; - thread = calloc(cnt, sizeof(struct udev_enumerate_thread)); + thread = calloc(cnt, sizeof(*thread)); if (!thread) { ret = 0; @@ -368,7 +367,7 @@ struct udev_enumerate *udev_enumerate_new(struct udev *udev) return NULL; } - udev_enumerate = calloc(1, sizeof(struct udev_enumerate)); + udev_enumerate = calloc(1, sizeof(*udev_enumerate)); if (!udev_enumerate) { return NULL; diff --git a/udev_list.c b/udev_list.c index a41ec43..95b08e8 100644 --- a/udev_list.c +++ b/udev_list.c @@ -69,7 +69,7 @@ struct udev_list_entry *udev_list_entry_add(struct udev_list_entry *list_entry, } } - list_entry2 = calloc(1, sizeof(struct udev_list_entry)); + list_entry2 = calloc(1, sizeof(*list_entry2)); if (!list_entry2) { return NULL; diff --git a/udev_monitor.c b/udev_monitor.c index 2843742..1e79310 100644 --- a/udev_monitor.c +++ b/udev_monitor.c @@ -248,7 +248,7 @@ struct udev_monitor *udev_monitor_new_from_netlink(struct udev *udev, const char return NULL; } - udev_monitor = calloc(1, sizeof(struct udev_monitor)); + udev_monitor = calloc(1, sizeof(*udev_monitor)); if (!udev_monitor) { return NULL;