Shahar Havivi has uploaded a new change for review.

Change subject: findbugs: method may pass null to method that cant get null 
value
......................................................................

findbugs: method may pass null to method that cant get null value

findbugs reason:
NP: Method call passes null to a nonnull parameter (NP_NONNULL_PARAM_VIOLATION)

Change-Id: Iec2c3225b58d8486cdf5d95c98f8b98593fb7fec
Signed-off-by: Shahar Havivi <[email protected]>
---
M 
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/tree/AbstractSubTabActionTree.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/94/15494/1

diff --git 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/tree/AbstractSubTabActionTree.java
 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/tree/AbstractSubTabActionTree.java
index 7400cf7..c04a411 100644
--- 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/tree/AbstractSubTabActionTree.java
+++ 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/tree/AbstractSubTabActionTree.java
@@ -80,7 +80,7 @@
         });
 
         actionPanel = createActionPanel(modelProvider);
-        if (actionPanel != null) {
+        if (actionPanel != null && tree != null) {
             actionPanelContainer.add(actionPanel);
             actionPanel.addContextMenuHandler(tree);
         }


-- 
To view, visit http://gerrit.ovirt.org/15494
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iec2c3225b58d8486cdf5d95c98f8b98593fb7fec
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to