Adam, Thanks for your patient explanation. I've attached a new diff, having followed your instructions. :) If you'd like, I can send you a PDB that implements line justification, for you to play with. Let me know.
- Jamis On Mon, 2002-04-29 at 11:37, Adam McDaniel wrote: > On Mon, Apr 29, 2002 at 11:21:10AM -0600, Jamis Buck wrote: > > Also, I found a bug in my word-wrap implementation: when the end of a > > series of adjacent dashes is encountered, "lastSpaceIsVisible" needs to > > be set to false (otherwise, "spaceCount" may be decremented > > incorrectly). > > You know whats funny, I dont have a PDB handy that implements line > justification :) .. not too easy for me to test. > > > I've attached a diff...hope I did it right. It includes the change to > > paragraph.c for the word wrap stuff, too. > > Looks better, except it looks as though you diffed the hires branch off > of the main trunk with your changes :) (hence removing hires.h from the > top of paragraph.c) Just make sure that when you create the diff, you're > using two exact copies of the code, except one has your changes. > > What I do in the CVS is always have a plucker dir, then a plucker-orig > dir. I make my changes in the plucker dir, run 'cvs update' in both to > make sure everythings the latest.. then run > > diff -Naur -x CVS plucker-orig plucker > diff > > Be sure that before you run this command, run 'make distclean' and > delete the configure script, just to make sure everything is 100% fresh. > > Finally, always review the diff manually, just to make sure nothing > stray popped in, and it's expressing only EXACTLY what you want/changed. > > -- > Adam McDaniel > Array Networks > Calgary, AB, Canada -- Jamis Buck [EMAIL PROTECTED] http://hippa-potta.jamisandtarasine.net . "I'd horsewhip you if I had a horse." -- Groucho Marx
