You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by "Benjamin Lerer (JIRA)" <ji...@apache.org> on 2016/03/17 10:38:33 UTC
[jira] [Commented] (CASSANDRA-10748) UTF8Validator.validate() wrong
??
[ https://issues.apache.org/jira/browse/CASSANDRA-10748?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15199219#comment-15199219 ]
Benjamin Lerer commented on CASSANDRA-10748:
--------------------------------------------
{quote}In Modified UTF-8, the null character (U+0000) is encoded as {{0xC0,0x80}}.{quote}
This seems to indicate that this code is needed. I believe the problem is more that an else is missing.
> UTF8Validator.validate() wrong ??
> ---------------------------------
>
> Key: CASSANDRA-10748
> URL: https://issues.apache.org/jira/browse/CASSANDRA-10748
> Project: Cassandra
> Issue Type: Bug
> Reporter: Robert Stupp
> Assignee: Benjamin Lerer
> Priority: Minor
>
> The switch-statement in {{UTF8Validator.validate()}} can never go into {{case TWO_80}} as the assignment {{state = State.TWO_80;}} in line 75 is dead.
> I assume that the {{TWO_80}} case is completely superfluous - but would like to have a 2nd opinion on this.
> /cc [~carlyeks] (CASSANDRA-4495)
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)