Github user DaanHoogland closed the pull request at:
https://github.com/apache/cloudstack/pull/578
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/578#issuecomment-121140173
LGTM :+1:
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user koushik-das commented on the pull request:
https://github.com/apache/cloudstack/pull/578#issuecomment-121138178
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user bhaisaab commented on the pull request:
https://github.com/apache/cloudstack/pull/578#issuecomment-120455619
LGTM (squash maybe)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
GitHub user DaanHoogland opened a pull request:
https://github.com/apache/cloudstack/pull/578
Coverity resource leaks
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/DaanHoogland/cloudstack
coverity-resource-leaks
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/564#issuecomment-119917601
LGTM :+1:
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/564
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user karuturi commented on the pull request:
https://github.com/apache/cloudstack/pull/564#issuecomment-119546107
changes looks good. :+1:
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not
GitHub user DaanHoogland opened a pull request:
https://github.com/apache/cloudstack/pull/564
coverity resource leaks
This PR holds fixes for different resource leaks of the type that can be
fixed by using try-with resource. The complete category is to be absent from
coverity
