[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
(ITS#4948) #if/#elif argument in .../include/ac/time.h
Full_Name: Lucio De Re
Version: 2.3.35 and earlier
OS: Plan 9
URL: ftp://ftp.openldap.org/incoming/
Submission from: (NULL) (196.25.255.250)
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
context.