this check always will result in true

Actually I believe that pData is non-NULL in all cases now but proving
it is quite complicated because there are a few too many friends of the
class.

Change-Id: I024605676281c0a4a46b42827b00ef98bf68ea79

sc/source/core/data/attarray.cxx

@@ -142,12 +142,9 @@ void ScAttrArray::Reset( const ScPatternAttr* pPattern )
 
         nCount = nLimit = 1;
         pData = new ScAttrEntry[1];
-        if (pData)
-        {
-            ScPatternAttr* pNewPattern = (ScPatternAttr*) &pDocPool->Put(*pPattern);
-            pData[0].nRow = MAXROW;
-            pData[0].pPattern = pNewPattern;
-        }
+        ScPatternAttr* pNewPattern = (ScPatternAttr*) &pDocPool->Put(*pPattern);
+        pData[0].nRow = MAXROW;
+        pData[0].pPattern = pNewPattern;
     }
 }