Comment 22 for bug 836707

This is looking very nice. I like how parent id works and the new folder handling in particular, we've had a lot of issues related to broken/ ambiguous parents.

If "TODO: Function never used" is true, by all means, let's drop the function.

I wonder, can the "if (convert) sqlite3_exec" with multiple statements in theory fail so that there's a new table without importing the old one? Should it use explicit commit statements?

Apart from that, I'd like to get this in shortly after release. Following that Vincent's test work.