I would store pColours[*pPixels] in a temporary, for clarity and ease of refactoring.