Bug #590: tmpdir on mswin32

• Status: Closed
• Assignee: -
• Target version: -

[ko1 (Koichi Sasada)]

=begin
　ささだです．

　tmpdir 初期化時，Win32API の SHGetFolderPath でパスを取ってくる
と，末尾に nul が入るため，expand_path が失敗するようです（rstrip し
ても nul は取れない）．とりあえず chop してしまったんですが，どうす
るのが良いでしょうか．

Index: lib/tmpdir.rb

--- lib/tmpdir.rb (リビジョン 19506)
+++ lib/tmpdir.rb (作業コピー)
@@ -18,7 +16,7 @@
begin
getdir = Win32API.new('shell32', 'SHGetFolderPath', 'LLLLP', 'L')
raise RuntimeError if getdir.call(0, CSIDL_LOCAL_APPDATA, 0,
0, windir) != 0

•  windir = File.expand_path(windir.rstrip)
  

•  windir = File.expand_path(windir.rstrip.chop)
  

  rescue RuntimeError
  begin
  getdir = Win32API.new('kernel32',
  'GetSystemWindowsDirectory', 'PL', 'L')

--
// SASADA Koichi at atdot dot net
=end

[matz (Yukihiro Matsumoto)]

=begin
まつもと ゆきひろです

In message "Re: [ruby-dev:36493] [Bug: 1.9] tmpdir on mswin32"
on Wed, 24 Sep 2008 11:25:33 +0900, SASADA Koichi [e-mail] writes:

|　tmpdir 初期化時，Win32API の SHGetFolderPath でパスを取ってくる
|と，末尾に nul が入るため，expand_path が失敗するようです（rstrip し
|ても nul は取れない）．とりあえず chop してしまったんですが，どうす
|るのが良いでしょうか．

rstripは末尾のnulを取る作りになってるはずです。とりあえず手
元で試した範囲内では取れます。

=end

[ko1 (Koichi Sasada)]

=begin
　ささだです．

Yukihiro Matsumoto wrote:

|　tmpdir 初期化時，Win32API の SHGetFolderPath でパスを取ってくる
|と，末尾に nul が入るため，expand_path が失敗するようです（rstrip し
|ても nul は取れない）．とりあえず chop してしまったんですが，どうす
|るのが良いでしょうか．

rstripは末尾のnulを取る作りになってるはずです。とりあえず手
元で試した範囲内では取れます。

　ここで得られる文字列は，"...\x00 " と，末尾に nul + スペース，と
いう構成になっているため「末尾の nul」ではなかったようです．

str = "foo\x00 "
p [str, str.rstrip, str.rstrip.rstrip]
#=> ["foo\000 ", "foo\000", "foo"]

　rstrip 2回やるのがいいんでしょうか．

--
// SASADA Koichi at atdot dot net

=end

[matz (Yukihiro Matsumoto)]

=begin
まつもと ゆきひろです

In message "Re: [ruby-dev:36495] Re: [Bug: 1.9] tmpdir on mswin32"
on Wed, 24 Sep 2008 11:45:09 +0900, SASADA Koichi [e-mail] writes:

|　ここで得られる文字列は，"...\x00 " と，末尾に nul + スペース，と
|いう構成になっているため「末尾の nul」ではなかったようです．
|
|str = "foo\x00 "
|p [str, str.rstrip, str.rstrip.rstrip]
|#=> ["foo\000 ", "foo\000", "foo"]
|
|　rstrip 2回やるのがいいんでしょうか．

うーん、じゃあ、「rstripを末尾のnulかスペースを削除」という仕
様にしてしまう(1.9では)というのはどうでしょうか。

=end

[usa (Usaku NAKAMURA)]

=begin
こんにちは、なかむら(う)です。

In message "[ruby-dev:36495] Re: [Bug: 1.9] tmpdir on mswin32"
on Sep.24,2008 11:45:09, [e-mail] wrote:

　ここで得られる文字列は，"...\x00 " と，末尾に nul + スペース，と
いう構成になっているため「末尾の nul」ではなかったようです．

17行目を、
windir = "\0"*(max_pathlen+1)
に変えるだけでよかったりしませんか。

それでは。

U.Nakamura [e-mail]

=end

[ko1 (Koichi Sasada)]

=begin
　ささだです．

U.Nakamura wrote:

17行目を、
windir = "\0"*(max_pathlen+1)
に変えるだけでよかったりしませんか。

　良いようです．

--
// SASADA Koichi at atdot dot net

=end

[usa (Usaku NAKAMURA)]

=begin
こんにちは、なかむら(う)です。

というわけで本件は解決したのですが、それはそれとして、

In message "[ruby-dev:36497] Re: [Bug: 1.9] tmpdir on mswin32"
on Sep.24,2008 14:15:17, [e-mail] wrote:

うーん、じゃあ、「rstripを末尾のnulかスペースを削除」という仕
様にしてしまう(1.9では)というのはどうでしょうか。

というのは嬉しいのでぜひとも。

あと、strip系にdeleteのように引数として削除する文字集合を渡せ
るようにするというアイデアもありますが、こっちはスクリプトの
互換性を破壊しないので、やるとしても1.9.1以降でもいいのかな。

それでは。

U.Nakamura [e-mail]

=end

[usa (Usaku NAKAMURA)]

• Status changed from Open to Closed
• % Done changed from 0 to 100

=begin
Applied in changeset r19513.
=end

[usa (Usaku NAKAMURA)]

=begin
こんにちは、なかむら(う)です。

In message "[ruby-dev:36500] Re: [Bug: 1.9] tmpdir on mswin32"
on Sep.24,2008 14:44:32, [e-mail] wrote:

In message "[ruby-dev:36497] Re: [Bug: 1.9] tmpdir on mswin32"
on Sep.24,2008 14:15:17, [e-mail] wrote:

うーん、じゃあ、「rstripを末尾のnulかスペースを削除」という仕
様にしてしまう(1.9では)というのはどうでしょうか。

というのは嬉しいのでぜひとも。

早速入りましたが、lstripも同様にしちゃいけないでしょうか?

あと、元々あったわけですが、t[-1]はVC++8 x64でバグることが

あるので避けて欲しいです。

それでは。

U.Nakamura [e-mail]

=end

[matz (Yukihiro Matsumoto)]

=begin
まつもと ゆきひろです

In message "Re: [ruby-dev:36503] Re: [Bug: 1.9] tmpdir on mswin32"
on Wed, 24 Sep 2008 17:02:14 +0900, "U.Nakamura" [e-mail] writes:

|早速入りましたが、lstripも同様にしちゃいけないでしょうか?

どーなんでしょうねえ。あんまり困ってないんですが。

|# あと、元々あったわけですが、t[-1]はVC++8 x64でバグることが
|# あるので避けて欲しいです。

こっちは私にはよくわかんないんで、適当に修正してコミットして
ください。どうしたらバグが避けられるか知りようがないので。

=end