On Fri, Dec 12, 2008 at 06:58, Joost van der Sluis <jo...@cnoc.nl> wrote:
>> Attached patch does two things:
>> 1) Makes TLookupList.FList protected instead of private
>
> Maybe it's a better idea to add a protected property for this? Or is
> that not possible?

Actually, with other parts of this patch accepted FList may even remain private,
since the procedures I added are exactly what Lazarus (currently) needs.
However, they will only be available after fpc 2.4 is out,
which is IIUC still at least half a year away.
So to avoid such issues in the future I think 'protected' is
appropriate safeguard.

As for the property -- sure, it can be used, but I do not see any
essential difference
in this case. So whatever you prefer.


> And maybe next time, open a bug report with the patch, so it won't be
> forgotten. ;)
Actually, from my recent experience, the best course of action is to use
both mailing list and Mantis at once ;-)

-- 
Alexander S. Klenin
Insight Experts Ltd.
_______________________________________________
fpc-devel maillist  -  fpc-devel@lists.freepascal.org
http://lists.freepascal.org/mailman/listinfo/fpc-devel

Reply via email to