Date: Fri, 28 Sep 2001 10:40:53 +0300 From: Ruslan Ermilov <ru@FreeBSD.org> To: Julian Elischer <julian@elischer.org> Cc: Garrett Wollman <wollman@khavrinen.lcs.mit.edu>, obrien@FreeBSD.org, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/sys proc.h Message-ID: <20010928104053.C30062@sunbay.com> In-Reply-To: <Pine.BSF.4.21.0109271716250.65838-100000@InterJet.elischer.org>; from julian@elischer.org on Thu, Sep 27, 2001 at 05:27:14PM -0700 References: <200109272214.f8RME1v06093@khavrinen.lcs.mit.edu> <Pine.BSF.4.21.0109271716250.65838-100000@InterJet.elischer.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Sep 27, 2001 at 05:27:14PM -0700, Julian Elischer wrote:
>
> Personally I'd like to see something that says like:
>
> "small variations to this rule, for the purpose of aligning
> related fields and increasign readability are accepteble."
>
> and maybe an example
>
> struct foo {
> int foo_a; /* comment */
> struct freddy foo_b; /* Comment aligned as above. */
> union bigwords *foo_c; /* All comments aligned */
> unsigned char foo_chars; /* to allow the eye to scan them */
> /* With overflow also aligned */
> /* the same as above. */
> struct bigstructures foo_vals[NUMSTRUCTS]; /* sometimes you just */
> /* have to bend the rules */
> };
>
I would even allow the following, similar to prototypes:
struct foo {
int foo_a; /* comment */
struct freddy foo_b; /* Comment aligned as above. */
union bigwords *foo_c; /* All comments aligned */
unsigned char foo_chars; /* to allow the eye to scan them */
/* With overflow also aligned */
/* the same as above. */
struct bigstructures foo_vals[NUMSTRUCTS]; /* sometimes you just */
/* have to bend the rules */
};
--
Ruslan Ermilov Oracle Developer/DBA,
ru@sunbay.com Sunbay Software AG,
ru@FreeBSD.org FreeBSD committer,
+380.652.512.251 Simferopol, Ukraine
http://www.FreeBSD.org The Power To Serve
http://www.oracle.com Enabling The Information Age
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010928104053.C30062>