From 556373c41fddbac6cbee1c89e7707631270a1852 Mon Sep 17 00:00:00 2001
From: Caolán McNamara <caolanm@redhat.com>
Date: Wed, 11 Mar 2015 09:06:08 +0000
Subject: V668 no sense in testing the result of new against null

Change-Id: I4a33bd92fc8448638a4bfe1eab7e5041a4c5cc39
---
 idlc/source/parser.y | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

(limited to 'idlc')

diff --git a/idlc/source/parser.y b/idlc/source/parser.y
index af911c655d68..e041de7b8eb5 100644
--- a/idlc/source/parser.y
+++ b/idlc/source/parser.y
@@ -566,8 +566,7 @@ interface_dcl :
             pInterface = new AstInterface(
                 *$1->getName(),
                 static_cast< AstInterface const * >(resolveTypedefs($1->getInherits())), pScope);
-            if ( pInterface &&
-                (pDecl = pScope->lookupByName(pInterface->getScopedName())) )
+            if ( (pDecl = pScope->lookupByName(pInterface->getScopedName())) )
             {
                 /*
                  * See if we're defining a forward declared interface.
-- 
cgit