Merge pull request #38 from pavoljuhas/fix-crystalvector-leak

vincefn · web-flow · commit a06411dea1e9 · 2018-04-22T14:41:14.000+02:00
Fix leak in CrystVector reported by valgrind
diff --git a/ObjCryst/CrystVector/CrystVector.cpp b/ObjCryst/CrystVector/CrystVector.cpp
@@ -182,7 +182,7 @@ template<class T> void CrystVector<T>::resize(const long newNbElements)
    if(mNumElements==newNbElements) return;
    VFN_DEBUG_MESSAGE("CrystVector<T>::resize():("<<mNumElements<<"->"
       <<newNbElements<<").",0)
-   if((mIsAreference==false) && (mNumElements != 0))
+   if(!mIsAreference)
    {
       delete[] mpData ;
    }

Original file line number	Diff line number	Diff line change
`@@ -182,7 +182,7 @@ template<class T> void CrystVector<T>::resize(const long newNbElements)`
`182`	`182`	`if(mNumElements==newNbElements) return;`
`183`	`183`	`VFN_DEBUG_MESSAGE("CrystVector<T>::resize():("<<mNumElements<<"->"`
`184`	`184`	`<<newNbElements<<").",0)`
`185`		`- if((mIsAreference==false) && (mNumElements != 0))`
	`185`	`+ if(!mIsAreference)`
`186`	`186`	`{`
`187`	`187`	`delete[] mpData ;`
`188`	`188`	`}`