]> git.ipfire.org Git - thirdparty/systemd.git/commit - src/network/networkd-link.c
tree-wide: drop redundant _cleanup_ macros (#8810)
authorLennart Poettering <lennart@poettering.net>
Wed, 25 Apr 2018 10:31:45 +0000 (12:31 +0200)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Wed, 25 Apr 2018 10:31:45 +0000 (12:31 +0200)
commit8e766630f006fcb17ad575bd4f3000e2d2dc890f
tree5b80e86c17142add180769d82db29e2627874304
parent0d6f0ea6c1994a5769704d2150ce907a1132a8c8
tree-wide: drop redundant _cleanup_ macros (#8810)

This drops a good number of type-specific _cleanup_ macros, and patches
all users to just use the generic ones.

In most recent code we abstained from defining type-specific macros, and
this basically removes all those added already, with the exception of
the really low-level ones.

Having explicit macros for this is not too useful, as the expression
without the extra macro is generally just 2ch wider. We should generally
emphesize generic code, unless there are really good reasons for
specific code, hence let's follow this in this case too.

Note that _cleanup_free_ and similar really low-level, libc'ish, Linux
API'ish macros continue to be defined, only the really high-level OO
ones are dropped. From now on this should really be the rule: for really
low-level stuff, such as memory allocation, fd handling and so one, go
ahead and define explicit per-type macros, but for high-level, specific
program code, just use the generic _cleanup_() macro directly, in order
to keep things simple and as readable as possible for the uninitiated.

Note that before this patch some of the APIs (notable libudev ones) were
already used with the high-level macros at some places and with the
generic _cleanup_ macro at others. With this patch we hence unify on the
latter.
74 files changed:
src/analyze/analyze.c
src/backlight/backlight.c
src/basic/blkid-util.h
src/basic/lockfile-util.h
src/core/device.c
src/core/swap.c
src/core/umount.c
src/gpt-auto-generator/gpt-auto-generator.c
src/journal-remote/journal-remote-write.h
src/journal-remote/journal-remote.c
src/journal/journalctl.c
src/libsystemd-network/dhcp-identifier.c
src/libsystemd-network/sd-dhcp-server.c
src/login/logind-acl.c
src/login/logind-core.c
src/login/logind-dbus.c
src/login/logind.c
src/login/sysfs-show.c
src/mount/mount-tool.c
src/network/netdev/bridge.c
src/network/netdev/geneve.c
src/network/netdev/netdev.c
src/network/netdev/netdev.h
src/network/networkd-address-label.c
src/network/networkd-address-label.h
src/network/networkd-address.c
src/network/networkd-address.h
src/network/networkd-dhcp4.c
src/network/networkd-dhcp6.c
src/network/networkd-fdb.c
src/network/networkd-fdb.h
src/network/networkd-ipv4ll.c
src/network/networkd-link.c
src/network/networkd-link.h
src/network/networkd-manager.c
src/network/networkd-manager.h
src/network/networkd-ndisc.c
src/network/networkd-network.c
src/network/networkd-network.h
src/network/networkd-radv.c
src/network/networkd-radv.h
src/network/networkd-route.c
src/network/networkd-route.h
src/network/networkd-routing-policy-rule.c
src/network/networkd-routing-policy-rule.h
src/network/networkd.c
src/network/test-network.c
src/nspawn/nspawn-network.c
src/nspawn/nspawn.c
src/rfkill/rfkill.c
src/shared/bootspec.c
src/shared/dissect-image.c
src/shared/install.c
src/shared/machine-image.c
src/shared/machine-pool.c
src/shared/path-lookup.h
src/shared/udev-util.h
src/systemctl/systemctl.c
src/sysv-generator/sysv-generator.c
src/test/test-libudev.c
src/test/test-path-lookup.c
src/test/test-udev.c
src/udev/ata_id/ata_id.c
src/udev/scsi_id/scsi_id.c
src/udev/udev-builtin-blkid.c
src/udev/udev-builtin-hwdb.c
src/udev/udev-builtin-net_id.c
src/udev/udev-builtin-path_id.c
src/udev/udevadm-control.c
src/udev/udevadm-info.c
src/udev/udevadm-monitor.c
src/udev/udevadm-test.c
src/udev/udevadm-trigger.c
src/udev/udevd.c