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

#17 Exception on simpleble_free #18

Merged
merged 1 commit into from
Oct 17, 2024
Merged

Conversation

magdel
Copy link
Contributor

@magdel magdel commented Jun 14, 2024

On win64 I catch exception but not always. Seems like we pass only part of pointer

@eriklins
Copy link
Owner

I tried this on Win11 but still does not work for me. Still only able to scan for devices but cannot connect. Did you apply any further changes to the code other than the declaration of SimpleBleFree...?

@magdel
Copy link
Contributor Author

magdel commented Oct 15, 2024

No, it's the only changes in simpleble.pas
I used https://github.com/eriklins/Pascal-Bindings-For-SimpleBLE-Library/blob/main/SimpleBleNotifyExample/SimpleBleNotifyExample.lpr as base for my application (https://github.com/magdel/ble2com) that scans and connects and then reads data stream.

And before fix it was working very very bad, some days it was not working at all. Fixing SimpleBleFree remove memory corruption as I see so it may help in a not straightforward way.
And also take a look at dll`s version, try versions that is in my app release.

@magdel magdel mentioned this pull request Oct 17, 2024
@eriklins eriklins merged commit 63a6497 into eriklins:main Oct 17, 2024
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