[libvirt] [PATCH 11/24] conf: numa: Refactor logic in virDomainNumatuneParseXML

[John Ferlan]

On 02/16/2015 01:51 PM, Peter Krempa wrote:
> Shuffling around the logic will allow to simplify the code quite a bit.
> As an additional bonus the change in the logic now reports an error if
> automatic placement is selected and individual placement is configured.
> ---
>  src/conf/numa_conf.c | 53 +++++++++++++++++++++-------------------------------
>  1 file changed, 21 insertions(+), 32 deletions(-)
> 

hmm... perhaps one extra thing that could be done...

Now that the last/5th param to virDomainNumatuneSet isn't NULL for any
caller (and the 4th isn't -1) - it looks like to me you can also modify
numa_conf.h to add a ATTRIBUTE_NONNULL(5) for the function...

ACK

John