[hatari-devel] 68030 Debug mode and new CPU
Eero Tamminen
oak at helsinkinet.fi
Mon Feb 21 22:20:16 CET 2011
Hi,
On maanantai 21 helmikuu 2011, Eero Tamminen wrote:
> On maanantai 21 helmikuu 2011, Laurent Sallafranque wrote:
> > But as likely is also used in cpummu.h, shouldn't I include "main.h"
> > only in cpummu.h ?
>
> There are some Hatari include files which include other includes, but
> Thomas thinks it's better to do the inclusions in C-files, otherwise you
> could get into funny issues despite the header ifdef multiple-inclusion
> protection.
>
> Thomas, what were the issues into which you've run with that?
Ah, sorry, it was of course documented in doc/coding.txt :-)
I added there also comment about magic values.
(And removed the comment about where the braces belong as
the document already says that coding convention of the existing
source file should be followed. I just added a comment that new
files should use one of the existing conventions...)
- Eero
More information about the hatari-devel
mailing list