----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22454/#review45476 -----------------------------------------------------------
server/src/com/cloud/configuration/ConfigurationManagerImpl.java <https://reviews.apache.org/r/22454/#comment80335> podid wont be null/invalid as it is already checked in the dispatcher layer and hence it can be safely unboxed without a NPE. - Rajani Karuturi On June 11, 2014, 9:18 a.m., Rajani Karuturi wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22454/ > ----------------------------------------------------------- > > (Updated June 11, 2014, 9:18 a.m.) > > > Review request for cloudstack, daan Hoogland, Koushik Das, and Santhosh > Edukulla. > > > Repository: cloudstack-git > > > Description > ------- > > unused assignments, boxing and unboxing of values etc. > > > Diffs > ----- > > server/src/com/cloud/configuration/ConfigurationManagerImpl.java 55736f9 > > Diff: https://reviews.apache.org/r/22454/diff/ > > > Testing > ------- > > > Thanks, > > Rajani Karuturi > >
