Re: svn commit: r24791 - in trunk/subversion/bindings/javahl: native src/org/tigris/subversion/javahl src/org/tigris/subversion/javahl/tests
On Tue, 01 May 2007, Mark Phippard wrote:
> On 5/1/07, Hyrum K. Wright <firstname.lastname@example.org> wrote:
> >The attached patch uses a byte array instead of a String to move the
> >property values from the Java layer to the C++ layer. I have a little
> >misgiving about using a ClientException, but I don't see a better way to
> >implement the error handling when specifying the UTF-8 encoding.
> I have only loosely followed this thread and do not really understand
> the C++ code enough to comment. But stepping back, we had two methods
> that presumably worked and did roughly the same thing. We wanted to
> apparently consolidate the C++ code to a single method. If both of
> the previous methods worked correctly than what is the complication
> here in terms of the encoding?
> In general I am just concerned when we are taking working code and
> changing it to eliminate redundancy but are not 100% sure we are not
> now adding problems.
I am unconvinced that the previous code actually worked for multi-byte
Received on Wed May 2 01:53:13 2007
- application/pgp-signature attachment: stored
This is an archived mail posted to the Subversion Dev