Re: [PATCH] test driver: File test_driver.c was splitted into smaller files.

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

 



On 09/16/2016 08:30 AM, Daniel P. Berrange wrote:
> On Fri, Sep 16, 2016 at 03:06:27PM +0200, Tomáš Ryšavý wrote:
>> Because the file had over 7000 lines. Now its splitted into 5
>> smaller files. It looks clearer and it's better for reading the code.
>>
>> Signed-off-by: Tomáš Ryšavý <tom.rysavy.0@xxxxxxxxx>
>> ---

> 
> Doing this is all in one patch is *way* too much change at
> once.
> 
> Please split this into at least 4 patches, one for each new new .c
> file you create.

Agreed.  Here's another hint: code motion patches can be made VERY easy
to review, but only if your commit is limited to moving contents from
one file to another in the same ordering (any time you scattershot
functions from one file into multiple others, or reorder things on move,
this trick doesn't work):

http://wiki.qemu.org/Contribute/SubmitAPatch#Make_code_motion_patches_easy_to_review

(Not all of the qemu patch submission rules apply to libvirt, but this
hint about code motion reviews can be applied to any project)

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

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]