GIFLoad array subscript below array bounds

[pdziepak]

Due to variable range propagation enabled GCC notices (incorrectly) that

newEntry[fOldCodeLength] = *fOldCode;

may be unsafe due to fOldCodeLength being a signed integer. When building under Haiku it results in a warning which is treated as an error.

[pdziepak]

To be honest I was a bit concerned about these copying loops but since I know nothing about GIF format and memcpy() is not an exact equivalent of them I decided to leave it as it is. It is out of scope of this patch anyway.

[jscipione]

Note that the warning causes Haiku not to build from within Haiku.

[leavengood]

Now that you have commit access I say +1 in committing this Pawel.

[pdziepak]

Fixed in hrev45358.