Re: [PATCH] Drop unused function declarations

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On a Tuesday in 2024, Michal Privoznik wrote:
When moving function and/or renaming them sometimes corresponding
change to corresponding header file is not done. This leaves us
with functions that are declared in header files, but nowhere
implemented. Drop such declarations.

Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
---

These were identified by a bash oneliner which basically tried to list
every "function" in every header file (function was matched as '\w\+(')
and for every function from the list, git grep was done to see if it
lives in a .c file.

It's nowhere exhaustive or perfect, but if found at least something.

src/ch/ch_monitor.h           |  4 ----
src/conf/device_conf.h        |  1 -
src/conf/virinterfaceobj.h    |  3 ---
src/conf/virstorageobj.h      |  3 ---
src/libxl/libxl_domain.h      |  3 ---
src/logging/log_handler.h     |  2 --
src/lxc/lxc_cgroup.h          |  1 -
src/lxc/lxc_hostdev.h         |  6 ------
src/lxc/lxc_monitor.h         |  3 ---
src/lxc/lxc_process.h         |  9 ---------
src/qemu/qemu_command.h       |  3 ---
src/qemu/qemu_domain.h        | 12 ------------
src/qemu/qemu_monitor.h       |  4 ----
src/qemu/qemu_monitor_json.h  | 29 -----------------------------
src/qemu/qemu_process.h       |  3 ---
src/rpc/virnetlibsshsession.h |  1 -
src/rpc/virnetserverclient.h  |  2 --
src/util/virmdev.h            |  3 ---
src/util/virnetdev.h          | 10 ----------
src/vz/vz_utils.h             |  8 --------
tests/testutilsqemu.h         |  3 ---
21 files changed, 113 deletions(-)


Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx>

Jano

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux