Message378843
I don't think we have any idea how the OP stumbled into this. Looks like it "just happened". The case you construted is quadratic-time, but not quite as bad: BBBBBaaaaaBBBBB BBBBBBBBBB Fails at once, because 'a' doesn't match the trailing needle 'B'. The Bloom filter is useless because the next haystack 'B' _is_ in the needle. So shift right 1: BBBBBaaaaaBBBBB xBBBBBBBBBB Last character matches, so goes on to compare 4 Bs and an aB mismatch. 6 in all. Bloom filter useless again, and another shift by 1: BBBBBaaaaaBBBBB xxBBBBBBBBBB Now there's 1 less compare, because only 3 Bs match. And so on. After the next shift, only 2 Bs match, and after the shift following that, only 1 B. Getting to: BBBBBaaaaaBBBBB xxxxxBBBBBBBBBB Now after matching the trailing Bs, aB mismatches at once. And we're done, because another shift by 1 moves the end of the needle beyond the end of the haystack (although I bet the Bloom filter reads up the trailing NUL byte from the haystack and actually makes a "giant" shift). You're right that two-way yawns at this ;-) 'B' * (K*2) is split into u = "" # empty string! v = 'B' * (K*2) so only the "match the right half" part of the searching algorithm comes into play. BBBBBaaaaaBBBBB BBBBBBBBBB first mismatches at the 6th character (1-based counting - at index 5), so it shifts right by 6: BBBBBaaaaaBBBBB xxxxxxBBBBBBBBBB And it's done, because the needle has moved beyond the end of the haystack. The brainpower that went into making this "look trivial" is quite impressive :-) | |
| Date | User | Action | Args | | 2020-10-18 01:04:29 | tim.peters | set | recipients: + tim.peters, gvanrossum, gregory.p.smith, vstinner, pmpp, serhiy.storchaka, josh.r, ammar2, corona10, Dennis Sweeney, Zeturic | | 2020-10-18 01:04:29 | tim.peters | set | messageid: <1602983069.42.0.896218504553.issue41972@roundup.psfhosted.org> | | 2020-10-18 01:04:29 | tim.peters | link | issue41972 messages | | 2020-10-18 01:04:29 | tim.peters | create | | |