On Sat, May 13, 2017 at 9:06 PM, Jeff King <peff@xxxxxxxx> wrote: > On Sat, May 13, 2017 at 09:01:16PM -0700, Stefan Beller wrote: > >> + for (i = 0; i < o->line_buffer_nr; i++); >> + free((void*)o->line_buffer[i].line); > > I haven't looked at the patches yet, but this ";" on the for line is > almost certainly a typo (gcc catches it due to the misleading > indentation of the next line). Grr :/ I have spent hours trying to figure out why this does not work, questioning the design, my mental model of how pointers work and programming in general. /me should get gcc 6 and set -Wmisleading-indentation Thanks, Stefan > > -Peff