postgres.git / summary / log / commit / refs
commit 1f7452fa598ee5f1ed32f1fcce101c63c6f1933f
Author: Tom Lane <tgl@sss.pgh.pa.us>
Date: Mon May 13 17:52:17 2024 +0000
First released in: 17.0
Remove COMMAND_TAG_NEXTTAG from enum CommandTag.
COMMAND_TAG_NEXTTAG isn't really a valid command tag. Declaring it
as if it were one prompts complaints from Coverity and perhaps other
static analyzers. Our only use of it is in an entirely-unnecessary
array sizing declaration, so let's just drop it.
Ranier Vilela
Discussion: https://postgr.es/m/CAEudQAoY0xrKuTAX7W10zsjjUpKBPFRtdCyScb3Z0FB2v6HNmQ@mail.gmail.com
src/backend/tcop/cmdtag.c | 2 +-
src/include/tcop/cmdtag.h | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/backend/tcop/cmdtag.c b/src/backend/tcop/cmdtag.c
index 68689b3e0d9..0870064fdd8 100644
--- a/src/backend/tcop/cmdtag.c
+++ b/src/backend/tcop/cmdtag.c
@@ -30,7 +30,7 @@ typedef struct CommandTagBehavior
#define PG_CMDTAG(tag, name, evtrgok, rwrok, rowcnt) \
{ name, (uint8) (sizeof(name) - 1), evtrgok, rwrok, rowcnt },
-static const CommandTagBehavior tag_behavior[COMMAND_TAG_NEXTTAG] = {
+static const CommandTagBehavior tag_behavior[] = {
#include "tcop/cmdtaglist.h"
};
diff --git a/src/include/tcop/cmdtag.h b/src/include/tcop/cmdtag.h
index da210e54fa8..23c99d7eca6 100644
--- a/src/include/tcop/cmdtag.h
+++ b/src/include/tcop/cmdtag.h
@@ -22,7 +22,6 @@
typedef enum CommandTag
{
#include "tcop/cmdtaglist.h"
- COMMAND_TAG_NEXTTAG
} CommandTag;
#undef PG_CMDTAG
[parent: 11c1984cccad]