On 12/29/16, John Vogel <[email protected]> wrote: > On Thu, 29 Dec 2016 12:11:33 +0100 > Hiltjo Posthuma <[email protected]> wrote: > >> Nitpicks: >> - Check if digit in date_field(). >> - Style: don't declare variables in the inner scope, just use the >> function scope for struct tm date; and struct timespec ts; >> >> Kind regards, >> Hiltjo >> > > Thanks for your input.
Hmm, I think your fix for this made the patch a bit awkward. I suggest keeping the original structure, but changing date_field to eprintf when it hits a non-digit, so you don't have to deal with all that error checking in main.
