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: Anderson Lizardo <hidden>
Date: 2012-09-28 11:28:21

Hi Johan,

On Fri, Sep 28, 2012 at 7:08 AM, Johan Hedberg [off-list ref] wrote:
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.

Regards,
-- 
Anderson Lizardo
Instituto Nokia de Tecnologia - INdT
Manaus - Brazil
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help