On Tue, Mar 24, 2015 at 2:06 PM, Juha Manninen <juha.mannine...@gmail.com>
wrote:

>   // ToDo: Fix Result and the loop. "I" is not used for anything.
>   for I := 0 to FrameWidth - 1 do
>     Result := Boolean(DrawEdge(DC, ARect, Edge[Style], BF_RECT or
> BF_ADJUST));
>

false positive.

https://msdn.microsoft.com/en-us/library/windows/desktop/dd162477%28v=vs.85%29.aspx
BF_ADJUST causes ARect to be different on every call. Side-effect function
:)

hmm... of is DrawEdge declaration wrong having ARect be passed as "const"
rather than "var"?

thanks,
Dmitry
--
_______________________________________________
Lazarus mailing list
Lazarus@lists.lazarus.freepascal.org
http://lists.lazarus.freepascal.org/mailman/listinfo/lazarus

Reply via email to