> It is the Plan 9 Way (TM) to avoid > nested inclusion of header files,
$ arch/dat.h includes port/portdat.h in kernel. Exempted too?
Álvaro Jurado Cuevas
http://colmenar.biz.tm
> Which version?
>
> "The id_t and pid_t types shall be defined as described in
> <sys/types.h>." in issue 6
>
> "The <sys/wait.h> header shall define the id_t and pid_t types as
> described in <sys/types.h>." in issue 7
>
> in the sys/wait.h part of the headers section of base definitions
I haven't looked at cinap's work, but...
It is the Plan 9 Way (TM) to avoid nested inclusion of header files,
although I guess the APE may be exempted. I also appreciate that
adding conditional definitions of id_t and pid_t in <sys/wait.h> that
match those in <sys/types.h> could lead to eventual inconsistencies,
but I would still prefer to follow the Plan 9 guidelines.
But without a more formal code review structure and the apparent
absence of guidance from Bell Labs, I suppose I'm just farting in the
wind.
Lucio.