Changes between Version 35 and Version 36 of Ticket #38582, comment 6
- Timestamp:
- Aug 7, 2013, 2:25:50 PM (11 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #38582, comment 6
v35 v36 19 19 || `AC_CHECK_FUNCS([strstr])` || `src/machista1.0/machista_wrap.c:1383` || `where = strstr(where, meth->name);` || idk || I don't get this file; just check the `strstr` function anyways... || 20 20 || `AC_CHECK_FUNCS([strtol])` || `src/pextlib1.0/Pextlib.c:206` || `pwent = getpwuid(strtol(user, 0, 0));` || in the `ExistsuserCmd` function || There is no commentary around this, but it would probably be a good idea to check for the `strtol` function anyways... || 21 || `AC_CHECK_FUNCS([strtoul])` || `src/machista1.0/machista_wrap.c:1820` || `v = strtoul(nptr, &endptr,0);` || something swiggy || `/* If v is negative, then this could be a negative number, or an unsigned value which doesn't fit in a signed long, so try to get it as a string so we can distinguish these cases. */` Just check for `strtoul` for this ||21 || `AC_CHECK_FUNCS([strtoul])` || `src/machista1.0/machista_wrap.c:1820` || `v = strtoul(nptr, &endptr,0);` || something swiggy || `/* If v is negative, then this could be a negative number, or an unsigned value which doesn't fit in a signed long, so try to get it as a string so we can distinguish these cases. */` Just check for `strtoul` for this anyways (and just because, as well) || 22 22 || `AC_CHECK_FUNCS([utime])` || `src/pextlib1.0/curl.c:438` || `utime(theFilePath, ×); /* set the time we got */` || For checking `curl` times || Checking for `utime` would be good for `curl` || 23 23 || `AC_CHECK_HEADERS([mach/mach.h])` || `src/programs/daemondo/main.c:66` || `#include <mach/mach.h>` || an included header || `daemondo` does not say what it needs this header for, but I'm assuming it is something important... ||