On Wed, Sep 28, 2022 at 10:51:12 -0500, Jonathon Jongsma wrote:
On 9/19/22 9:09 AM, Peter Krempa wrote:
+{ + qemuNbdkitCaps *nbdkit = QEMU_NBDKIT_CAPS(object); + + g_clear_pointer(&nbdkit->path, g_free); + g_clear_pointer(&nbdkit->version, g_free); + g_clear_pointer(&nbdkit->flags, virBitmapFree); + + G_OBJECT_CLASS(qemu_nbdkit_caps_parent_class)->finalize(object); +} + + +void qemu_nbdkit_caps_init(qemuNbdkitCaps *caps)
Same here and also inconsistent capitalization format.
regarding the capitalization format, this is the implementation of a function that is declared by G_DEFINE_TYPE, so it needs to be formatted this way.
Ah, right. I forgot about that.
+ +typedef struct _qemuNbdkitCaps qemuNbdkitCaps; + +typedef enum {
You should go for the same markers you have when implementing the enum.
I'm afraid I can't quite figure out what you're trying to suggest here.
Yeah, I had to look back too to try to figure out what I meant :D. In the .c file where you define the conversion functions via the VIR_ENUM_IMPL macro you've used grouping separators (/* 0 */) similarly to what we do for the qemu capabilities, but not in the header. So they should be used in both places.