[ruby-list:26240] Re: String#sub! on RAA: auto bookmark ????

ども、時田です。


Fri, Dec 01, 2000 at 09:51:42AM +0900 において
Yukihiro Matsumoto さん曰く:

> まつもと ゆきひろです
> 
> In message "[ruby-list:26232] String#sub!  on RAA: auto bookmark ????"
>     on 00/11/30, Tokita Kousui <[e-mail]> writes:
> 
> |例の、String#concat で追加が出来なくなったから String#<< を使うようにする
> |のはわかったんですが
> 
> String#concatで追加できないってなんでしたっけ？
> Array#concatで非配列要素の追加ができなくなったってのはありま
> したが。

すいません、間違いでした
Array#concat での話でした。


> |bkmksubrt.rb の中の print_dir_selection って関数で
> |i.sub!( /^#{d}/, "" )
> |の結果、 変数 d がクリアされてしまいます
> |これの理由が良くわかりません
> 
> 私にも分かりません。再現するスクリプトが用意できますか？

原因がわからないので、再現する最小構成は用意できません すいません
最小構成を作ろうとして、うまく動いてしまったので
それはスレッド先頭の記事に書きました

RAA の WWW の所に登録されている[た] 「ringo's auto bookmark」の旧版 0.32
autobkmk-0.32.tar.gz での話ですが

Linux version 2.2.12-32 (古くてすいません レンタルサーバなので たしか LASER) で
ruby 1.6.1 と、0.32 autobkmk-0.32.tar.gz で
bkmksubrt.rb に 下記パッチをあてて、適当にデータを作った後に

print_dir_selection 関数にあたった部分のパッチの
i = "#{i}"
をコメントアウトして、ブックマークの適当なエントリを MOVE すると
WWWサーバのエラーログで   C val d:  の所で変数 d が空白文字列になります

変数 d は使い回しているようなので ログの先頭で再現されて
それ以降 変数 d は全て空白文字列になります


なにか、全然はずしてるのだろうか？
作者の修正が concat の部分だけで済んだって事は こっちの環境？？？ まさか(^^;



パッチはあまり作らないので、付けるべきオプションを入れてないかも知れません
とりあえず、 -c だけは指定しました。

＊＊＊＊パッチここから＊＊＊＊

*** bkmksubrt.rb	Sun Jan 23 23:03:17 2000
--- ../new/bkmksubrt.rb	Fri Dec  1 11:45:11 2000
***************
*** 124,129 ****
--- 124,130 ----
  	dirs.each { |i|
  		t += "#{i}/"
  		fols.push( get_line_file( "#{dd}/#{t}/info", 0 ) )
+ #		fols << get_line_file( "#{dd}/#{t}/info", 0 )
  	}
  	return fols.join( "/" )
  end
***************
*** 132,147 ****
  
  # d : db directory name
  def print_dir_selection( d )
- 
  	findout = findfunc ( d, "d", true )
  	print findout
  	print "<option selected value=\"/\">/\n"
  	findout.each { |i|
  		i.tr!( "\n", "" )
  		i.sub!( /^#{d}/, "" )
  		next if ( i == "" )
  		next if ( i =~ /_ck/ )
- 		STDERR.print "I: #{i}\n"
  		foln = replace_dir_with_folder_name( i , d )
  		print "<option value=\"#{i}\">#{foln}"
  	}
--- 133,151 ----
  
  # d : db directory name
  def print_dir_selection( d )
  	findout = findfunc ( d, "d", true )
  	print findout
  	print "<option selected value=\"/\">/\n"
  	findout.each { |i|
+ 		STDERR.print "A val d: #{d}\n"
+ 		i = "#{i}"
  		i.tr!( "\n", "" )
+ 		STDERR.print "B val d: #{d}\n"
  		i.sub!( /^#{d}/, "" )
+ 		STDERR.print "C val d: #{d}\n"
+ 
  		next if ( i == "" )
  		next if ( i =~ /_ck/ )
  		foln = replace_dir_with_folder_name( i , d )
  		print "<option value=\"#{i}\">#{foln}"
  	}
***************
*** 195,205 ****
  	if ( recursive == nil )
  		prefix = "#{dir}/"
  		Dir::foreach( dir ) do |f|
! 			allfiles.concat( f )
  		end
  	else
  		Find::find( dir ) do |f|
! 			allfiles.concat( f )
  		end
  	end
  
--- 199,211 ----
  	if ( recursive == nil )
  		prefix = "#{dir}/"
  		Dir::foreach( dir ) do |f|
! #			allfiles.concat( f )
! 			allfiles << f
  		end
  	else
  		Find::find( dir ) do |f|
! #			allfiles.concat( f )
! 			allfiles << f
  		end
  	end
  
***************
*** 208,216 ****
  	allfiles.each do |f|
  		if ( File::ftype( "#{prefix}#{f}" ) == findtype )
  			if ( regexp == nil )
! 				result.concat( f )
  			else
! 				result.concat( f ) if /#{regexp}/ =~ f
  			end
  		end
  	end
--- 214,224 ----
  	allfiles.each do |f|
  		if ( File::ftype( "#{prefix}#{f}" ) == findtype )
  			if ( regexp == nil )
! #				result.concat( f )
! 				result << f
  			else
! #				result.concat( f ) if /#{regexp}/ =~ f
! 				( result << f )  if /#{regexp}/ =~ f
  			end
  		end
  	end


＊＊＊＊パッチここまで＊＊＊＊

-- 
時田幸粋  <[e-mail]>
http://ibis.where-i.net/index.htm