[fpc-devel] Twice stored record RTTI data

[Maciej Izak]

2016-03-02 19:30 GMT+01:00 Sven Barth <pascaldragon at googlemail.com>:

> I can already tell you now that this part of your code will definitely not
> be merged then.
>
ok. no problem with that, I got used to, similar like many other Delphi
compatible code - for example Generics.Collections. ;)


> It will break code that relies on non-managed fields being present in the
> RTTI. As Jonas said our RTTI is not guaranteed to be Delphi compatible, so
> *introducing* Delphi compatibility while *sacrificing* backwards
> compatibility (namely to enumerate non-managed fields) is *not* acceptable.
>

Who is using non-managed fields from array that presents managed fields?
Srsly? Proper code will not work with current implementation. It can be
different in *details*, but it should be *functional similar* to Delphi -
in this case it is not.

The usage of this FPC part is marginal so patch probably don't break
anything, and even more - it provide better Delphi compatibility on
"functional" level.

FPC is holding non-managed fields in array for managed fields - terrible.
Nothing more to say.
-- 
Best regards,
Maciej Izak
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freepascal.org/pipermail/fpc-devel/attachments/20160302/5afbe559/attachment.html>