"Marko Kreen" <markokr at gmail.com> writes:
> Final API:

> int8          txid_current()
> txid_snapshot txid_current_snapshot()
> int8          txid_snapshot_xmin(snap)
> int8          txid_snapshot_xmax(snap)
> setof int8    txid_snapshot_xip(snap)
> bool          txid_visible_in_snapshot(int8, snap)
> bool          txid_not_visible_in_snapshot(int8, snap)

The last seems pretty thoroughly redundant?

> - removed use of "unsigned long long" in txid_snapshot_out,
>   instead I try to guess the correct fromat string for uint64
>   from HAVE_LONG_INT_64

What's to guess?  Use UINT64_FORMAT

			regards, tom lane