On Thu, Mar 16, 2006 at 05:48:05PM +0100, Juergen Spitzmueller wrote: > Martin Vermeer wrote: > > Just for fun, have a look at stripLeadingSpaces in paragraph.C, > > and try to imagine how this should ever work with change tracking on... > > (remember that erase(0) only does a logical erase in this case. > > No whitespace is actually removed) > > pandora's box... > > > The method is called from DEPM. I am tempted to shortcut the whole > > of DEPM if CT is on, but perhaps I should only shortcut > > stripLeadingSpaces. > > > > Please advise ;-) > > I think the attached patch is enough and better than switching DESM off. > Works > for me at least. > > What do you think?
Yes this is the minimal patch. Put it in. (Aren't there too many parentheses?) BTW to answer Lars's question, it would mean that CT would "leak" blanks and paragraph breaks. I think. They would disappear without CT keeping track of them; but only these "superfluous" ones. Presumably the stuff that matters (the words and paragraphs themselves) would still be conserved. - Martin
pgpaxzzpK7S7a.pgp
Description: PGP signature
