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

Re: [libvirt] [PATCH] esx: Reduce code duplication in generator



On 05/02/2013 07:09 AM, Matthias Bolte wrote:
> 2013/5/2 Eric Blake <eblake redhat com>:
>> On 05/01/2013 12:44 PM, Matthias Bolte wrote:
>>> ---
>>>  src/esx/esx_vi_generator.py |  430 ++++++++++++-------------------------------
>>>  1 file changed, 118 insertions(+), 312 deletions(-)
>>
>> My python is weak, so I reviewed this by comparing the generated files
>> before and after this patch; added comments in the generated code are
>> nice, but there are also some added frees.  Are these additions
>> intentional to fix a leak, or are they representing a bug in your patch?
>>  If intentional, then this is 1.0.5 material if you improve the commit
>> message and push in time; if accidental and no real bug is being fixed,
>> then a v2 should wait until after the release.
> 
> The changes in the generated files you're seeing here are intended,
> but they are coming from another patch that I pushed yesterday:
> 
> http://libvirt.org/git/?p=libvirt.git;a=commit;h=4e650435edeb2870c907721ea8b11380256bf6a7
> 
> This refactoring patch should not change the output at all. And it
> didn't in my test.

Then that must be an effect of me grabbing my snapshot at the wrong
point when swapping between incremental builds.  I'll try again, and
sorry for the hassle that my confusion caused...

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


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