On Wed, 18 Jul 2018 19:07:21 +0200, Ingo Schwarze wrote: > I think it's more elegant to just break; when getline() returns -1, > you already have the free() at the end: > > if ((len = getline(&line, &linesize, F->fp)) == -1) > break; > > But you get to choose the colour of the bikeshed, OK either way.
Good idea. I made that change and committed. - todd