Thread (8 messages) flat view 8 messages, 3 authors, 2012-09-28

Re: [PATCH v3 BlueZ 1/4] gatt: Add support for find included services

From: Johan Hedberg <hidden>
Date: 2012-09-28 11:32:12

Hi Lizardo,

On Fri, Sep 28, 2012, Anderson Lizardo wrote:
On Fri, Sep 28, 2012 at 7:08 AM, Johan Hedberg [off-list ref] wrote:
quoted
Btw, if you guys are wondering why the stricter coding style checks,
I've added the following to .git/hooks/pre-{commit,applypatch}:

git diff --cached | ~/src/linux/scripts/checkpatch.pl --no-signoff --ignore INITIALISED_STATIC,NEW_TYPEDEFS,VOLATILE --show-types --mailback -

You might want to do the same (with the correct path to checkpatch.pl)).
And thanks to Luiz for the idea (he's using the same when applying
patches).
Thanks for the very useful tip! But which checkpatch.pl version are
you using? Mine (from bluetooth-next tree) lacks --ignore and
--show-types options.
I'm using Linus' tree.

Johan
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help