You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Benoy Antony (JIRA)" <ji...@apache.org> on 2014/01/07 01:38:51 UTC
[jira] [Commented] (HADOOP-10208) Remove duplicate initialization
in StringUtils.getStringCollection
[ https://issues.apache.org/jira/browse/HADOOP-10208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13863712#comment-13863712 ]
Benoy Antony commented on HADOOP-10208:
---------------------------------------
Attaching a simple patch which removes the duplicate initialization.
> Remove duplicate initialization in StringUtils.getStringCollection
> ------------------------------------------------------------------
>
> Key: HADOOP-10208
> URL: https://issues.apache.org/jira/browse/HADOOP-10208
> Project: Hadoop Common
> Issue Type: Improvement
> Affects Versions: 2.2.0
> Reporter: Benoy Antony
> Assignee: Benoy Antony
> Priority: Trivial
> Attachments: HADOOP-10208.patch
>
>
> The *values* is initialized twice.
> {code:title=StringUtils.java|borderStyle=solid}
> public static Collection<String> getStringCollection(String str){
> List<String> values = new ArrayList<String>();
> if (str == null)
> return values;
> StringTokenizer tokenizer = new StringTokenizer (str,",");
> values = new ArrayList<String>();
> {code}
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)