aboutsummaryrefslogtreecommitdiff
path: root/contrib/auto_explain/sql/alter_reset.sql
diff options
context:
space:
mode:
authorPeter Eisentraut <peter@eisentraut.org>2025-08-05 12:11:36 +0200
committerPeter Eisentraut <peter@eisentraut.org>2025-08-05 12:11:36 +0200
commit0f5ade7a367c16d823c75a81abb10e2ec98b4206 (patch)
treedc115967254082955b5bfd12935cf215c3cddc8e /contrib/auto_explain/sql/alter_reset.sql
parent2ad6e80de9a6300daffcc0987667e45012fbecde (diff)
downloadpostgresql-master.tar.gz
postgresql-master.zip
Fix varatt versus Datum type confusionsHEADmaster
Macros like VARDATA() and VARSIZE() should be thought of as taking values of type pointer to struct varlena or some other related struct. The way they are implemented, you can pass anything to it and it will cast it right. But this is in principle incorrect. To fix, add the required DatumGetPointer() calls. Or in a couple of cases, remove superfluous PointerGetDatum() calls. It is planned in a subsequent patch to change macros like VARDATA() and VARSIZE() to inline functions, which will enforce stricter typing. This is in preparation for that. Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us> Discussion: https://www.postgresql.org/message-id/flat/928ea48f-77c6-417b-897c-621ef16685a6%40eisentraut.org
Diffstat (limited to 'contrib/auto_explain/sql/alter_reset.sql')
0 files changed, 0 insertions, 0 deletions