From: Simon Wunderlich <sw@simonwunderlich.de>
To: b.a.t.m.a.n@lists.open-mesh.org
Cc: Sven Eckelmann <sven@narfation.org>
Subject: Re: [B.A.T.M.A.N.] alfred: checkpatch cleanup
Date: Mon, 16 Mar 2015 17:10:06 +0100 [thread overview]
Message-ID: <6847523.RbkIcKIa8T@prime> (raw)
In-Reply-To: <1426333379-17777-1-git-send-email-sven@narfation.org>
[-- Attachment #1: Type: text/plain, Size: 457 bytes --]
On Saturday 14 March 2015 12:42:45 Sven Eckelmann wrote:
> Hi,
>
> just some patches which should address some of the checkpatch
> errors/warnings. It doesn't address things in list.h, debugfs.(c|h) or
> vis/gpsd.
>
> Kind regards,
> Sven
Thanks a lot Sven! Applied the whole series except for patch 14, since I don't
really like that comment style and don't see much value in making the comments
more ugly (for my personal taste).
Thanks,
Simon
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 181 bytes --]
prev parent reply other threads:[~2015-03-16 16:10 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-14 11:42 [B.A.T.M.A.N.] alfred: checkpatch cleanup Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 01/14] alfred: Add spaces around operators Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 02/14] alfred: checkpatch - else is not generally useful after a break or return Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 03/14] alfred: Reduce number of lines with more than 80 chars Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 04/14] alfred: Remove double semicolons Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 05/14] alfred: Remove trailing whitespace at end of line Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 06/14] alfred: Remove spaces before tabs Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 07/14] alfred: Remove else and break after return Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 08/14] alfred: Remove multiple blank lines Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 09/14] alfred: Fix typo transfered -> transferred Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 10/14] alfred: Fix alignment of multi-line statements Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 11/14] alfred: Simplify comparison of pointer with NULL Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 12/14] alfred: Remove spaces before ',' Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 13/14] alfred: Remove empty lines after '{' Sven Eckelmann
2015-03-14 11:42 ` [B.A.T.M.A.N.] [PATCH 14/14] alfred: Use Linux-net style comment blocks Sven Eckelmann
2015-03-16 16:10 ` Simon Wunderlich [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6847523.RbkIcKIa8T@prime \
--to=sw@simonwunderlich.de \
--cc=b.a.t.m.a.n@lists.open-mesh.org \
--cc=sven@narfation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.