diff options
| author | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2013-01-30 04:56:56 +0100 |
|---|---|---|
| committer | Markus Mohrhard <markus.mohrhard@googlemail.com> | 2013-01-30 19:32:03 +0100 |
| commit | e8226a93aca391e9abe63edebc87aec05d3bf5bf (patch) | |
| tree | 2536bba223b6a16a1fcff99298e2181bda75a973 | |
| parent | 1830e257d85481bed61774fc8febf302a35c1dea (diff) | |
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
| -rw-r--r-- | sc/source/core/data/attarray.cxx | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/sc/source/core/data/attarray.cxx b/sc/source/core/data/attarray.cxx index 39c87d024ca8..50a13971cf96 100644 --- a/sc/source/core/data/attarray.cxx +++ b/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; } } |