On Tue, May 15, 2018 at 12:35:34PM -0400, Doug Ledford wrote: > On Mon, 2018-05-14 at 09:51 -0500, Steve Wise wrote: > > > > On 5/13/2018 8:24 AM, Leon Romanovsky wrote: > > > On Mon, May 07, 2018 at 08:53:16AM -0700, Steve Wise wrote: > > > > This enhancement allows printing rdma device-specific state, if provided > > > > by the kernel. This is done in a generic manner, so rdma tool doesn't > > > > > > Double space between "." and "This". > > > > > > > need to know about the details of every type of rdma device. > > > > > > > > Driver attributes for a rdma resource are in the form of <key, > > > > [print_type], value> tuples, where the key is a string and the value can > > > > be any supported driver attribute. The print_type attribute, if present, > > > > > > ditto > > > > I'll fix these. > > Fix it if you want, but don't do it because Leon told you to. A double > space after period is perfectly acceptable. It is very controversial thing [1], "Most style guides indicate that single sentence spacing is proper for final or published work today, and most publishers require manuscripts to be submitted as they will appear in publication—single sentence spaced." [1] https://en.wikipedia.org/wiki/Sentence_spacing > > -- > Doug Ledford <dledford@xxxxxxxxxx> > GPG KeyID: B826A3330E572FDD > Key fingerprint = AE6B 1BDA 122B 23B4 265B 1274 B826 A333 0E57 2FDD
Attachment:
signature.asc
Description: PGP signature
