Re: [Xen-devel] [PATCH] xen-blkback: constify instance of "struct attribute_group"

From: Konrad Rzeszutek Wilk
Date: Fri Jul 08 2016 - 09:56:50 EST


On Thu, Jul 07, 2016 at 09:57:10AM -0600, Jan Beulich wrote:
> >>> On 07.07.16 at 17:37, <konrad.wilk@xxxxxxxxxx> wrote:
> > On Thu, Jul 07, 2016 at 10:52:18AM +0100, Andrew Cooper wrote:
> >> On 07/07/16 10:45, Roger Pau Monne wrote:
> >> > On Thu, Jul 07, 2016 at 01:38:58AM -0600, Jan Beulich wrote:
> >> >> The functions such get passed to have been taking pointers to const
> >> >> since at least 2.6.16.
> >> >>
> >> >> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> >> > Acked-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
> >> >
> >> > Although the wording in the commit message looks weird to me, but I'm not a
> >
> >> > native speaker anyway.
> >>
> >> As a native speaker, I can't parse it either.
> >>
> >> I think s/such/these/ is needed.
> >
> > The functions such as these have been taking pointers to const
> > since at least 2.6.16.
>
> I had taken Andrew's suggestion literally and changed it to "The
> functions these get passed to have been taking pointers to const
> since at least 2.6.16" for a possible (if needed) resubmission.

No need to resubmit. I will use Andrew's suggestion.
>
> Jan