Messages in this thread | | | Date | Thu, 07 Jul 2016 09:57:10 -0600 | From | "Jan Beulich" <> | Subject | Re: [Xen-devel] [PATCH] xen-blkback: constify instance of "struct attribute_group" |
| |
>>> On 07.07.16 at 17:37, <konrad.wilk@oracle.com> 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@suse.com> >> > Acked-by: Roger Pau Monné <roger.pau@citrix.com> >> > >> > 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.
Jan
| |