Don't declare variables as "unsigned char *" that are passed to functions that take "char *" arguments. Where such chars are assigned to int or passed to ctype functions, explicitly cast them to unsigned char. For OpenBSD' clang, -Wpointer-sign has been disabled by default, but when the parse.y code was built elsewhere, the compiler would complain. (originally by naddy@ for various parse.y in the tree)

amongst other things, walk now correctly uses fds for each step, so we're not limited by PATH_MAX for the whole walk, but only for the single path component.