> > + errno = 0; > > + tmp = strtoul(argv, &endptr, 0); > > + if (errno || *endptr != '\0' || tmp > (uint32_t) -1UL) > > What about using UINT32_MAX instead of '(uint32_t) -1UL'? Good idea! Thanks. - Alistair