All of lore.kernel.org
 help / color / mirror / Atom feed
From: Randy Dunlap <rdunlap@infradead.org>
To: Theodore Ts'o <tytso@mit.edu>, Al Viro <viro@zeniv.linux.org.uk>
Cc: linux-doc@vger.kernel.org, Jonathan Corbet <corbet@lwn.net>,
	linux-fsdevel@vger.kernel.org,
	linux-f2fs-devel@lists.sourceforge.net,
	linux-xfs@vger.kernel.org, Christian Brauner <brauner@kernel.org>,
	Seth Forshee <sforshee@kernel.org>
Subject: Re: [PATCH] Documentation: filesystems: correct possessive "its"
Date: Tue, 30 Aug 2022 18:22:22 -0700	[thread overview]
Message-ID: <ab1a33e5-01b9-79d4-662e-44c6e0e74113@infradead.org> (raw)
In-Reply-To: <Yw6Hp8l/7p3wbiGq@mit.edu>



On 8/30/22 14:56, Theodore Ts'o wrote:
> On Tue, Aug 30, 2022 at 10:01:33PM +0100, Al Viro wrote:
>> On Mon, Aug 29, 2022 at 04:54:29PM -0700, Randy Dunlap wrote:
>>>  compress_log_size=%u	 Support configuring compress cluster size, the size will
>>> -			 be 4KB * (1 << %u), 16KB is minimum size, also it's
>>> +			 be 4KB * (1 << %u), 16KB is minimum size, also its
>>>  			 default size.
>>
>> That one doesn't look like possesive to me - more like "default size is 16KB and
>> values below that are not allowed"...

I have to disagree about the possessive part...

> That being said, it could also be rewritten to be easier to
> understand.  e.g., "The default and minimum size is 16kb."

but sure, it can be rewritten.

thanks.
-- 
~Randy

WARNING: multiple messages have this Message-ID (diff)
From: Randy Dunlap <rdunlap@infradead.org>
To: Theodore Ts'o <tytso@mit.edu>, Al Viro <viro@zeniv.linux.org.uk>
Cc: Christian Brauner <brauner@kernel.org>,
	Jonathan Corbet <corbet@lwn.net>,
	linux-doc@vger.kernel.org,
	linux-f2fs-devel@lists.sourceforge.net,
	linux-xfs@vger.kernel.org, Seth Forshee <sforshee@kernel.org>,
	linux-fsdevel@vger.kernel.org
Subject: Re: [f2fs-dev] [PATCH] Documentation: filesystems: correct possessive "its"
Date: Tue, 30 Aug 2022 18:22:22 -0700	[thread overview]
Message-ID: <ab1a33e5-01b9-79d4-662e-44c6e0e74113@infradead.org> (raw)
In-Reply-To: <Yw6Hp8l/7p3wbiGq@mit.edu>



On 8/30/22 14:56, Theodore Ts'o wrote:
> On Tue, Aug 30, 2022 at 10:01:33PM +0100, Al Viro wrote:
>> On Mon, Aug 29, 2022 at 04:54:29PM -0700, Randy Dunlap wrote:
>>>  compress_log_size=%u	 Support configuring compress cluster size, the size will
>>> -			 be 4KB * (1 << %u), 16KB is minimum size, also it's
>>> +			 be 4KB * (1 << %u), 16KB is minimum size, also its
>>>  			 default size.
>>
>> That one doesn't look like possesive to me - more like "default size is 16KB and
>> values below that are not allowed"...

I have to disagree about the possessive part...

> That being said, it could also be rewritten to be easier to
> understand.  e.g., "The default and minimum size is 16kb."

but sure, it can be rewritten.

thanks.
-- 
~Randy


_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

  reply	other threads:[~2022-08-31  1:22 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-29 23:54 [PATCH] Documentation: filesystems: correct possessive "its" Randy Dunlap
2022-08-29 23:54 ` [f2fs-dev] " Randy Dunlap
2022-08-30  0:16 ` Darrick J. Wong
2022-08-30  0:16   ` [f2fs-dev] " Darrick J. Wong
2022-08-30  7:50 ` Christian Brauner
2022-08-30  7:50   ` [f2fs-dev] " Christian Brauner
2022-08-30 18:12 ` Jaegeuk Kim
2022-08-30 18:12   ` [f2fs-dev] " Jaegeuk Kim
2022-08-30 21:01 ` Al Viro
2022-08-30 21:01   ` [f2fs-dev] " Al Viro
2022-08-30 21:56   ` Theodore Ts'o
2022-08-30 21:56     ` [f2fs-dev] " Theodore Ts'o
2022-08-31  1:22     ` Randy Dunlap [this message]
2022-08-31  1:22       ` Randy Dunlap

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ab1a33e5-01b9-79d4-662e-44c6e0e74113@infradead.org \
    --to=rdunlap@infradead.org \
    --cc=brauner@kernel.org \
    --cc=corbet@lwn.net \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-f2fs-devel@lists.sourceforge.net \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-xfs@vger.kernel.org \
    --cc=sforshee@kernel.org \
    --cc=tytso@mit.edu \
    --cc=viro@zeniv.linux.org.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.