dead code in wparser_def state machine? - Mailing list pgsql-hackers

From Tom Lane
Subject dead code in wparser_def state machine?
Date
Msg-id 24632.1193161239@sss.pgh.pa.us
Whole thread Raw
List pgsql-hackers
I find a number of places coded like this:

static TParserStateActionItem actionTPS_InHyphenLatWordFirst[] = {{p_isEOF, 0, A_POP, TPS_Null, 0, NULL},{p_islatin, 0,
A_NEXT,TPS_InHyphenLatWord, 0, NULL},{p_isnonlatin, 0, A_NEXT, TPS_InHyphenUWord, 0, NULL},{p_isdigit, 0, A_NEXT,
TPS_InHyphenValue,0, NULL},{p_isdigit, 0, A_NEXT, TPS_InHyphenUWord, 0, NULL},{NULL, 0, A_POP, TPS_Null, 0, NULL}
 
};

If I'm not completely misunderstanding this code, the second p_isdigit
line is dead code because it can never be reached.  Correct?
        regards, tom lane


pgsql-hackers by date:

Previous
From: Neil Conway
Date:
Subject: Re: PostgreSQL performance issues
Next
From: Gregory Stark
Date:
Subject: Re: Feature Freeze date for 8.4