----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11750/#review21997 -----------------------------------------------------------
Ship it! Applied to master: 564013bec0d4356232d93ac52a3e44638578bff0 Thanks for the patch! - Chip Childers On June 8, 2013, 9:15 p.m., Laszlo Hornyak wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11750/ > ----------------------------------------------------------- > > (Updated June 8, 2013, 9:15 p.m.) > > > Review request for cloudstack. > > > Description > ------- > > Removed a String instntiation, test case added > > > Diffs > ----- > > utils/src/com/cloud/utils/net/NetUtils.java 8c094c8 > utils/test/com/cloud/utils/net/NetUtilsTest.java 16d3402 > > Diff: https://reviews.apache.org/r/11750/diff/ > > > Testing > ------- > > Test included > > > Thanks, > > Laszlo Hornyak > >