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

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

まだまだpalmをつかっている鯉江です。

  Message-Id: <20081008.091752.215106266.nom@xxxxxxxxxxxxx>
  Date:       Wed, 08 Oct 2008 09:17:52 +0900 (JST)
  From:       Yoshinari Nomura <nom@xxxxxxxxxxxxx>
  Subject:    [mhc:02304] Re: mhc_pilib.c for pilot-link 0.12

>> 対応ありがとうございます、先程確認しました。
>> ところで、普段使っていないので気付いていなかったのですが、(前回の
>> パッチとは無関係に) 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|

ここなんですが、実は↓じゃないでしょうか(文脈的に)。

@@ -669,7 +728,7 @@ class PilotApptRecord < PilotRecord
     end

     part1 .to_s .split("\n") .each{|line|
-      if !(string =~ /^[ \t]+/ or string =~ /^[A-Za-z0-9_-]+:/)
+      if !(line =~ /^[ \t]+/ or line =~ /^[A-Za-z0-9_-]+:/)
        part1_is_header = false
       end
     }

--
KOIE Hidetaka <hide@xxxxxxxx>