From 7078a5d6dbcd7dcd4ce9ffbe2872cecead217797 Mon Sep 17 00:00:00 2001 From: =?utf8?q?C=C3=A9dric=20Villemain?= Date: Thu, 15 Sep 2016 18:51:11 +0200 Subject: [PATCH] Fix varbit initialisation in some places it was set in a else {} part of the code, safe but trigger a warning from gcc. --- pgfincore.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/pgfincore.c b/pgfincore.c index 668b736..8ac5c63 100644 --- a/pgfincore.c +++ b/pgfincore.c @@ -675,8 +675,8 @@ pgfadvise_loader(PG_FUNCTION_ARGS) if (PG_ARGISNULL(5)) elog(ERROR, "pgfadvise_loader: databit argument shouldn't be NULL"); - else - databit = PG_GETARG_VARBIT_P(5); + + databit = PG_GETARG_VARBIT_P(5); /* initialize nulls array to build the tuple */ memset(nulls, 0, sizeof(nulls)); @@ -1113,8 +1113,8 @@ pgfincore_drawer(PG_FUNCTION_ARGS) if (PG_ARGISNULL(1)) elog(ERROR, "pgfincore_drawer: databit argument shouldn't be NULL"); - else - databit = PG_GETARG_VARBIT_P(0); + + databit = PG_GETARG_VARBIT_P(0); len = VARBITLEN(databit); result = (char *) palloc((len/FINCORE_BITS) + 1); -- 2.39.5