This variable, end_shift, only ever holds the value 0. It is used in
only one place, where its value is assigned to another variable.
The history is interesting:
start_shift and end_shift were added in
c277df42229, which added sub-
string optimisations.
146174a91a192 rearranged stuff a bit, causing
these variables to be unused.
155aba94f6 commented out the unused
start_shift to suppress a compiler warning.
1df70142a966e remove=
the start_shift comment, leaving the comment describing end_shift
nonsensical.
char *startpos;
I32 minlen; /* must match at least this many chars */
I32 dontbother = 0; /* how many characters not to try at end */
- I32 end_shift = 0; /* Same for the end. */ /* CC */
const bool utf8_target = cBOOL(DO_UTF8(sv));
I32 multiline;
RXi_GET_DECL(prog,progi);
/* XXXX check_substr already used to find "s", can optimize if
check_substr==must. */
- dontbother = end_shift;
+ dontbother = 0;
strend = HOPc(strend, -dontbother);
while ( (s <= last) &&
(s = fbm_instr((unsigned char*)HOP3(s, back_min, (back_min<0 ? strbeg : strend)),