2011/5/16 Eric Blake <eblake(a)redhat.com>:
On 05/01/2011 01:57 PM, Matthias Bolte wrote:
> Break long lines and change spacing of keyword arguments to match
> Python style standards better.
>
> No functional change included.
> ---
> src/esx/esx_vi_generator.py | 369 ++++++++++++++++++++++++++++--------------
> 1 files changed, 246 insertions(+), 123 deletions(-)
I don't know the python style standards, but I could at least review
this on the grounds of looking like safe whitespace changes.
ACK.
> @@ -830,7 +908,8 @@ class Object(Type):
>
> if self.features & Object.FEATURE__LIST:
> source += "/* esxVI_%s_SerializeList */\n" %
self.name
> - source += "ESX_VI__TEMPLATE__LIST__SERIALIZE(%s)\n\n"
% self.name
> + source += "ESX_VI__TEMPLATE__LIST__SERIALIZE(%s)\n\n"
\
> + % self.name
>
> # deserilaize
You may want to do a followup patch to fix typos like this.
As this one was the only typo I cloud find in the generator I've just
folded the fix into this patch and pushed the result.
Matthias