Issue #1319: findnext inconsistency, option 1

[jrfnl]

Fixes #1319 with the smallest possible BC break.

[gsherwood]

I'm going to close the two PRs that provide options for #1319 because I only want to make a change like this in 3.x, and I'm not entirely sure which change yet. Discussion can continue over on the main issue.

[jrfnl]

@gsherwood Would you like me to pull these PRs against the v3 branch ?

[gsherwood]

Would you like me to pull these PRs against the v3 branch ?

No thanks. I don't see any reason for a PR until there is a decision about what change would be made. And I'm not going to make any changes before the 3.0 release anyway.

[jrfnl]

So what would you need to help you decide ? And what can I do to help ?

As this - as you correctly pointed out before - would be a BC break, this feels like something which should be in the 3.0.0 version and not in a later 3.x.

[gsherwood]

this feels like something which should be in the 3.0.0 version and not in a later 3.x

It's just not going to be in 3.0. I'm not making a change that would require even more upgrades and another RC. I need to get it released, so this change is basically on hold for the time being.

[jrfnl]

On hold until version 4 in that case ?

[gsherwood]

On hold until version 4 in that case ?

Maybe