wrapping method



	Hi all,
I'm still on wrapping problems and I've read the code, it seems to me 
that we make it the wrong way.
So to see if I understand well, here is what I think we are doing :
- we load the body and immediately wrap it with libbalsa_wrap_string; 
this function just wrap lines to fit the width.
- next we parse line by line and with regular expression for urls and 
quotation to colorize the text.
I think it's wrong because libbalsa_wrap_string doesn't care about 
quotation, so if you have the following quotation :
>>>> this is a quotation exceeding the line width
it's wrapped that way :
>>>> this is a quotation
exceeding the line width
so when we colorize we've lost the second part, like if the second was 
not part of the quotation.
So I think we have to be smarter when wrapping and we must integrate 
quotation level to correctly duplicate quotation marks when we wrap the 
text.
The problem is that we can no more wrap the string "in place" (ie 
without duplicating it) because we should be able to add a bunch of 
chars to the string.
Is it worth the pain ?
Thanks
Bye
Manu






[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]