[CALL FOR TESTERS] New system call: abort2()

Wojciech A. Koszek dunstan at freebsd.czest.pl
Thu Dec 15 22:19:08 PST 2005

On Fri, Dec 16, 2005 at 12:17:14AM +0100, Václav Haisman wrote:
> Wojciech A. Koszek wrote:
> [...]
> > Comments are welcome!
> As for the patch, the use of do {} while(0) instead of goto looks odd to me.

This can be changed easily in final version of the patch if needed.

> I would like to comment on FreeBSD style(9) a bit. Why does not mention
> or even encourage C99 style // comments? They are nice when one wants to
> comment out bigger chunks of code with /**/ comment.
> On the similar note, the ability to move declarations closer to the
> point of use in code is IMO nice feature, too. The style(9) doesn't
> mention this either.

This creates unnecessary problems: large blocks can duplicate name of
the variable so that they overlap, which has happened in the past.
Additionally, some files use this kind of declaration, and it makes
source hard to read.

Also please note that current comments probably needs changing. I'm
looking forward to hearing new comments in that field.

* Wojciech A. Koszek && dunstan at FreeBSD.czest.pl

More information about the freebsd-hackers mailing list