[
https://issues.apache.org/jira/browse/HBASE-3444?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13475253#comment-13475253
]
Ted Yu commented on HBASE-3444:
-------------------------------
No the patch is not the same as current code. See this boundary check:
{code}
+ char next = i+1 < in.length() ? in.charAt(i+1) : ch; // check for
array out of bounds
{code}
Without the above, in.charAt(i+1) would produce ArrayIndexOutOfBoundsException
Thanks
> Bytes.toBytesBinary and Bytes.toStringBinary() should be reversible
> --------------------------------------------------------------------
>
> Key: HBASE-3444
> URL: https://issues.apache.org/jira/browse/HBASE-3444
> Project: HBase
> Issue Type: Bug
> Reporter: Prakash Khemani
> Assignee: Ken Dallmeyer
> Priority: Minor
> Fix For: 0.96.0
>
> Attachments: 3444.txt
>
>
> Bytes.toStringBinary() doesn't escape \.
> Otherwise the transformation isn't reversible
> byte[] a = {'\', 'x' , '0', '0'}
> Bytes.toBytesBinary(Bytes.toStringBinary(a)) won't be equal to a
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
