assume malloc() does not fail in QVarLengthArray<T>::reallocData()

even when malloc() is called with 0 as size it returns non-null pointer
(some random garbage)

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

diff --git a/src/core/tools/qvarlengtharray.h b/src/core/tools/qvarlengtharray.h
index 92da592..c0eca94 100644 (file)
--- a/src/core/tools/qvarlengtharray.h
+++ b/src/core/tools/qvarlengtharray.h
@@ -206,23 +206,18 @@ Q_OUTOFLINE_TEMPLATE void QVarLengthArray<T, Prealloc>::reallocData(int asize, i
     if (aalloc != a) {
         ptr = static_cast<T *>(malloc(aalloc * sizeof(T)));
         Q_CHECK_PTR(ptr);
-        if (ptr) {
-            s = 0;
-            a = aalloc;
-
-            if (QTypeInfo<T>::isStatic) {
-                // copy all the old elements
-                while (s < copySize) {
-                    new (ptr+s) T(*(oldPtr+s));
-                    (oldPtr+s)->~T();
-                    s++;
-                }
-            } else {
-                memcpy(ptr, oldPtr, copySize * sizeof(T));
+        s = 0;
+        a = aalloc;
+
+        if (QTypeInfo<T>::isStatic) {
+            // copy all the old elements
+            while (s < copySize) {
+                new (ptr+s) T(*(oldPtr+s));
+                (oldPtr+s)->~T();
+                s++;
             }
         } else {
-            ptr = oldPtr;
-            return;
+            memcpy(ptr, oldPtr, copySize * sizeof(T));
         }
     }
     s = copySize;