Github user asfgit closed the pull request at:
https://github.com/apache/cloudstack/pull/600
---
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 wido commented on the pull request:
https://github.com/apache/cloudstack/pull/600#issuecomment-122246414
@DaanHoogland You can merge it I think
---
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 d
Github user koushik-das commented on the pull request:
https://github.com/apache/cloudstack/pull/600#issuecomment-15132
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
e
Github user DaanHoogland commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/600#discussion_r34869004
--- Diff:
framework/ipc/src/org/apache/cloudstack/framework/serializer/OnwireClassRegistry.java
---
@@ -108,7 +107,7 @@ public void scan() {
Github user koushik-das commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/600#discussion_r34867930
--- Diff:
framework/ipc/src/org/apache/cloudstack/framework/serializer/OnwireClassRegistry.java
---
@@ -108,7 +107,7 @@ public void scan() {
Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/600#issuecomment-122014762
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 this
GitHub user DaanHoogland opened a pull request:
https://github.com/apache/cloudstack/pull/600
Coverity issues in OmniwireClassRegistry
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/DaanHoogland/cloudstack coverity-omniwire
Alt