cvs commit: src/sys/boot/arc/include arcfuncs.h src/sys/boot/i386/boot2 boot2.c src/sys/dev/aic7xxx/aicasm aicasm.c iir.h src/sys/dev/isp ispvar.h src/sys/dev/mcd mcdreg.h ...

[Alexander Leidinger]

On Fri, 12 Mar 2004 23:46:30 +0100
Maxime Henrion <mux at freebsd.org> wrote:

> It would be even better IMHO if you had used __GNUC_PREREQ__ in other
> files, to simplify all those tests.

It would be better to change this to feature tests and set the
appropriate defines at the beginning depending on the compiler used.
This way you only have to check the compiler once. This was suggested to
me by Matthew Dillon a while ago, but I think such a change should be
separate from the icc change, as it makes it more obvious what happens
in each change (so it's more easy to review).

Bye,
Alexander.

-- 
           I will be available to get hired in April 2004.

http://www.Leidinger.net                       Alexander @ Leidinger.net
  GPG fingerprint = C518 BC70 E67F 143F BE91  3365 79E2 9C60 B006 3FE7