On: Git security vulnerabilities announced

[jcparkyn]

Probably because you'd want to access the value in either case, depending on your application.

[howinteresting → jcparkyn]

Could be a `Result<T, T>` which would seem to express the intent better.

[3836293648 → howinteresting]

Sometimes, sure. But an overflow doesn't have to be an error, it can be what you're after and you just want to know when it happens.

[howinteresting → 3836293648]

Binary search is similar and the return type there is already Result<usize, usize>: https://doc.rust-lang.org/std/vec/struct.Vec.html#method.bin...