On Mon, Dec 16, 2024 at 12:41:04 +0300, Alexander Kuznetsov wrote: > These functions return value is invariant since 18f3771, so change > its type and remove all dependent checks. > > Best regards, > Alexander Kuznetsov > > --- > v3: > - Fix indentation errors. > - 2: add missing virSCSIDeviceSetUsedBy call > - 3: remove non-relevant cosmetic fix > > v2: > - One function per commit. > > Alexander Kuznetsov (4): > util: Change return type of virPCIDeviceSetUsedBy to void > util: Change return type of virSCSIDeviceSetUsedBy to void > util: Change return type of virSCSIVHostDeviceSetUsedBy to void > logging: Change return type of virLogDaemonConfigFilePat to void > > src/hypervisor/virhostdev.c | 22 ++++++++-------------- > src/logging/log_daemon.c | 8 ++------ > src/logging/log_daemon_config.c | 4 +--- > src/logging/log_daemon_config.h | 2 +- > src/util/virpci.c | 4 +--- > src/util/virpci.h | 6 +++--- > src/util/virscsi.c | 4 +--- > src/util/virscsi.h | 6 +++--- > src/util/virscsivhost.c | 4 +--- > src/util/virscsivhost.h | 6 +++--- > tests/virscsitest.c | 6 ++---- > 11 files changed, 26 insertions(+), 46 deletions(-) Reviewed-by: Jiri Denemark <jdenemar@xxxxxxxxxx> and pushed. Thanks.