lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <473C7C62.9030100@nortel.com>
Date:	Thu, 15 Nov 2007 11:05:38 -0600
From:	"Chris Friesen" <cfriesen@...tel.com>
To:	Michael Gerdau <mgerdau@...cali.de>
CC:	Philippe Elie <phil.el@...adoo.fr>,
	Russell Leighton <russ@...gant-software.com>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: OT: Does Linux have any "Perfect Code"

Michael Gerdau wrote:

> That having said:
> I really do like such obvious (as in: for those knowing the stuff anyway)
> comments when looking at code and probably concepts I'm not familiar with.
> 
> IMO there is no need to belittle this type of comment. IMO any casual
> reader not familiar with the whole set of functions possibly involved
> will be thankful for this type of comment - I certainly am.

Obvious comments explaining _what_ the code is doing are annoying. 
They're fluff.  You can see what the code is doing.

The critical comments are the ones that explain _why_ and _how_.  The 
comments should give the higher level view as to the rationale for that 
code.

Of course if you've got some highly-optimized bit of black magic there 
had better be some good comments explaining what's going on...but there 
should also be some comments explaining why we need the magic in the 
first place.

Chris
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ