pgsql: Constify fields and parameters in spell.c
От | Heikki Linnakangas |
---|---|
Тема | pgsql: Constify fields and parameters in spell.c |
Дата | |
Msg-id | E1sbQX3-00315Q-9a@gemulon.postgresql.org обсуждение исходный текст |
Список | pgsql-committers |
Constify fields and parameters in spell.c I started by marking VoidString as const, and fixing the fallout by marking more fields and function arguments as const. It proliferated quite a lot, but all within spell.c and spell.h. A more narrow patch to get rid of the static VoidString buffer would be to replace it with '#define VoidString ""', as C99 allows assigning "" to a non-const pointer, even though you're not allowed to modify it. But it seems like good hygiene to mark all these as const. In the structs, the pointers can point to the constant VoidString, or a buffer allocated with palloc(), or with compact_palloc(), so you should not modify them. Reviewed-by: Andres Freund Discussion: https://www.postgresql.org/message-id/54c29fb0-edf2-48ea-9814-44e918bbd6e8@iki.fi Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/d5f139cb6814f0af2d2e1106899361e45c305630 Modified Files -------------- src/backend/tsearch/spell.c | 58 +++++++++++++++++++++------------------ src/include/tsearch/dicts/spell.h | 16 +++++------ 2 files changed, 39 insertions(+), 35 deletions(-)
В списке pgsql-committers по дате отправления: