Thread (24 messages) 24 messages, 4 authors, 2009-02-03

Re: [PATCH v2 0/4] virtio_net: Add MAC and VLAN filtering

From: Rusty Russell <hidden>
Date: 2009-01-30 05:03:51
Also in: kvm

On Friday 30 January 2009 09:35:02 Alex Williamson wrote:
I'm not completely sure it's everything
you're looking for Rusty, but it does seem cleaner.  Let me know if
this is closer to what you're thinking.
Yep, now I'll go through the patches more carefully.

I know it's a pain to get this kind of nit-picking feedback, but it's an ordeal I force on everyone the first time they make a major contribution; it ensures the code "feels" consistent and means that when I read the code in future I'm not surprised by things which aren't *quite* how I expect.

Note that I tend to comment on *everything* where I would have done it differently.  So I expect you to argue at least half of them.

Thanks!
Rusty.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help