[libvirt] [PATCH] Simplify virDomainParseMemory
Andrea Bolognani
abologna at redhat.com
Fri Jan 29 17:47:39 UTC 2016
On Fri, 2016-01-29 at 18:12 +0100, Ján Tomko wrote:
> Do not store the return value of virDomainParseScaledValue,
> it was overwritten anyway.
>
> Delete the cleanup label, there is nothing to clean up.
> ---
> src/conf/domain_conf.c | 16 +++++-----------
> 1 file changed, 5 insertions(+), 11 deletions(-)
>
> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
> index 1ea74a6..f663969 100644
> --- a/src/conf/domain_conf.c
> +++ b/src/conf/domain_conf.c
> @@ -7547,28 +7547,22 @@ virDomainParseMemory(const char *xpath,
> bool required,
> bool capped)
> {
> - int ret = -1;
> unsigned long long bytes, max;
>
> max = virMemoryMaxValue(capped);
>
> - ret = virDomainParseScaledValue(xpath, units_xpath, ctxt,
> - &bytes, 1024, max, required);
> - if (ret < 0)
> - goto cleanup;
> + if (virDomainParseScaledValue(xpath, units_xpath, ctxt,
> + &bytes, 1024, max, required) < 0)
> + return -1;
>
> /* Yes, we really do use kibibytes for our internal sizing. */
> *mem = VIR_DIV_UP(bytes, 1024);
>
> if (*mem >= VIR_DIV_UP(max, 1024)) {
> virReportError(VIR_ERR_OVERFLOW, "%s", _("size value too large"));
> - ret = -1;
> - goto cleanup;
> + return -1;
> }
> -
> - ret = 0;
> - cleanup:
> - return ret;
> + return 0;
> }
You might want to include the 'conf:' prefix in the summary.
ACK regardless :)
Cheers.
--
Andrea Bolognani
Software Engineer - Virtualization Team
More information about the libvir-list
mailing list