it never did, but at: commit025bfa7e51Date: Thu Apr 23 10:03:58 2015 +0100 cppcheck: unreadVariable the wrong choice was made wrt the dead store, the dead loop should have been fixed rather than let the unused fill colors accumulate in the globalcolors buffer. This function doesn't need to consume the unused data, all callers don't expect the entire record to be read so just drop the loop. Signed-off-by: Caolán McNamara <caolan.mcnamara@collabora.com> Change-Id: I02bce4dd78dc566f18edc3d45603534c3f724cc3 Reviewed-on: https://gerrit.collaboraoffice.com/c/online/+/1689 (cherry picked from commit 07faeadace3d8d93225f9dd72c4f911e331070e2) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/205089 Tested-by: Jenkins Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org> (cherry picked from commit9ba100c0d2) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/205097