lkml.org 
[lkml]   [2002]   [Jan]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] Combined APM patch
> Just browsing the diff between my patch and Stephen's, I have
> a couple of questions.

> < static int suspends_pending; /* = 0 */
> ---
> > static int suspends_pending;

> Is it not good practice to note when the code _assumes_ zero-
> initialization? I have seen comments like these elsewhere in
> the kernel sources.

Comments should _not_ echo code. One should assumes that the reader
knows the language. Instead, comments should explain _what the purpose_
of that line of code or section of code is if it is not obvious to those
that the comments are written to.

Thus, the following should be considered correct:

static int suspends_pending;

Both of the following lines have superfluous code or commants and should
be avoided:

static int suspends_pending; /* = 0 */
static int suspends_pending = 0;
Likewise, parentheses where none are needed just get in the way of useful
stuff and slows down understanding the program. Some parentheses in
complex expressions or when the precedence of operator evaluation is
confusing, such as bitwise ops can be helpful.

Btw, I have been writing system-level c code for 26 years and learned by
reading and modifying Unix system code at Berkeley.

Best regards,

Bob Toxen, CTO
Fly-By-Day Consulting, Inc. "Experts in Linux & network security"
bob@cavu.com
http://www.cavu.com [Linux/Unix & Network Security Consulting]
http://www.realworldlinuxsecurity.com [My 5* book: "Real World Linux Security"]
http://www.cavu.com/sunset.html [Sunset Computer]
Quality Linux, UNIX and network security and software consulting since 1990.

GPG Public key available at http://www.cavu.com/pubkey.txt (book@cavu.com)
and at http://pgp5.ai.mit.edu/pks-commands.html#extract
and on the CD-ROM that comes sealed and attached to Real World Linux Security
pub 1024D/E3A1C540 2000-06-21 Bob Toxen <book@cavu.com>
Key fingerprint = 30BA AA0A 31DD B68B 47C9 601E 96D3 533D E3A1 C540
sub 2048g/03FFCCB9 2000-06-21
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:15    [from the cache]
©2003-2011 Jasper Spaans