Changes between Version 37 and Version 38 of Ticket #38582, comment 6


Ignore:
Timestamp:
Aug 7, 2013, 2:37:02 PM (11 years ago)
Author:
cooljeanius (Eric Gallager)
Comment:

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #38582, comment 6

    v37 v38  
    2929|| `AC_CHECK_HEADERS([sys/time.h])` || `src/pextlib1.0/sha2.c:38` || `#include <sys/time.h>` || an included header || something useful ||
    3030|| `AC_CHECK_HEADER_STDBOOL` || `src/machista1.0/libmachista.c:247` || `bool universal = false;` || In a mach-o parsing function || `/* Parse the Mach-O header */` I do not get why `autoscan` says this macro is needed for this line instead of line 33 which has the actual `#include <stdbool.h>` on it, but whatever... ||
    31 || `AC_C_INLINE` || `src/darwintracelib1.0/darwintrace.c:117` || `static inline int __darwintrace_strbeginswith(const char *str, const char *prefix);` || looks like a global variable? || not sure why this uses `inline` (it looks like regular C to me), but if we're going to be using `inline` stuff, it'd probably be a good idea to have `autoconf` check how the compiler handles it... ||
     31|| `AC_C_INLINE` || `src/darwintracelib1.0/darwintrace.c:117` || `static inline int __darwintrace_strbeginswith(const char *str, const char *prefix);` || looks like a function prototype? || not sure why this uses `inline` (it looks like regular C to me), but if we're going to be using `inline` stuff, it'd probably be a good idea to have `autoconf` check how the compiler handles it... (also throw in the `AC_C_PROTOTYPES` macro in there while you're at it) ||
    3232|| `AC_C_RESTRICT` || `src/darwintracelib1.0/darwintrace.c:774` || `static void frecv(void *restrict buf, size_t size) {` || function declaration || `/* Helper function to recieve a number of bytes from the tracelib communication socket and deal with any errors that might occur. */` Using `restrict` can be good for performance, have `autoconf` check how the compiler handles it to make sure ||
    3333|| `AC_FUNC_CHOWN` || `src/pextlib1.0/xinstall.c:1077` || `if ((gid != (gid_t)-1 || uid != (uid_t)-1) && chown(path, uid, gid))` || in a condition || Helps build the directory "`heirarchy` [sic]". Plus checking how `chown`ing works is probably a good idea anyways. ||