-
-
Notifications
You must be signed in to change notification settings - Fork 218
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
Fix for #8082 by making engine to use user buffers directly #8145
base: master
Are you sure you want to change the base?
Conversation
At this point the testcase for #8082 produces exactly expected output. |
Modified testcase that also check coercion of input data. |
#8185 is already fixed in different ways in master and v5. I see no relation of #8189 with this PR. I actually see no relation of this PR with actual problem of #8082. In reality, I do not understand what is this PR about. |
Sorry, "reference to cursor" meant here. And Delphi applications are not affected more than others. My fault. |
…ate message buffer
Additionally failed QA tests:
|
Could someone review this, please? |
Updated, ready to merge. |
I wonder why these Android runners are so unstable. |
Plan "A" is simple: