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

Please add Package Files for Delphi 10 (Seattle to Rio) #3

Closed
wants to merge 4 commits into from
Closed

Please add Package Files for Delphi 10 (Seattle to Rio) #3

wants to merge 4 commits into from

Conversation

CPsoftBE
Copy link

No description provided.

Limagito added 3 commits December 28, 2019 19:41
@CPsoftBE
Copy link
Author

Please add Package Files for Delphi 10 (Seattle to Rio)

@pult pult closed this Dec 29, 2019
@pult
Copy link
Owner

pult commented Dec 29, 2019

Merged! Thanks!

@pult pult reopened this Dec 29, 2019
@pult pult closed this Dec 29, 2019
Update from Pult Fork
@CPsoftBE
Copy link
Author

CPsoftBE commented Dec 30, 2019 via email

@pult
Copy link
Owner

pult commented Dec 30, 2019

Need check FormatSettings. It is equal from windows-user system settings.
My sample work correctly for my DateFormat: dd/mm/yyyy

var D: TDateTime; S: string; V: Variant;
begin
  S := '30/12/2019';
  D := StrToDate(S);
  V := D;
  writeln(  VarToStr(V) );
end.

rpsRunner.exe -i="a".rps

18:13:45:398 # 30/12/2019

ps: rpsRunner.exe -it my main tooll (x86) for execute rps scripts.

@pult
Copy link
Owner

pult commented Dec 30, 2019

x64 result:
18:27:38:131 # 00:00:00
:(
Need detect defect ...

@CPsoftBE
Copy link
Author

CPsoftBE commented Dec 30, 2019 via email

@CPsoftBE
Copy link
Author

CPsoftBE commented Dec 30, 2019 via email

@pult
Copy link
Owner

pult commented Dec 30, 2019

bug in x64.inc ... ide-screenshot

@pult
Copy link
Owner

pult commented Dec 30, 2019

writeln( DateToStr(Now) );
It seems like an error when forming parameters before calling x64call.
Probably wrong address of variable.

@CPsoftBE
Copy link
Author

CPsoftBE commented Dec 30, 2019 via email

@CPsoftBE
Copy link
Author

Yes, the problem is in x64.inc
When I replace this file with the one from RemObject svn 496 then it works again

@pult
Copy link
Owner

pult commented Dec 30, 2019

? svn 496

@CPsoftBE
Copy link
Author

CPsoftBE commented Dec 30, 2019 via email

@CPsoftBE
Copy link
Author

CPsoftBE commented Dec 30, 2019 via email

@pult
Copy link
Owner

pult commented Dec 30, 2019

Without REG_STACK_PTR_OFFSET0 your app can fall(GPF) when exist exception in any wrappered script-function.

Implementation of InvokeCall.inc is promising for the future. It needs to be fixed so that it supports all types of parameters.

@pult
Copy link
Owner

pult commented Dec 31, 2019

Fixed !!! Actual commit: 7d5ab60

@pult pult reopened this Dec 31, 2019
@pult pult closed this Dec 31, 2019
@CPsoftBE
Copy link
Author

CPsoftBE commented Dec 31, 2019 via email

@pult
Copy link
Owner

pult commented Dec 31, 2019

// Don...
Consider It a New Year Gift ! :)

@CPsoftBE
Copy link
Author

CPsoftBE commented Dec 31, 2019 via email

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

Successfully merging this pull request may close these issues.

2 participants