Thanks for looking. On 2018/11/07 12:32, Alvaro Herrera wrote: > ISTM patch 0004 is impossible to review just because of size -- I > suppose the bulk of it is just code that moves from one file to another, > but that there are also code changes in it. Maybe it would be better to > split it so that one patch moves the code to the new files without > changing it, then another patch does the actual functional changes? If > git shows the first half as a rename, it's easier to be confident about > it.
Okay, will post a new version structured that way shortly. Thanks, Amit