On 06/06/2016 03:26 AM, Peter Krempa wrote:
On Fri, Jun 03, 2016 at 06:52:52 -0400, John Ferlan wrote:
> Just move the code into qemuBuildObjectSecretCommandLine.
>
> Signed-off-by: John Ferlan <jferlan(a)redhat.com>
> ---
> src/qemu/qemu_command.c | 57 +++++++++++--------------------------------------
> 1 file changed, 12 insertions(+), 45 deletions(-)
Again. If this is going to be reused in the monitor then it doesn't make
sense to move it to the builder directly. If it's not going to be used
with the monitor then it doesn't make sense to use the json parser and
the object formatter to achieve the same at all since it's rather
untrivial code that can be achieved with a single printf.
Huh? This patch just moves code from qemuBuildSecretInfoProps into
qemuBuildObjectSecretCommandLine as the only caller to the former was
the latter.
I can drop this though - it's not that important.
John