Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Support for Serializing TStringList #1

Open
kralo opened this issue Jul 7, 2017 · 0 comments
Open

Add Support for Serializing TStringList #1

kralo opened this issue Jul 7, 2017 · 0 comments

Comments

@kralo
Copy link
Owner

kralo commented Jul 7, 2017

There currently is no support for serializing a tkMethod, like the FOnChange/ of a TStringList.

As a starter, in DeHL.Types.pas around 5114, you could do
// WARNING: dangerous hack to skip serializing TStringList Methods if (AField.Name = 'FOnChange') or (AField.Name = 'FOnChanging') then Exit(true);
to skip the fields. Better would be to add proper support in MethodType.DoSerialize.

Todo:
[ ] Write a test case that test an object with one member of type TStringList
[ ] Implement support

EDIT: Sh**, sorry. Wanted to add the Issue to my fork for own reference. Any chance to move it there? Or simply delete...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant