This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
Any objections? Did I miss a detail?
You're missing documentation. And a reason why it's appropriate to put this in libiberty, instead of in gcc. Is there some standard that defines this function? Will anyone else use it besides gcc?
Also, the file doesn't quite follow FSF coding conventions, and doesn't include libiberty.h or config.h nor use any of the HAVE_* macros.
You should also add code to configure.ac to maybe set the HAVE_DECL_* macro for libiberty's own use.
Attachment:
libiberty-strverscmp-1.patch
Description: Binary data
Attachment:
smime.p7s
Description: S/MIME cryptographic signature
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |