Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk> writes:
> The gist one is just that the old code was abusing DatumGetHStoreP by
> applying it to something that wasn't an hstore. This didn't matter
> before the format upgrade code was put in, and it didn't show up in
> tests because you need to index a very large number of hstores before
> any problem shows up.
Actually, since ghstore is not marked toastable (and hardly needs to
be, since its max length is 24 bytes), that function seems completely
useless. Why isn't it just
PG_RETURN_POINTER(PG_GETARG_POINTER(0));
(compare gbt_decompress in btree_gist, for instance).
regards, tom lane