From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55752) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zm4Nj-00009o-B4 for qemu-devel@nongnu.org; Tue, 13 Oct 2015 14:33:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Zm4Ne-0003RH-AK for qemu-devel@nongnu.org; Tue, 13 Oct 2015 14:33:03 -0400 Received: from mx1.redhat.com ([209.132.183.28]:58967) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Zm4Ne-0003Qb-5p for qemu-devel@nongnu.org; Tue, 13 Oct 2015 14:32:58 -0400 From: Markus Armbruster References: <1444710158-8723-1-git-send-email-eblake@redhat.com> <1444710158-8723-16-git-send-email-eblake@redhat.com> <87zizm6c67.fsf@blackfin.pond.sub.org> <561D24CA.7030804@redhat.com> <87mvvmvghf.fsf@blackfin.pond.sub.org> <561D42C3.1050207@redhat.com> Date: Tue, 13 Oct 2015 20:32:54 +0200 In-Reply-To: <561D42C3.1050207@redhat.com> (Eric Blake's message of "Tue, 13 Oct 2015 11:43:31 -0600") Message-ID: <87wpuqmxex.fsf@blackfin.pond.sub.org> MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Qemu-devel] [PATCH v8 15/18] qapi: Move duplicate member checks to schema check() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eric Blake Cc: qemu-devel@nongnu.org, Michael Roth Eric Blake writes: > On 10/13/2015 11:13 AM, Markus Armbruster wrote: > >>>> >>>> I've come to the conclusion that we should get rid of the self-inflicted >>>> pain before we attempt to detect all collisions. >>> >>> Then that sounds like I should try harder to get the kind/type naming, >>> the boxed base naming, and even the anonymous union naming all hoisted >>> into this subset, and spin a v9? >> >> I can take PATCH 01-09,12 into my tree right away, with PATCH 07's two >> redundant is_implicit() methods dropped, and PATCH 12's comment touched >> up. > > Okay. Done & pushed to http://repo.or.cz/qemu/armbru.git branch qapi-next. >> >> I could take PATCH 10, but let's at least try to make a plan for >> c_name() first. If we fail, I'll take the patch, perhaps less the % to >> + change, and we'll revisit c_name() later when we see more clearly. > > At this point, I'm not sure whether 10 disappears completely after the > type/kind fix, so that alone is a good enough reason to leave 10 out of > your tree for another round. > >> >> You want to move PATCH 11 to later in the queue, and I like that. >> >> PATCH 13 needs a fix squashed in, and a few nits touched up. If you >> want me to do that on commit, please propose a patch for me to squash >> in. But a respin is probably easier for all. >> >> PATCH 14 is fine, but it depends on 13. >> >> I haven't finished review of PATCH 15-18. >> >> Taken together, I think the easiest way forward is I take 01-09,12, and >> you respin the rest after we finish its review. Makes sense? >> > > Sounds like we're agreed then: take the obvious patches into your tree, > and let me rework the tail of this subset on top of cleanups that reduce > self-inflicted collisions. Yes, please. I'll try to review v8 16-18 quickly.