On Sat, Jan 7, 2017 at 1:32 PM, Michael Paquier <michael.paqu...@gmail.com> wrote:
> On Sat, Jan 7, 2017 at 12:04 AM, Magnus Hagander <mag...@hagander.net> > wrote: > > On Wed, Jan 4, 2017 at 10:43 AM, Magnus Hagander <mag...@hagander.net> > > wrote: > >> Meh, just as I was going to respond "committed" I noticed this second > >> round of review comments. Apologies, pushed without that. > >> > >> I agree on the change with includewal/streamwal. That was already the > case > >> in the existing code of course, but that doesn't mean it couldn't be > made > >> better. I'll take a look at doing that as a separate patch. > >> > > > > Here's a patch that does this. Does this match what you were thinking? > > Yes, that's it. I have looked at the patch in details and that looks > correct to me. > OK. Pushed. I agree it made it more readable, if nothing else. -- Magnus Hagander Me: http://www.hagander.net/ Work: http://www.redpill-linpro.com/
