On 05/04/2016 10:30 AM, Erik Skultety wrote:
> Now that methods to free logging related lists were introduced, put them to a
> use.
> ---
> src/util/virlog.c | 15 ++++-----------
> 1 file changed, 4 insertions(+), 11 deletions(-)
>
> diff --git a/src/util/virlog.c b/src/util/virlog.c
> index a1f5872..7e0936c 100644
> --- a/src/util/virlog.c
> +++ b/src/util/virlog.c
> @@ -239,11 +239,8 @@ virLogSetDefaultPriority(virLogPriority priority)
> static void
> virLogResetFilters(void)
> {
> - size_t i;
> -
> - for (i = 0; i < virLogNbFilters; i++)
> - VIR_FREE(virLogFilters[i]->match);
> - VIR_FREE(virLogFilters);
> + virLogFilterListFree(virLogFilters, virLogNbFilters);
> + virLogFilters = NULL;
> virLogNbFilters = 0;
> virLogFiltersSerial++;
> }
> @@ -321,12 +318,8 @@ virLogFilterNew(const char *match,
> static void
> virLogResetOutputs(void)
> {
> - size_t i;
> -
> - for (i = 0; i < virLogNbOutputs; i++)
> - virLogOutputFree(virLogOutputs[i]);
> -
> - VIR_FREE(virLogOutputs);
> + virLogOutputListFree(virLogOutputs, virLogNbOutputs);
> + virLogOutputs = NULL;
> virLogNbOutputs = 0;
> }
>
>
I'd rather see this patch split and combined in with the previous two patches.
Like you did for the virLogOutputFree patch. With those changes, ACK to the
previous two patches, and would be fine to push now since this is a reasonable
cleanup IMO
- Cole
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
