Not all systems default to unicode. Though that looks doofy to me. Your code assumes they do. You'd need a flag saying "amIOnAnAS400()" or something ;-)

-Andy

Jason Height wrote:
All,

Any idea why the following line from UnicodeRecord (current HEAD rev and previous) is actually required? String unicodeString = new String(getString().getBytes("Unicode"),"Unicode");

If i remove it and use:
String unicodeString = getString();

1) All of the unit tests still pass, and
2) There is a 33x performance improvement with workbooks containing a large numbers of strings

I am tempted to apply a patch to use my approach. Any objections?

Jason

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
Mailing List:    http://jakarta.apache.org/site/mail2.html#poi
The Apache Jakarta POI Project: http://jakarta.apache.org/poi/




--
Andrew C. Oliver
SuperLink Software, Inc.

Java to Excel using POI
http://www.superlinksoftware.com/services/poi
Commercial support including features added/implemented, bugs fixed.


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
Mailing List:    http://jakarta.apache.org/site/mail2.html#poi
The Apache Jakarta POI Project: http://jakarta.apache.org/poi/

Reply via email to