https://issues.apache.org/bugzilla/show_bug.cgi?id=50726
--- Comment #3 from Konstantin Kolinko <knst.koli...@gmail.com> 2011-02-08
04:15:35 EST ---
> field = (new String("longstring_part1") +
> new String("longstring_part2") +
> ...
> new String("longstring_partn")).toCharArray();
I think that wouldn't work. What will work is to generate several arrays and
call out.write() with each one in turn.
Note that the 64K limit is not the char count aka str.length(), but the count
of bytes in UTF-8 encoding. So, either that count should be calculated (better
without String.toBytes() call) or the threshold should be like (64K / (max
count of bytes in char in UTF-8)).
I wonder also whether initializing the array as = {'s', 't', 'r', 'i', 'n',
'g'} performs worse or better than the existing code.
Anyway, a test case is needed.
BTW, using mappedFile=false will probably hit the same limit.
(That is when the "breakAtLF" variable is false in that
visit(Node.TemplateText) method).
--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
