Hello, Antonin! Some comments for 0003:
> /* allocate in transaction context */ It may be any context now, because it is a function now. > result = CopySnapshot(snapshot); > /* Restore the original values so the source is intact. */ > snapshot->xip = oldxip; > snapshot->xcnt = oldxcnt; I think it is worth to call pfree(newxip) here. > "This difference does has no impact" should be "This difference has no impact"? Best regards, Mikhail.
