Re: [PATCH virt-viewer 6/9] Simplify virt_viewer_initial_connect()

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

 




On Fri, Nov 14, 2014 at 2:24 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
Well, I beg to disagree on the 'easy to read' part as I would not have split
it if I had found it that easy to read ;)

I agree, the original code is not so easy to read, the rewritten version however is.
 
"fairly small" and "easy to read" are really secondary to "one change by
patch", which I don't think the initial patch achieved.

It is a rewrite, really no steps involved when I did it, it's hard to come up with steps which I think are artificial, but as I said, let's go with your 3 patches then


--
Marc-André Lureau
_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list

[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux