On 10/01/2018 01:58 PM, Erik Skultety wrote: > On Mon, Oct 01, 2018 at 12:57:18PM +0200, Fabiano FidĂȘncio wrote: >> Signed-off-by: Fabiano FidĂȘncio <fiden...@redhat.com> >> --- >> I went through the commits since last release and added a few notes >> which seem to be relevant for the news. >> >> In case you notice there's something missing, please, either send me the >> text to add it (and I'll re-sping this patch) and add the text before >> merging this one. >> >> Changes since v1: >> - Actually run make-check in order to be sure the changes are fine. >> --- >> docs/news.xml | 69 +++++++++++++++++++++++++++++++++++++++++++++++++++ >> 1 file changed, 69 insertions(+) >>
> Now, this is just a matter of preference, you did a good job, but I'd probably > also mention the following: > 09d35afd2c util: file: introduce VIR_AUTOCLOSE macro to close fd of the file > automatic Is it something that should interest our users? In my view it's just a matter of our internal implementation. We haven't mention VIR_AUTOFREE() back in the day. Michal -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list