> 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.


Reply via email to