Multiple returns are evil
Return text length if no break points was found

The code was supposedly doing that, but probably misunderstood the
Graphite API.
Remove a wrong XXX note
Reduce variable scope