On 09-10-2013, at 5:02 PM, Nicolas Cellier <nicolas.cellier.aka.nice at gmail.com> wrote:

> If the encoding changes, then we never reach (self isBreakableAt: lastIndex in: sourceString in: charset) because we ^#endOfRun.
> Else, the encoding does not change, so we can proceed with charsetAt: startEncoding.
> So, as I see it, the charsetAt: can safely go outside the loop, no?

Sorry - I misread your diffs! I looked too quickly and though you were removing the check for 
encoding ~= startEncoding ifTrue: [lastIndex := lastIndex - 1. ^ stopConditions endOfRun].


tim
--
tim Rowledge; tim at rowledge.org; http://www.rowledge.org/tim
Useful random insult:- Has an inferiority complex, but not a very good one.