BugFixes, Extended RTTIExposer and extended TimeFunctions
Issue #213
resolved
Have no permission to make pull request, so make it in fork.
Commit description:
1 dwsComp.pas : fixed bug when using function declaring like
FDWSUnitImport.Functions.Add('function WeeksBetween(const ANow, AThen: Float): Integer;');
with Name parsing for Unit, and then overriding function Result with empty string (default param value in "Add" method)2 Extended RTTIExposer (dyn arrays, class indexed properties, sets, overloads for constructors, and more.)
3 Small extension for TimeFunctions - added all IncXXX methods (and removed unused 3rd param, no sense to have it there).
Later on will expand RTTIExposer.
Comments (6)
-
repo owner
-
repo owner
- changed status to resolved
Fixed issue
#213merged dwsRTTIExposer changes (compiles, but untested)→ <<cset af174e9c87bb>>
-
reporter
Tomorrow will arrange for you test cases from working methods, today have no time for PC.
-
repo owner
Thanks, there is no hurry, it has been in stasis for years now :)
-
reporter
regarding your questions:
Would you have any RTTIExposer tests or have a look at restoring URTTIExposeTests ?
I make no any tests here and have no idea how to do this, use folder “Test” and some sort of help info only :)
we are not using the RTTI features here as most of the VCL/RTL is not “safe” to expose to potential scripting abuse
I’m using RRTI only for easy access to typed data. VCL not interesting at all - DWS works in thread as non-gui script engine.
So, about tests:
file with tests here:https://easyupload.io/aj20vg
-
reporter
- Log in to comment
Thanks, I will add test cases for the new IncXxxx functions.
Would you have any RTTIExposer tests or have a look at restoring URTTIExposeTests ? Those have fallen in disrepair (we are not using the RTTI features here as most of the VCL/RTL is not “safe” to expose to potential scripting abuse)