2021-06-09 14:02:41

by Johannes Berg

[permalink] [raw]
Subject: Re: [PATCH] nl80211: fix a mistake in grammar

On Wed, 2021-06-09 at 01:15 -0700, [email protected] wrote:
>
> - /* don't allow or configure an mcast address */
> + /* don't allow or configure a mcast address */

Arguable? I'd say "an M-cast" address, and I guess that's why it's
written that way. Not sure how else you'd say it, unless you always
expand it to "multicast" when reading.

johannes


2021-06-09 15:57:51

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH] nl80211: fix a mistake in grammar

On 6/9/21 1:55 AM, Johannes Berg wrote:
> On Wed, 2021-06-09 at 01:15 -0700, [email protected] wrote:
>>
>> - /* don't allow or configure an mcast address */
>> + /* don't allow or configure a mcast address */
>
> Arguable? I'd say "an M-cast" address, and I guess that's why it's
> written that way. Not sure how else you'd say it, unless you always
> expand it to "multicast" when reading.

Agreed, it's ok as is.