qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 13/22] qapi: add code generators for QMP command


From: Anthony Liguori
Subject: Re: [Qemu-devel] [PATCH 13/22] qapi: add code generators for QMP command marshaling
Date: Mon, 07 Mar 2011 07:44:44 -0600
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.15) Gecko/20101027 Lightning/1.0b1 Thunderbird/3.0.10

On 03/07/2011 07:27 AM, Stefan Hajnoczi wrote:
On Mon, Mar 7, 2011 at 1:22 AM, Anthony Liguori<address@hidden>  wrote:
+def print_definition(name, required, optional, retval):
This function is pretty long.  Is there a logical way to split it up
into several smaller meaningful functions?

It seems long because of the code generation but there's really not an awful lot of logic in it.

It might be worth spending a little time looking at a template system. I'm not terribly familiar with any of the popular ones and I think they usually focus on HTML generation but it might help readability.

Regards,

Anthony Liguori

Stefan





reply via email to

[Prev in Thread] Current Thread [Next in Thread]