Feature #1947

rb_w32_getenv has naive implementation

Added by vo.x (Vit Ondruch) about 11 years ago. Updated over 9 years ago.

Target version:


In change set 10650 (and according to discussion in ruby-talk:205123), the rb_w32_getenv implementation was changed. However, I believe that the correct approach according to windows documentation should be to call GetEnvironmentVariable and check GetLastError. If the variable exists, then S_OK is returned or some error code in other case. Call to GetEnvironmentStrings and home made enumeration is cumbersome.


Updated by usa (Usaku NAKAMURA) about 11 years ago

  • Category set to core
  • Status changed from Open to Rejected
  • Assignee set to usa (Usaku NAKAMURA)
  • Priority changed from Normal to 3
  • Target version set to 2.0.0

There is a patch to use GetEnvironmentVariable() instead of using GetEnvironmentStrings(), written by Nobu.

However, this patch is as you see complex at the same level of current code.
Nobu and I discussed about this issue, and reached to a conclusion that we maintain the status quo for the time being.

Also available in: Atom PDF