Im CreateSpecialImage kann ein IF/Assert nicht schaden, wenn das PixelFormat nicht stimmt, sonst kann man sich hier den Speicher zerschießen (z.B. wenn pf1 bis bf16 und somit der Speicher kleiner wäre)
Dein Code ist viel zu kompliziert
Da es überall der selbe Code/Threshold ist ...
Delphi-Quellcode:
var OutByte: PByte; // mit aktiver Pointer-Arithmetic
for y := 0 to Height - 1 do
begin
OutByte := OutBmp.ScanLine[y];
for x := Width * 3 - 1 downto 0 do
begin
if OutByte^ > Threshold then
OutByte^ := Threshold;
Inc(OutByte);
end;
end;
for y := 0 to Height - 1 do
begin
OutByte := OutBmp.ScanLine[y];
for OutByte := OutByte to OutByte + Width * 3 - 1 do
if OutByte^ > Threshold then
OutByte^ := Threshold;
end;
Man bräuchte nur "einmal" ScanLine abzufragen, da alle Lines hintereinander liegen,
aber da die Linien an im Speicher ausgerichtet/aligned sind (auf 4 Bytes),
sind eventuell nachträglich noch Bytes eingefügt.
Wenn Width ein Vielfaches von 4, dann nicht ... sonst nach jeder Line aufgerundet werden
oder man arbeitet z.B. mit pf32bit.
Außerdem aufpassen, dass die Lines meistens unten anfangen zu zählen, also die letzte Line zuerst im Speicher liegt.