Let's merge this as-is for 4.6 and solve a blocker. The improvement can be done later in a separate PR.
Sent from my iPhone > On 02 Oct 2015, at 10:29, DaanHoogland <g...@git.apache.org> wrote: > > Github user DaanHoogland commented on the pull request: > > https://github.com/apache/cloudstack/pull/885#issuecomment-144958879 > > Had a look, the NPE won't occur within the scope of this fix. I would > suggest some changes anyway to prevent future issues as the method > isPowerStateUpToDate() is public. > > > --- > 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 enabled but not working, please > contact infrastructure at infrastruct...@apache.org or file a JIRA ticket > with INFRA. > ---
