flatland/useful — comment on commit 4607940f438d9f87bc73355f487405c5d7fe435e
File: src/useful/string.clj line 40
By amalloy:

I just realized this test is sorta unnecessary. If `idx` is -1, then `(inc idx)` will be 0, and the subs call will return the whole string. Of course, it means `subs` does some work to discover what we already know: it can just return the original string. But (a) I'm not sure if leaving this out is faster, and (b) it might be a bit trickier to read. Thoughts, anyone? @Raynes, say?