[Date Prev][Date Next]
(ITS#4948) #if/#elif argument in .../include/ac/time.h
Full_Name: Lucio De Re
Version: 2.3.35 and earlier
OS: Plan 9
Submission from: (NULL) (126.96.36.199)
The arguments on lines 20 and 23 (#if and #elif, respectively) in
.../include/ac/time.h ought to be surrounded with "defined()". Whether the
existing format is legitimate or not is a matter for the C standards, but I
believe that the alternative would be both clearer and more portable.
The Plan 9 C pre-processor has a definite issue with it and it seems a
reasonable change to implement. There may be other places where a similar
change would be beneficial, I'll identify them as I encountered and will provide
a comprehensive list on demand.
Naturally, there are many other issues applicable specific to porting OpenLDAP
to Plan 9, but these seem trivial enough to warrant addressing in a broader