[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[mhc:02304] Re: mhc_pilib.c for pilot-link 0.12



乃村です.

On Tue, 07 Oct 2008 23:19:17 +0900,
	Kazuhiro Ito <kzhr@xxxxxxxxxxxxx> said:

> 対応ありがとうございます、先程確認しました。
> ところで、普段使っていないので気付いていなかったのですが、(前回の
> パッチとは無関係に) palm2mhc が Ruby 1.8.6 ではエラーが出て
> 動作しなくなっていました。
> 
> 添付のパッチで動くようにはなりしたが、Ruby 的に正しい書き方かどうかは
> よくわかりませんでしたので、どなたか確認いただければと思います。

最初の to_s は,ありがちな感じがするのですが,2番目は
何故ダメなのかよく分かりませんね.何というエラーが出るのでしょうか.

--- ruby-ext/lib/mhc-palm.rb	22 Jun 2004 10:09:15 -0000	1.11
+++ ruby-ext/lib/mhc-palm.rb	19 Sep 2008 08:57:46 -0000
@@ -668,7 +668,7 @@
       part1_is_header = false
     end
 
-    part1 .split("\n") .each{|line|
+    part1 .to_s .split("\n") .each{|line|


--- ruby-ext/lib/mhc-schedule.rb	3 Oct 2005 14:23:03 -0000	1.23
+++ ruby-ext/lib/mhc-schedule.rb	19 Sep 2008 08:57:47 -0000
@@ -1298,7 +1298,7 @@
     parent = File .dirname(dir)
     if makedir_or_higher(parent)
       Dir .mkdir(dir)
-      File .open(dir, "r") .fsync .close if File .method_defined?("fsync")
+      File .open(dir, "r") {|f| f .sync} if File .method_defined?("fsync")