Although I know I am guilty of transgressing on this in the first change (with the ``i`` identifier), it would be preferable to use full-length names for identifiers. ``i`` should be ``index``, ``len`` should be ``length``.