Re: [Qemu-devel] [PATCH v2 00/17] introduce OptsVisitor, rebase -net/-netdev parsing

[Luiz Capitulino <lcapitulino@redhat.com>]

On Wed, 13 Jun 2012 10:22:31 +0200
Laszlo Ersek <lersek@redhat.com> wrote:

> Inspired by [1], the first half of this series attempts to implement a new
> visitor that should clean up defining and processing command line options.
> For a more detailed description, please see "[PATCH 05/17] qapi: introduce
> OptsVisitor".
> 
> The second half converts -net/-netdev parsing to the new visitor.

The general approach looks fine to me, I've made comments to individual patches
and have two general comments:

 1. This doesn't build for me:

In file included from /home/lcapitulino/work/src/qmp-unstable/net/slirp.c:24:0:
/home/lcapitulino/work/src/qmp-unstable/net/slirp.h:41:28: error: unknown type name ‘QemuOptsList’
/home/lcapitulino/work/src/qmp-unstable/net/slirp.c:741:5: error: no previous prototype for ‘net_slirp_parse_legacy’ [-Werror=missing-prototypes]
cc1: all warnings being treated as errors
make: *** [net/slirp.o] Error 1
make: *** Waiting for unfinished jobs....

 2. I don't think this should go in through qmp's branch because this is more
    about QemuOpts than about QMP. I suggest three alternatives:

      - If you're going to go forward and convert more users, then I think
        you should open your own branch, send pull requests etc

      - Go through some -net three

      - Ask Anthony to apply this directly

    I'll, of course, review it though