The commit summary is too broad. A prefix to say which part of the codebase is touched would be nice: util: convert char pointers to use g_autofree
On a Friday in 2020, Ryan Gahagan wrote:
From: bschoney <bscho...@utexas.edu> additional conversions to the GLib API in src/util per issue #11.
Please let me know if there are additional changes I should make in the files updated so far. I intend to submit work on additional files, and I want to be sure the changes so far are correct.
The changes itself look good.
Related issue: https://gitlab.com/libvirt/libvirt/-/issues/11 Signed-off-by: bschoney <bscho...@utexas.edu>
Can you change both the author field and the sign-off to contain your full name? I would fix it up before merging the patch, but touching these two fields is off-limits to me. Something like git config --global user.name "Your Name" to set it in ~/.gitconfig and git commit --amend --reset-author -s should do the trick.
--- src/util/vircgroupv1.c | 3 +-- src/util/virhostcpu.c | 4 +--- src/util/virlockspace.c | 6 ++---- src/util/virmacmap.c | 3 +-- src/util/virresctrl.c | 25 ++++++++----------------- src/util/virsysinfo.c | 9 +++------ 6 files changed, 16 insertions(+), 34 deletions(-)
With the full name added: Reviewed-by: Ján Tomko <jto...@redhat.com> Jano
signature.asc
Description: PGP signature
