[Freeciv-Dev] Re: [POLL] Variables descriptions
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
On Thu, Feb 21, 2002 at 05:00:46PM -0600, Mike Kaufman wrote:
> For me, and most likely, for other people, when I'm reading down fresh
> code, this is _not_ hard for me to parse:
>
> /* bar bar bar bar bar ... barbar
> * barbarbar bar bar */
> char *foo;
>
> this however, _is_ hard:
>
> /* bar bar bar bar bar ... barbar
> * barbarbar bar bar bar bar bar */
> char *foo = bar * bar - total_power;
> /* bar bar bar bar bar ... barbar
> * barbarbar this is hard for me to parse */
> char *foobar = belligerence->no;
> /* bar bar bar bar bar ... barbar
> * barbarbar bar bar peew */
> char *foo_really = 0;
Yes. I would precede the comments with empty lines.
>
> because everything runs together.
> when varables _must_ have lengthy comments, I would prefer:
>
> char *foo; /* bar bar bar bar bar ... barbar
> * barbarbar bar bar bar bar bar */
> char *foobar; /* more blah blah
> * barbarbar bar bar */
> char *foo_really = 0; /* bar bar bar bar bar ... barbar
> * barbarbar bar bar peew */
I tend to use
char *foo;
/* bar bar bar bar bar ... barbar
barbarbar bar bar bar bar bar */
The indentation and sometimes newline make it obvious the comment
belongs to the previous statement. Comments pertaining to multiple
statements precede them, comments pertaining to a single statement
are treated like this. But this is just what I happen to do.
--
Reinier
- [Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#1264), (continued)
- [Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#1264), Petr Baudis, 2002/02/20
- [Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#1264), Raimar Falke, 2002/02/20
- [Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#1264), Gregory Berkolaiko, 2002/02/21
- [Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#1264), Petr Baudis, 2002/02/21
- [Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#1264), Petr Baudis, 2002/02/21
- Message not available
- [Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#1264), Petr Baudis, 2002/02/21
- [Freeciv-Dev] [POOL] Variables descriptions, Petr Baudis, 2002/02/21
- [Freeciv-Dev] Re: [POLL] Variables descriptions, Mike Kaufman, 2002/02/21
- [Freeciv-Dev] Re: [POLL] Variables descriptions,
Reinier Post <=
- [Freeciv-Dev] Re: [POLL] Variables descriptions, Raimar Falke, 2002/02/22
- [Freeciv-Dev] Re: [POLL] Variables descriptions, Raahul Kumar, 2002/02/22
- [Freeciv-Dev] Re: [POLL] Variables descriptions, Daniel L Speyer, 2002/02/22
- [Freeciv-Dev] [POLL] Variables descriptions: Votes on Style, Raahul Kumar, 2002/02/22
- [Freeciv-Dev] Re: [POLL] Variables descriptions, Raimar Falke, 2002/02/22
- [Freeciv-Dev] Freeciv Combat System, Raahul Kumar, 2002/02/21
- [Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#1264), Gregory Berkolaiko, 2002/02/21
- [Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#1264), Gregory Berkolaiko, 2002/02/20
- [Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#1264), Mike Kaufman, 2002/02/20
- [Freeciv-Dev] Re: [PATCH] aiunit.c ai_military_findvictim() cleanup (PR#1264), Petr Baudis, 2002/02/20
|
|