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)