[pve-devel] [PATCH cluster 1/5] pmxcfs: status: add one more missing g_free
Thomas Lamprecht
t.lamprecht at proxmox.com
Fri Aug 1 14:52:24 CEST 2025
On Fri, 01 Aug 2025 11:54:27 +0200, Lukas Wagner wrote:
> Otherwise we leak the memory that was already allocated for `filename`.
>
>
Applied, thanks!
But can you please add a cover-letter for any actual patch series (i.e., not
just a single patch), as that provides some value even if it's empty, as it
makes it very clear if trailers like T-b or R-b are intended for the full
series or just a single patch out of it. b4 also uses the same logic to
determine for which commits to apply the R-b and T-b too, so here I had to
manually add Maximiliano's R-b and T-b to patch 2 to 5, as from b4's POV they
where only supplied for patch 1/5.
A diffstat over all patches is a side benefit, a short sentence as overview
might be nice too, but for things like these here it can be even fine to not
have that.
Not a big thing, but would make my (and others using b4) life a bit easier and
better ensure that tests and reviews get encoded into git, providing some
recognition of that important work.
[1/5] pmxcfs: status: add one more missing g_free
commit: 8239d5c3657a0e3fe751689bce920b7c2c87e531
[2/5] pmxcfs: status: avoid unnecessary string allocations
commit: 426cb07ed853db202aaa219b06be80e7bbb4c74c
[3/5] pmxcfs: status: call g_strdup instead of g_strdup_printf when no formatting is needed
commit: c66917260e8bdf2a76878baee0ba91c6f4469207
[4/5] pmxcfs: status: remove string literal params for g_strdup_printf
commit: 0795244f98b6e5e34491b9a1b3967d5bf935afd8
[5/5] pmxcfs: status: avoid g_strdup when not needed
commit: 054995ba3d43e2bfe1e8fcd3d63549e3579e6ef9
More information about the pve-devel
mailing list