On Tue, Feb 05, 2019 at 04:26:18PM +0900, Michael Paquier wrote: > On Tue, Feb 05, 2019 at 05:46:55PM +1300, David Rowley wrote: >> I did leave a couple untouched as there was quite a bit of escaping >> going on already. I didn't think switching between \Q and \E would >> have made those ones any more pleasing to the eye.
Indeed. I have stuck with your version here. > - qr/^ALTER TABLE dump_test.test_table ENABLE ROW LEVEL SECURITY;/m, > + qr/^\QALTER TABLE dump_test.test_table ENABLE ROW LEVEL SECURITY;/m, > I would have just appended the \E for consistency at the end of the > strings. Except that it looks fine. No need to send an updated > version, it seems that you have all the spots. I'll do an extra pass > on it tomorrow and see if I can commit it. And done after checking the whole set. -- Michael
signature.asc
Description: PGP signature