Re: [libvirt] [PATCH v3 04/17] qemuBuildMemoryBackendStr: Reorder args and update comment

2017-03-14 Thread John Ferlan
On 03/09/2017 11:06 AM, Michal Privoznik wrote: > Frankly, this function is one big mess. A lot of arguments, > complicated behaviour. It's really surprising that arguments were > in random order (input and output arguments were mixed together), > the documentation was outdated, the description

[libvirt] [PATCH v3 04/17] qemuBuildMemoryBackendStr: Reorder args and update comment

2017-03-09 Thread Michal Privoznik
Frankly, this function is one big mess. A lot of arguments, complicated behaviour. It's really surprising that arguments were in random order (input and output arguments were mixed together), the documentation was outdated, the description of return values was bogus. Signed-off-by: Michal