[ wxwindows-Patches-1882985 ] Fix formatting of multi-line strings

[SourceForge.net]

Patches item #1882985, was opened at 2008-01-30 21:59
Message generated for change (Comment added) made by vadz
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=309863&aid=1882985&group_id=9863

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Grid
Group: bug fix
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: Jon Trauntvein (j_trauntvein)
Assigned to: Nobody/Anonymous (nobody)
Summary: Fix formatting of multi-line strings

Initial Comment:
This patch fixes a formatting bug for a multi-line string that is composed with DOS line endings (\r\n).  If the following string was supplied:

"this is\r\n"
"a multi-line\r\n"
"string"

The trailing 'e' on the second line was lost when the cell was formatted.

----------------------------------------------------------------------

>Comment By: Vadim Zeitlin (vadz)
Date: 2008-02-03 22:05

Message:
Logged In: YES 
user_id=71618
Originator: NO

I guess the second occurrence of value.Mid() should be replaced with
tval.Mid() too, shouldn't it? I've changed it too but I don't have any test
code to check this so it would be great if you could retest please.

Also, at least in the trunk we really should remove this function
completely and use wxTextBuffer instead but this will have to wait for
another day, I'll just leave a TODO comment in the sources for now...

Thanks!

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=309863&aid=1882985&group_id=9863