On Fri, Oct 05, 2018 at 01:38:05PM +0200, Michael Banck wrote:
> It's too late for v11 though at this point I guess?
Unfortunately yes.
> I think it would be easy to also test the -r command-line option, as we
> already create a table.
Good idea. Let's add this test.
> That comment should read 'that checksums are enabled', right?
Indeed. Fixed.
> Otherwise, LGTM and I've tested it without finding any problems.
What do you think about the updated version attached?
--
Michael