Skip to content

Commit ff1b5e0

Browse files
tonycozmauke
authored andcommitted
regatom: e is initialized and only used to update itself
e is initialized, but the only uses are to update itself. It does appear to have been used when originally added, but it no longer is. Possibly there's a test missing. Picked up by clang static analyzer.
1 parent 54a72ba commit ff1b5e0

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

regcomp.c

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6025,7 +6025,6 @@ S_regatom(pTHX_ RExC_state_t *pRExC_state, I32 *flagp, U32 depth)
60256025
I32 num;
60266026
char * endbrace = NULL;
60276027
char * s = RExC_parse;
6028-
char * e = RExC_end;
60296028

60306029
if (*s == 'g') {
60316030
bool isrel = FALSE;
@@ -6062,12 +6061,6 @@ S_regatom(pTHX_ RExC_state_t *pRExC_state, I32 *flagp, U32 depth)
60626061
while (isBLANK(*s)) {
60636062
s++;
60646063
}
6065-
6066-
/* Ignore trailing blanks */
6067-
e = endbrace;
6068-
while (s < e && isBLANK(*(e - 1))) {
6069-
e--;
6070-
}
60716064
}
60726065

60736066
/* Here, have isolated the meat of the construct from any

0 commit comments

Comments
 (0)