----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11767/#review21998 -----------------------------------------------------------
Ship it! Applied to master: c88d8fb3a2f6c418c6c7af8ff702a93bcdb2d752 Thanks for the patch! - Chip Childers On June 15, 2013, 4:32 p.m., Laszlo Hornyak wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11767/ > ----------------------------------------------------------- > > (Updated June 15, 2013, 4:32 p.m.) > > > Review request for cloudstack. > > > Description > ------- > > commons-lang is already a transitive dependency of the utils project, which > allows removing some duplicated functionality. > This patch replaces StringUtils.join(String, Object...) with it's > commons-lang counterpart. > It also replaces calls to String join(Iterable<? extends Object>, String) in > cases where an array is already exist and it is only wrapped into a List. > > > Diffs > ----- > > server/src/com/cloud/storage/s3/S3ManagerImpl.java 61e5573 > > services/secondary-storage/src/org/apache/cloudstack/storage/resource/NfsSecondaryStorageResource.java > e7fa5b2 > utils/src/com/cloud/utils/S3Utils.java b7273a1 > utils/src/com/cloud/utils/StringUtils.java 14ff4b1 > utils/test/com/cloud/utils/StringUtilsTest.java 3c162c7 > > Diff: https://reviews.apache.org/r/11767/diff/ > > > Testing > ------- > > - Unit test added > > > Thanks, > > Laszlo Hornyak > >
