[PATCH v4 02/18] RISC-V: Cleanup some of the sysdep.h code

Maciej W. Rozycki macro@wdc.com
Fri Aug 21 17:13:10 GMT 2020


On Tue, 18 Aug 2020, Carlos O'Donell wrote:

> >  Please comment #else:
> > 
> > #else /* !__ASSEMBLER__ */
> > 
> > especially as the conditional part above is very long.  Otherwise OK.
> > 
> >  This is not actually mentioned in our coding style document, unlike 
> > #endif, giving an impression we don't want #else statements commented: 
> > <https://sourceware.org/glibc/wiki/Style_and_Conventions#Commenting_.23endif>.  
> > This is however covered by the generic GNU Coding Standards document:
> > <https://www.gnu.org/prep/standards/standards.html#Comments>.
> > 
> >  Carlos: Do we want our wiki updated here?  ISTM we should.  WDYT?
> 
> Yes!
> 
> The GNU Coding Standard explains that we should comment #else, but doesn't
> say what kind of comment should be used, and that content is up to the
> project.
> 
> The section should cover how glibc is unique and what we expect from the
> #else comment.
> 
> Your suggestion looks good to me.

 I have updated the wiki then, thank you for your opinion.

 Also we don't explicitly mention #elif, but I think it can be implied 
that the convention followed for conditional parts beyond that directive 
is the same as with plain #if.

  Maciej


More information about the Libc-alpha mailing list