From 9d449dbd97380e7085257e738e05fec741a4e21d Mon Sep 17 00:00:00 2001
From: Yegor Kozlov <yegor@apache.org>
Date: Thu, 1 Mar 2012 09:51:28 +0000
Subject: [PATCH] bugzilla 52682 - allow setting text with trailing carriage
 return in HSLF

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1295488 13f79535-47bb-0310-9956-ffa450edef68
---
 src/documentation/content/xdocs/status.xml                  | 1 +
 src/scratchpad/src/org/apache/poi/hslf/model/TextRun.java   | 6 ------
 .../testcases/org/apache/poi/hslf/model/TestTextRun.java    | 4 ++--
 3 files changed, 3 insertions(+), 8 deletions(-)

diff --git a/src/documentation/content/xdocs/status.xml b/src/documentation/content/xdocs/status.xml
index aeb943121b..64508fdb80 100644
--- a/src/documentation/content/xdocs/status.xml
+++ b/src/documentation/content/xdocs/status.xml
@@ -34,6 +34,7 @@
 
     <changes>
         <release version="3.8-beta6" date="2012-??-??">
+           <action dev="poi-developers" type="fix">52682 - allow setting text with trailing carriage return in HSLF</action>
            <action dev="poi-developers" type="fix">52244 - use correct text attributes when presentation has multiple TxMasterStyleAtoms of the same type</action>
            <action dev="poi-developers" type="add">support setting background color of sheet tab in XSSF</action>
            <action dev="poi-developers" type="add">51564 - support for enforcing fields update in XWPF</action>
diff --git a/src/scratchpad/src/org/apache/poi/hslf/model/TextRun.java b/src/scratchpad/src/org/apache/poi/hslf/model/TextRun.java
index 5eb3930638..ee69d87a7d 100644
--- a/src/scratchpad/src/org/apache/poi/hslf/model/TextRun.java
+++ b/src/scratchpad/src/org/apache/poi/hslf/model/TextRun.java
@@ -320,12 +320,6 @@ public final class TextRun
 	 *  touch the stylings.
 	 */
 	private void storeText(String s) {
-		// Remove a single trailing \r, as there is an implicit one at the
-		//  end of every record
-		if(s.endsWith("\r")) {
-			s = s.substring(0, s.length()-1);
-		}
-
 		// Store in the appropriate record
 		if(_isUnicode) {
 			// The atom can safely convert to unicode
diff --git a/src/scratchpad/testcases/org/apache/poi/hslf/model/TestTextRun.java b/src/scratchpad/testcases/org/apache/poi/hslf/model/TestTextRun.java
index f59eef3a5f..a82e551994 100644
--- a/src/scratchpad/testcases/org/apache/poi/hslf/model/TestTextRun.java
+++ b/src/scratchpad/testcases/org/apache/poi/hslf/model/TestTextRun.java
@@ -100,9 +100,9 @@ public final class TestTextRun extends TestCase {
 		run.setText(changeTo);
 		assertEquals(changeTo, run.getText());
 
-		// Ensure trailing \n's get stripped
+		// Ensure trailing \n's are NOT stripped, it is legal to set a text with a trailing '\r'
 		run.setText(changeTo + "\n");
-		assertEquals(changeTo, run.getText());
+		assertEquals(changeTo + "\n", run.getText());
 	}
 
 	/**
-- 
2.39.5