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

Changing 'sv_setpvn' to 'sv_setpvn_mg' #171

Open
lzsiga opened this issue Mar 4, 2024 · 0 comments
Open

Changing 'sv_setpvn' to 'sv_setpvn_mg' #171

lzsiga opened this issue Mar 4, 2024 · 0 comments

Comments

@lzsiga
Copy link

lzsiga commented Mar 4, 2024

I've found some strange behavior in YaSQL which is a Sql*Plus alternative written in Perl that uses DBD-Oracle.

I've created a test-script to show the problem: https://github.com/lzsiga/yasql-fx/blob/main/test/test0001.sh

The problem has something to do with utf8, function substr, caching string-len and 'magical variables'.

One way to solve it would be changing every call of sv_setpvn to sv_setpvn_mg in oci8.c. I admit I don't really know why both still exist, I'd say sv_setpvn_mg is the reliable variant, sv_setpvn is the quick but unreliable one.

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

No branches or pull requests

1 participant