> Do I replace vim_free with free in all my future patches? Why is vim_free bad?

Yes, `vim_free` has just been removed, see #627. So if you rebase, you shouldn't even be able to use `vim_free` anymore. Also, in this particular instance there should just be a single `xrealloc()` call :).