Makes a lot of sense. I get frustrated with how explicit I have to be with Rust sometimes (currently I'm writing something with a lot of reusable, immutable structures and am put off by how much Rc.clone() that requires) but making it known to the programmer that every indexing of a String is effectively an iteration is a good call.
In fact it looks like even standard indexes have been deprecated, in favor of Iterators over the string:
https://doc.rust-lang.org/stable/std/primitive.str.html#meth...
Iteration is better IMO.