Re: [PATCH 1/3] i/o bandwidth controller documentation

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Randy Dunlap
Date: Friday, June 20, 2008 - 10:08 am

On Fri, 20 Jun 2008 12:05:33 +0200 Andrea Righi wrote:


I would s/if/If/


s/the/The/


Please consistenly use "I/O" instead of "i/o".

Above comma makes a run-on sentence.  A period or semi-colon would be better IMO.


Ugh, this makes it look like the output does "pretty printing" (formatting),
which is generally not a good idea.  Let some app be responsible for that,
not the kernel.  Basically this means don't use leading spaces just to make the
":"s line up in the output.



Run-on sentence.  Change , to . (with This) or use ;


associated with (?)


---
~Randy
Linux Plumbers Conference, 17-19 September 2008, Portland, Oregon USA
http://linuxplumbersconf.org/
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 1/3] i/o bandwidth controller documentation, Andrea Righi, (Fri Jun 20, 3:05 am)
Re: [PATCH 1/3] i/o bandwidth controller documentation, Randy Dunlap, (Fri Jun 20, 10:08 am)
Re: [PATCH 1/3] i/o bandwidth controller documentation, Andrea Righi, (Sat Jun 21, 3:35 am)
Re: [PATCH 1/3] i/o bandwidth controller documentation, Randy Dunlap, (Sun Jun 22, 9:03 am)