Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1504362 > unrolled thread

Re: module: remove trailing whitespace

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: module: remove trailing whitespace Jessica Yu <jeyu@redhat.com> - 2016-10-20 00:20 +0200
    Re: module: remove trailing whitespace Anson Jacob <ansonjacob.aj@gmail.com> - 2016-10-20 01:10 +0200

#1504362 — Re: module: remove trailing whitespace

+++ Anson Jacob [07/10/16 01:48 -0400]:
>remove whitespace on blank line
>
>Signed-off-by: Anson Jacob <ansonjacob.aj@gmail.com>

It would be nice to mention the exact checkpatch.pl error this fixes,
and include that in the changelog. Other than that, looks good to me.

>---
> include/linux/module.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/include/linux/module.h b/include/linux/module.h
>index 0c3207d..91df94f 100644
>--- a/include/linux/module.h
>+++ b/include/linux/module.h
>@@ -412,7 +412,7 @@ struct module {
> 	/* Protected by RCU and/or module_mutex: use rcu_dereference() */
> 	struct mod_kallsyms *kallsyms;
> 	struct mod_kallsyms core_kallsyms;
>-	
>+
> 	/* Section attributes */
> 	struct module_sect_attrs *sect_attrs;
>
>-- 
>2.7.4
>

[toc] | [next] | [standalone]

#1504401

On Wed, Oct 19, 2016 at 06:11:59PM -0400, Jessica Yu wrote:
> It would be nice to mention the exact checkpatch.pl error this fixes,
> and include that in the changelog. Other than that, looks good to me.

Thank you for your comments, will send a revised patch.

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web