Hi
Sehe ich jetzt erst ... warum sind r,g,b float?
Sämtliche diese Werte annehmenden Methoden wollen dort ein Byte haben.
Auch ist mir der Sinn hinter int für die Pixelnummer nicht ganz klar, wobei unsigned int keinen Speicher spart und bei Beiden wohl der Wertebereich nicht ausgeschöpft wird.
Ein unsigned int (oder byte bei der geringen Anzahl an Pixeln) hätte aber durchblicken lassen, daß Du verstanden hast, was Da abläuft.
Naja - Hauptsache: Es läuft. Verbessern kann man IMMER, traurig, aber wahr 
MfG