Le Tuesday 30 Sep 2014 à 12:23:12 (+0200), Thibaut Robert a écrit :
> Remove unnecessary else after return.
> Follow coding style for curly braces around single statements.
> Fix a typo in error message.
>
> Signed-off-by: Thibaut Robert
> ---
> drivers/staging/gdm72xx/netlink_k.c | 24 +
Remove unnecessary else after return.
Follow coding style for curly braces around single statements.
Fix a typo in error message.
Signed-off-by: Thibaut Robert
---
drivers/staging/gdm72xx/netlink_k.c | 24 +++-
1 file changed, 11 insertions(+), 13 deletions(-)
diff --git a/d
Le Monday 29 Sep 2014 à 11:31:24 (-0700), Joe Perches a écrit :
> On Mon, 2014-09-29 at 20:15 +0200, Thibaut Robert wrote:
> > This patch remove a checkstyle warning in netlink_k.c.
> > Note: This is my homework of the day.
>
> The "Note: " line isn't useful for a commit message.
> Neither is the
On Mon, 2014-09-29 at 20:15 +0200, Thibaut Robert wrote:
> This patch remove a checkstyle warning in netlink_k.c.
> Note: This is my homework of the day.
The "Note: " line isn't useful for a commit message.
Neither is the checkpatch warning bit.
Maybe the commit message could be something like:
"
This patch remove a checkstyle warning in netlink_k.c.
Note: This is my homework of the day.
Signed-off-by: Thibaut Robert
---
drivers/staging/gdm72xx/netlink_k.c | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/gdm72xx/netlink_k.c
b/drivers/stag