[Support]

Thank you, code will be changed in the nearest release.

Quote:

if ColorsHolder = nil then // <= This should be if Assigned(ColorsHolder) then

Why this line should be changed?

[Lasse]

There is no reason to free an object if it is already nil (that would be an AV). That causes a part of that memory leak. Another part comes from the string list that is created twice.

Code:

if ColorsHolder nil then
ColorsHolder.Free;

Would make sense but ColorsHolder = nil does not… I rather use Assigned and not Assigned than nil and = nil…

[Support]

Ahh, really.. My mistake.

[Author]

Posts