Thread (1 message) 1 message, 1 author, 2014-02-17

Re: [PATCH net-next v2 01/10] ieee802154: add basic support for RF212 to at86rf230 driver

From: David Miller <hidden>
Date: 2014-02-17 05:09:46

From: Phoebe Buckheister <redacted>
Date: Fri, 14 Feb 2014 23:49:23 +0100
-	if (page != 0 || channel < 11 || channel > 26) {
+	if (page < 0
+		|| page > 31
+		|| !(lp->dev->phy->channels_supported[page] & BIT(channel))) {
This is not indented properly.

The first character on the second and subsequent lines should start
exactly at the first column after the openning parenthesis of the top-level
if() statement.

You must use the appropriate number of TAB and SPACE characters necessary
to achieve this.

In general, if you are indenting using only TAB characters, you are doing
it wrong.

------------------------------------------------------------------------------
Android apps run on BlackBerry 10
Introducing the new BlackBerry 10.2.1 Runtime for Android apps.
Now with support for Jelly Bean, Bluetooth, Mapview and more.
Get your Android app in front of a whole new audience.  Start now.
http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help