Re: w3m-bookmark-file-modtime

[Naohiro Aota <nao.aota@xxxxxxxxx>]

青田です。

Katsumi Yamaoka <yamaoka@xxxxxxx> writes:

>>>>>> In [emacs-w3m : No.10417] 青田さん wrote:
>> 詳しいテストありがとうございます。 docstring と実体が違っていたのですね…。
>> Emacs については etc/NEWS.22 に
>
>> .-------------------------------------------------------------------------------
>>| * Lisp Changes in Emacs 22.1
>>| <snip>
>>| *** `visited-file-modtime' and `calendar-time-from-absolute' now return
>>| a list of two integers, instead of a cons.
>> `-------------------------------------------------------------------------------
>
>> とあるので、以下のようなコードで解決できそうに思います。
>
> うーむ、それでも良いと思うんですが、ちょっと迷ってしまいました。
> 旧版の Emacs で 2種類あった modtime の表現形式を新版で統一したこ
> とが The Right Thing なのであれば、進取の emacs-w3m でも踏襲し、
> `w3m-bookmark-file-modtime' の出力は常に list にしておいて、版に
> よって変則のある `visited-file-modtime' の側で対処する方が良い気
> もするんですよね。加えて、bookmark ファイルが無いときの
> `w3m-bookmark-file-modtime' の戻り値は、単に nil でいいんじゃな
> いでしょうか?  こんな感じかな:
>
> (snip)
>
> まあ強くこだわるような内容でもないのですけれども。;-)

なるほど、そこまで考えていませんでした。 たしかにそちらのほうが良いと思い
ます。 ということで、こちらの案で commit してみました。
w3m-visited-file-modtime, w3m-bookmark-file-modtime の ChangeLog がおかし
かったら修正お願いします。

;; w3m-bookmark.el は他にも w3m-bookmark-undo のあたりの挙動がどうもおか
;; しい気がするのですがうまく再現できません… :-(

--
青田