294c870bed
Coverity threw an error that in the else branch, the expression colorTableRGB16[tableSize - 1] accesses uninit'ed data. The working theory is that it fails to perform the implication isEmpty() → size() == 0 This patch, therefore, checks for size() == 0 instead of isEmpty(), which is hardly less readable and might help Coverity understand the code better. Then again, Coverity might not understand that the tableSize can never be negative. If that's the case, another patch will be needed, but let's try the simpler solution first. Coverity-Id: 11420 Change-Id: Ibfe2a798c55af95c8001fa909aa94a6c5bc7c647 Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io> |
||
|---|---|---|
| .. | ||
| accessible | ||
| animation | ||
| doc | ||
| image | ||
| itemmodels | ||
| kernel | ||
| math3d | ||
| opengl | ||
| painting | ||
| text | ||
| util | ||
| Qt5GuiConfigExtras.cmake.in | ||
| QtGui.dynlist | ||
| gui.pro | ||