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

The AddHostObject demo has an access violation in CallCallbackAsynchronously #2

Open
salvadordf opened this issue Dec 4, 2021 · 1 comment
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@salvadordf
Copy link
Owner

TSampleHostObjectClass.CallCallbackAsynchronously has an access violation when it calls Param1.Invoke with an empty GUID

@salvadordf salvadordf added the bug Something isn't working label Dec 4, 2021
@salvadordf salvadordf changed the title AddHostObject access violation in CallCallbackAsynchronously The AddHostObject demo has an access violation in CallCallbackAsynchronously Dec 4, 2021
@salvadordf salvadordf added the help wanted Extra attention is needed label Mar 5, 2024
@the-Arioch
Copy link
Contributor

the-Arioch commented Jan 19, 2025

A.S. it should be GUID_NULL var not TGUID.Empty function for the same of earlier Delphi versions. XE 2 does have itm buyt D2007 does not.


i did not observe AVs running demoes, and i think i saw your comments in the demo source that suggest you already fixed it

maybe i would give it a thorough look, if to know where to look at

ATM i do not see i would need this method at all in the limited functionality i would need (replace old DCEF), OTOH we mostly use WSH/JScript now, shich is single-thread and synchronous. Maybe i actually would need it, just do not know yet :-)

I would need to call methods and read/write properties of Delphi 1-thread core. This seems more naturaly done in synchronous way, that WebView2 provides already, i guess.

DCEF code used fake domain and custom URL scheme for it, which i hope to repeat for easier migration, but Host Objects feel more natural approach, so would implement them too.


that said, D2007 is notoriously bad with debugging on Win10, so if i did not see AVs it is not a proof there were none :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants