Re: [Outreachy kernel] [PATCH 00/10] staging: iio: Remove exceptional & on functions name

From: Julia Lawall
Date: Sat Mar 11 2017 - 06:41:32 EST




On Sat, 11 Mar 2017, SIMRAN SINGHAL wrote:

> On Sat, Mar 11, 2017 at 2:57 PM, Julia Lawall <julia.lawall@xxxxxxx> wrote:
> >
> >
> > On Sat, 11 Mar 2017, SIMRAN SINGHAL wrote:
> >
> >> On Sat, Mar 11, 2017 at 12:12 PM, Julia Lawall <julia.lawall@xxxxxxx> wrote:
> >> >
> >> >
> >> > On Sat, 11 Mar 2017, simran singhal wrote:
> >> >
> >> >> This patch-series removes exceptional & on functions name.
> >> >
> >> > The semantic patch shown does nothing to check that the use of & is
> >> > exception in the given file. It just removes all the & on function names.
> >> >
> >> yes, I used this semantic patch only for only finding the functions with &
> >> and then I find out this drivers and use semantic patch for removing &.
> >
> > The commit message should be written with respect to what was actually
> > done, and not copied from a patch that actually did something else. The
> > original semantic patch, that was probably only put in the cover letter,
> > actually checked if the file mostly used function pointers in a particular
> > way.
> >
> Julia, so what should I do?
> I should not include this semantic patch as it does not check that that the use
> of & is exception in the given file.
> OR
> As I did it manually so I should mention this.

If you checked it manually, just mention that. I don't have the actual
patches handy any more, but you could also say something like:

Remove & from function pointers to conform to the style found elsewhere in
the file. Done using the following semantic patch.

This would be ok if the included semantic patch makes the change. It
doesn't suggest that the semantic patch checkked for the excpeptionality.

julia

>
> > julia
> >
> >>
> >> > julia
> >> >
> >> >>
> >> >> simran singhal (10):
> >> >> staging: iio: ad7192: Remove exceptional & on function name
> >> >> staging: iio: ad7780: Remove exceptional & on function name
> >> >> staging: iio: cdc: ad7746: Remove exceptional & on function name
> >> >> staging: iio: cdc: ad7152: Remove exceptional & on function name
> >> >> staging: iio: adis16240: Remove exceptional & on function name
> >> >> staging: iio: adis16201: Remove exceptional & on function name
> >> >> staging: iio: adis16209: Remove exceptional & on function name
> >> >> staging: iio: adis16203: Remove exceptional & on function name
> >> >> staging: iio: resolver: Remove exceptional & on function name
> >> >> staging: iio: gyro: Remove exceptional & on function name
> >> >>
> >> >> drivers/staging/iio/accel/adis16201.c | 4 ++--
> >> >> drivers/staging/iio/accel/adis16203.c | 4 ++--
> >> >> drivers/staging/iio/accel/adis16209.c | 4 ++--
> >> >> drivers/staging/iio/accel/adis16240.c | 4 ++--
> >> >> drivers/staging/iio/adc/ad7192.c | 12 ++++++------
> >> >> drivers/staging/iio/adc/ad7780.c | 2 +-
> >> >> drivers/staging/iio/cdc/ad7152.c | 6 +++---
> >> >> drivers/staging/iio/cdc/ad7746.c | 4 ++--
> >> >> drivers/staging/iio/gyro/adis16060_core.c | 2 +-
> >> >> drivers/staging/iio/resolver/ad2s1200.c | 2 +-
> >> >> drivers/staging/iio/resolver/ad2s90.c | 2 +-
> >> >> 11 files changed, 23 insertions(+), 23 deletions(-)
> >> >>
> >> >> --
> >> >> 2.7.4
> >> >>
> >> >> --
> >> >> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> >> >> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@xxxxxxxxxxxxxxxxx
> >> >> To post to this group, send email to outreachy-kernel@xxxxxxxxxxxxxxxxx
> >> >> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/1489203401-17518-1-git-send-email-singhalsimran0%40gmail.com.
> >> >> For more options, visit https://groups.google.com/d/optout.
> >> >>
> >>
> >> --
> >> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> >> To unsubscribe from this group and stop receiving emails from it, send an email to outreachy-kernel+unsubscribe@xxxxxxxxxxxxxxxxx
> >> To post to this group, send email to outreachy-kernel@xxxxxxxxxxxxxxxxx
> >> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/CALrZqyODTiF%2BgdwwYnmL1W1oi-R0u2rM4c40jb5iWXFxJpfdJw%40mail.gmail.com.
> >> For more options, visit https://groups.google.com/d/optout.
> >>
>