You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Devdeep Singh <de...@citrix.com> on 2013/06/27 18:49:06 UTC
Re: Review Request 11324: remove dead allocations
> On June 18, 2013, 6:08 a.m., Prasanna Santhanam wrote:
> > 93938ca716f85c24f2d9fb07c5856055f0061415
Dave, the request has been merged to master by Prasanna. Can you close this request.
- Devdeep
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11324/#review22047
-----------------------------------------------------------
On June 17, 2013, 1:40 p.m., Dave Brosius wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11324/
> -----------------------------------------------------------
>
> (Updated June 17, 2013, 1:40 p.m.)
>
>
> Review request for cloudstack and Devdeep Singh.
>
>
> Repository: cloudstack-git
>
>
> Description
> -------
>
> code allocates a collection, and then immediately overwrites the reference variable holding that collection. That original collection is just a dead allocation, and is not needed - patch removes.
>
>
> Diffs
> -----
>
> api/src/org/apache/cloudstack/api/command/admin/host/FindHostsForMigrationCmd.java b2d77b8
> api/src/org/apache/cloudstack/api/command/admin/host/ListHostsCmd.java 69c6980
> api/src/org/apache/cloudstack/api/command/admin/region/ListPortableIpRangesCmd.java 75bcce0
> api/src/org/apache/cloudstack/api/command/user/iso/ListIsosCmd.java f872c12
> api/src/org/apache/cloudstack/api/command/user/template/ListTemplatesCmd.java f0fc241
>
> Diff: https://reviews.apache.org/r/11324/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Dave Brosius
>
>