Re: [libvirt] [PATCH v2 0/6] Alter domain_conf to make use of autofree

On 3/1/19 8:38 AM, Erik Skultety wrote: A bit of history - based on the storage changes I made, I started thinking maybe it's time to just start the process of getting more code to use the VIR_AUTO* stuff. I figured domain_conf would be the most challenging and it's where I started (and before Peter wrote the VIR_AUTOUNREF patches). My initial pass was look for virBitmapPtr and VIR_AUTOFREE. I thought I got most although I do see a couple more that would be possible: * virDomainHugepagesFormatBuf for @nodeset * virDomainChrTargetDefFormat for @addr within TYPE_GUESTFWD * virDomainNetDefFormat for @str, @gueststr, & @hoststr within def->model processing * virDomainChannelDefCheckABIStability for @saddr & @daddr within TYPE_GUESTFWD Sure this is simple enough to do. Hopefully acceptible in one patch at the end of the series and not a multipatch step... VIR_AUTOPTR is beyond the scope of what I was doing and altering all the places for virObjectUnref really could have been done by when VIR_AUTOUNREF was introduced (VIR_XPATH_NODE_AUTORESTORE too). We keep adding this VIR_AUTO* things, but without someone actively trying to complete the work, it will never happen and we'll be continually stuck in this limbo state. For some it's more than first patch type contribution. I don't mind adding VIR_AUTOUNREF since it's already pushed, but adding new VIR_AUTOPTR's is an exercise for someone else eventually as far more than domain_conf would need to be touched in order to do it right. John I'll post a v3 shortly with the diffs requested as part of review and the few extra VIR_AUTOFREE's I found for you to look at.