Return-Path: X-Original-To: apmail-incubator-accumulo-dev-archive@minotaur.apache.org Delivered-To: apmail-incubator-accumulo-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 8A7BC9298 for ; Thu, 8 Mar 2012 03:47:40 +0000 (UTC) Received: (qmail 62102 invoked by uid 500); 8 Mar 2012 03:47:40 -0000 Delivered-To: apmail-incubator-accumulo-dev-archive@incubator.apache.org Received: (qmail 61898 invoked by uid 500); 8 Mar 2012 03:47:39 -0000 Mailing-List: contact accumulo-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: accumulo-dev@incubator.apache.org Delivered-To: mailing list accumulo-dev@incubator.apache.org Received: (qmail 61776 invoked by uid 99); 8 Mar 2012 03:47:39 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 08 Mar 2012 03:47:39 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=5.0 tests=ALL_TRUSTED,T_RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.116] (HELO hel.zones.apache.org) (140.211.11.116) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 08 Mar 2012 03:47:37 +0000 Received: from hel.zones.apache.org (hel.zones.apache.org [140.211.11.116]) by hel.zones.apache.org (Postfix) with ESMTP id EFEF3F719 for ; Thu, 8 Mar 2012 03:47:14 +0000 (UTC) Date: Thu, 8 Mar 2012 03:47:13 +0000 (UTC) From: "David Medinets (Work started) (JIRA)" To: accumulo-dev@incubator.apache.org Message-ID: <350005537.37706.1331178435074.JavaMail.tomcat@hel.zones.apache.org> In-Reply-To: <4182501.37694.1331177584726.JavaMail.tomcat@hel.zones.apache.org> Subject: [jira] [Work started] (ACCUMULO-451) Change byte val[] to byte[] val in Mutation.java per PMD suggestion. MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/ACCUMULO-451?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Work on ACCUMULO-451 started by David Medinets. > Change byte val[] to byte[] val in Mutation.java per PMD suggestion. > -------------------------------------------------------------------- > > Key: ACCUMULO-451 > URL: https://issues.apache.org/jira/browse/ACCUMULO-451 > Project: Accumulo > Issue Type: Improvement > Reporter: David Medinets > Assignee: David Medinets > Priority: Trivial > Fix For: 1.5.0-SNAPSHOT > > > This issue generates three PMD warnings like this: > [WARNING] Error while parsing /home/medined/Desktop/Dropbox/david/workspace/accumulo/src/core/src/main/java/org/apache/accumulo/core/data/Mutation.java: Encountered " ":" ": "" at line 403, column 23. > Was expecting one of: > "[" ... > ";" ... > "," ... > "=" ... > This issue is that the code is looping over an array of byte[] but the actual code says "byte val[]". Both code constructs work. I wrote a test program to verify. However, where possible we should eliminate warning messages. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira