|
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
[Prev in Thread] | Current Thread | [Next in Thread] |