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

Re: [Xen-devel] [PATCH v5 04/17] xenctx: Add command line options -b (--bytes-per-line) and -l (--lines)



On 03/27/14 11:55, Ian Campbell wrote:
On Mon, 2014-03-24 at 12:58 -0400, Don Slutz wrote:
On 03/24/14 06:44, Ian Campbell wrote:
I think we are way past intrusive...


Source code yes.  Output before v8 by default no.
The v8 drop of -T is the 1st visible change from just running it.

Please can you get out of the habit of just cutting and pasting emails
in response to questions, it's very hard to make sense of what you are
trying to say when you do this. Just answer the question directly.

Still adjusting to the right level.  Sorry about the cost to you.

[...]
I don't want things over 80 columns either.

I want to know that you have thought about the tradeoffs of adding a new
option and concluded that a new option is better than changing the
defaults -- the obvious way to stop me questioning this sort of thing
would be to explain the choice in the commit message in the first place.

Good to know.  I hope the commit message changes in v8 help.

   -Don Slutz

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.