[GitHub] commons-lang issue #327: (doc) Remove invalid example of `lastIndexOf` from ...

2018-06-08 Thread printfcoder
Github user printfcoder commented on the issue: https://github.com/apache/commons-lang/pull/327 @fedochet Oh ! base on your explaining, it is ok. But the param `startPos ` means `start`, I’m totally confused, I think it should be renamed as `endPos`, aHa ?. ---

[GitHub] commons-lang issue #327: (doc) Remove invalid example of `lastIndexOf` from ...

2018-06-08 Thread fedochet
Github user fedochet commented on the issue: https://github.com/apache/commons-lang/pull/327 @printfcoder according to [docs](https://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/StringUtils.html), ``` (lastIndexOf) Returns the index within seq of

[GitHub] commons-lang issue #327: (doc) Remove invalid example of `lastIndexOf` from ...

2018-06-08 Thread printfcoder
Github user printfcoder commented on the issue: https://github.com/apache/commons-lang/pull/327 `StringUtils.lastIndexOf("aabaabaa", "b", 1) = -1` Could you please explain why the result is `1`,please? I see it shoud be 2, because that `to search starts at 1 position and b

[GitHub] commons-lang issue #327: (doc) Remove invalid example of `lastIndexOf` from ...

2018-04-29 Thread PascalSchumacher
Github user PascalSchumacher commented on the issue: https://github.com/apache/commons-lang/pull/327 Thanks! 👍 ---

[GitHub] commons-lang issue #327: (doc) Remove invalid example of `lastIndexOf` from ...

2018-04-28 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-lang/pull/327 [![Coverage Status](https://coveralls.io/builds/16746292/badge)](https://coveralls.io/builds/16746292) Coverage remained the same at 95.14% when pulling