John Levon wrote: > No !! Wrong wrong wrong. It's meaningless to return -1. Simple fact is: > routines must be able to deal with empty paragraphs. Any spots that > can't are simply bugs.
Is _much_ less meaningless IMO than to return 0 (which is indeed a valid return value for 1 char paragraphs). Think of it as a reverse_iterator that points at the end (i.e. rend()): it should be -1! > And yes, I specifically changed this to return 0 not -1, because *it was > causing bugs*. Where? > computeBidiTables must be fixed. AFAIC it's not only computeBidiTables, i was getting crashes also on the table code that I don't get any more (not investigated so much, though) Alfredo
