You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by jo...@apache.org on 2009/06/18 23:10:45 UTC

svn commit: r786284 - /poi/trunk/src/testcases/org/apache/poi/hssf/record/TestExternalNameRecord.java

Author: josh
Date: Thu Jun 18 21:10:45 2009
New Revision: 786284

URL: http://svn.apache.org/viewvc?rev=786284&view=rev
Log:
Tweak to comments in TestExternalNameRecord (47384/786261)

Modified:
    poi/trunk/src/testcases/org/apache/poi/hssf/record/TestExternalNameRecord.java

Modified: poi/trunk/src/testcases/org/apache/poi/hssf/record/TestExternalNameRecord.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/hssf/record/TestExternalNameRecord.java?rev=786284&r1=786283&r2=786284&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/hssf/record/TestExternalNameRecord.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/hssf/record/TestExternalNameRecord.java Thu Jun 18 21:10:45 2009
@@ -137,7 +137,7 @@
 	}
 
 	public void testUnicodeName_bug47384() {
-		// 0x13A0
+		// data taken from bugzilla 47384 att 23830 at offset 0x13A0
 		byte[] dataUN = HexRead.readFromString(
 				"23 00 22 00" +
 				"00 00 00 00 00 00 " +
@@ -150,8 +150,6 @@
 		try {
 			enr = new ExternalNameRecord(in);
 		} catch (RecordFormatException e) {
-			// actual msg reported in bugzilla 47229 is different
-			// because that seems to be using a version from before svn r646666
 			if (e.getMessage().startsWith("Expected to find a ContinueRecord in order to read remaining 242 of 268 chars")) {
 				throw new AssertionFailedError("Identified bug 47384 - failed to read ENR with unicode name");
 			}



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org