Dean, * Dean Rasheed (dean.a.rash...@gmail.com) wrote: > On 27 January 2015 at 22:45, Stephen Frost <sfr...@snowman.net> wrote: > > Here's the latest set with a few additional improvements (mostly > > comments but also a couple missed #include's and eliminating unnecessary > > whitespace changes). Unless there are issues with my testing tonight or > > concerns raised, I'll push these tomorrow. > > I spotted a couple of minor things reading the patches: > > - There's a typo in the comment for the GetModifiedColumns() macros > ("...stick in into...").
Fixed. > - The new regression test is not tidying up properly after itself, > because it's trying to drop the table t1 as the wrong user. Urgh. Not sure how I managed to miss that; guess I was just too focused on what I was testing. :) > Other than that, this looks reasonable to me, and I think that for > most common situations it won't reduce the detail in errors. Thanks! I'll be pushing this soon (finally!). Thanks again, Stephen
signature.asc
Description: Digital signature