Mailing-List: contact avro-dev-help@hadoop.apache.org; run by ezmlm
Precedence: bulk
Reply-To: avro-dev@hadoop.apache.org
Message-ID: <87168150.1260918081970.JavaMail.jira@brutus>
Date: Tue, 15 Dec 2009 23:01:21 +0000 (UTC)
From: "Kevin Oliver (JIRA)" <jira@apache.org>
To: avro-dev@hadoop.apache.org
Subject: [jira] Commented: (AVRO-257) Dead code removal
In-Reply-To: <784046756.1260910278215.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/AVRO-257?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12791013#action_12791013 ] 

Kevin Oliver commented on AVRO-257:
-----------------------------------

Yep. Doug, you beat me to the punch by a couple of minutes. Same thing I now have locally. Looks good to me.

> Dead code removal
> -----------------
>
>                 Key: AVRO-257
>                 URL: https://issues.apache.org/jira/browse/AVRO-257
>             Project: Avro
>          Issue Type: Bug
>          Components: java
>            Reporter: Kevin Oliver
>            Assignee: Kevin Oliver
>            Priority: Trivial
>             Fix For: 1.3.0
>
>         Attachments: AVRO-257.patch, AVRO-257.patch
>
>
> There is dead code in org.apache.avro.reflect.ReflectData.validate().  The code is checking if datum is not an instance of Object. But it always is, as it is declared as Object and it is not null.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.