Re: checkpatch, a patch checking script.

Dave Jones
2007-04-28T17:11:19Z

On Sat, Apr 28, 2007 at 11:11:36AM -0500, Matt Mackall wrote:
> > I'm all ears for additional regexps, bug reports or other suggestions.
>
> Neat.
>
> Does it check for:
>
> functions marked extern?
> pulling in external functions or variables without a header file?
> return used as a function, eg return(foo);?

These sound a little more tricky than just dumb regexps.
I don't want to expand this to a fullblown C parser (given
that we have sparse which can do a better job), but I don't
object to adding some extra code to give the searches more
context.

Dave

--