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

SeExpr library and demos are vulnerable to the current C locale #100

Open
amyspark opened this issue Mar 13, 2020 · 1 comment
Open

SeExpr library and demos are vulnerable to the current C locale #100

amyspark opened this issue Mar 13, 2020 · 1 comment

Comments

@amyspark
Copy link

amyspark commented Mar 13, 2020

Hey,

I've tried compiling SeExpr's demos with Qt 5.12.7 on Manjaro and macOS Mojave (the latter through Homebrew).

In both cases, the ccolor widget in the Image Synthesis Editor initializes to a default yellow value and forcibly overrides the value set in the script. This doesn't happen in Windows 10 x64 (with the same Qt version).

The reason is that, in the Bison parser and the imageEditor demo all floating point parameters are parsed using atof(). Under Qt-based apps, the LANG and LC_ALL environment variables govern floating-point number format. I'm using es_ES.UTF8, which re-uses , as the decimal point separator.

@amyspark amyspark changed the title Qt ccolor() widget does not initialize with values on Linux and macOS SeExpr library and demos are vulnerable to the current C locale Jun 2, 2020
@davvid
Copy link
Member

davvid commented Nov 16, 2020

We'd be happy to take fixes in this area. It'd be good to make the code not dependent on the locale.

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

2 participants