Mailing-List: contact notifications-help@ant.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@ant.apache.org
From: bugzilla@apache.org
To: notifications@ant.apache.org
Subject: DO NOT REPLY [Bug 47382] Encoding attribute of loadproperties does
 not function correctly
In-Reply-To: <bug-47382-48744@https.issues.apache.org/bugzilla/>
References: <bug-47382-48744@https.issues.apache.org/bugzilla/>
Auto-Submitted: auto-generated
Content-Type: text/plain; charset="UTF-8"
MIME-Version: 1.0
Message-Id: <20090619135628.0D81E234C1E7@brutus.apache.org>
Date: Fri, 19 Jun 2009 06:56:27 -0700 (PDT)

https://issues.apache.org/bugzilla/show_bug.cgi?id=47382


Oliver Deakin <oliver.deakin@googlemail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |


--- Comment #4 from Oliver Deakin <oliver.deakin@googlemail.com>  2009-06-19 06:56:27 PST ---
Hi Stefan - thanks for looking at this so quickly! 

I just had a look at the change you made to call text.getBytes() without the
encoding specified and Im pretty sure that will also fail. If an encoding is
not specified, getBytes() defaults to converting from the default system
encoding to the Java default encoding.

So in my case calling text.getBytes() is the same as calling
text.getBytes("IBM-1047"), and will still try and convert text again from
EBCDIC->UTF-16. I think you need to explicitly specify that text is already in
UTF-16 to getBytes i.e. text.getBytes("UTF-16"). (Im assuming that UTF-16 is
the correct encoding here)

That said, I havn't tested the change yet locally - does the test you added
pass?

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