[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Sun, 23 Sep 2018 18:48:33 +0200
On Windows, the functions "getenv" and "putenv" still fail with Octave dev if
one wants to query or set environment variables containing non-ASCII characters.
This implies that some functions (e.g. "which") are failing if a user with
non-ASCII characters in their name tries to run Octave.
I have a patch that switches to using the Unicode API for these functions.
However, I'm unsure how that should integrate into the code base.
For "putenv" that's pretty clear because we are already using a wrapper for
"getenv" is in the namespace "octave::sys::env".
Should I add the OS dependent code directly in "octave::sys::env:do_getenv"? Or
is it cleaner to add a new wrapper "octave::sys::getenv_wrapper" in lo-sysdep
(see attached patch)?
Thanks for any advise
Description: Binary data
- Code organisation,
Markus Mützel <=