ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 33604] - classconstants filter reader reads a limited number of constants per class.
Date Sun, 17 Sep 2006 22:10:19 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=33604>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33604


peterreilly@apache.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         OS/Version|Windows Server 2003         |All
         Resolution|                            |FIXED
   Target Milestone|---                         |1.7




------- Additional Comments From peterreilly@apache.org  2006-09-17 22:10 -------
Martijn is correct, this is a character
encoding problem. Both UTF-8 (common as default on
linux) and CP152 (common as default on windows) readers cannot read
in an arbitary sequence of bytes, convert them
to characters and back again to bytes. CP152 can read/write
most, but not all, 8-bit bytes. this is the reason that the
Ronen (on windows) is able to read smaller files (less chance of illegal
bytes). The reason Sebastian sees failures for both the files
is that UTF-8 has a lot more illegal byte sequences than CP152.
Why it works for ant 1.6.1, I am not too sure - perhaps the
default character encoding was set to latin1 ?.

In any case using the default encoding is clearly incorrect, so I have
made a modification to ant to use ISO-8859-1 as the character
encoding to use to convert the characters back to bytes, and updated the manual
with a big block of text to say that one has to use ISO-8859-1 as
the encoding. The handling of exceptions has also been altered
to remove the spurious IOException, as the exception reporting
the problem was thrown away.
Thanks for the report and for the brillant test case (will
be incorporated into ant test suite when I get a chance).
(Missed the cut for ant 1.7.0beta2, will be in release after that!)

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org


Mime
View raw message