Re: [RESEND PATCH] trivial: treewide: remove extra semicolon addedby module_init/exit
From: Felipe Balbi
Date: Wed Jun 06 2012 - 10:52:03 EST
On Wed, Jun 06, 2012 at 07:47:42AM -0700, Joe Perches wrote:
> On Wed, 2012-06-06 at 17:02 +0300, Felipe Balbi wrote:
> > The definition of module_init() and module_exit()
> > already added the final semicolon creating a situation
> > where most of our drivers end up with two semicolons
> > at the end of every module_init()/module_exit() call.
> >
> > To fix that, we add missing semicolons to the places
> > which didn't have them and remove the semicolons from
> > the definition of module_init/module_exit.
> >
> > Patch generated with the following sed script:
> >
> > $ sed -i 's/^module_\(init\|exit\)\s*\((\w\+)\)$/module_\1\2;/g' \
> > $(git ls-files)
>
> I don't think that script works completely.
> It doesn't find the module_init in fs/autofs4/init.c
hehe, there's a trailing whitespace at the end of that module_init()
call. Will fix the patch and resend.
--
balbi
Attachment:
signature.asc
Description: Digital signature