[Message Prev][Message Next][Thread Prev][Thread Next][Message Index][Thread Index]

Re: [MD:6777] Re: [MEADOW DEVELOPMENT] Issue #MW136



gmailすか.

# Mew を CVS HEAD にしたら k-M, k-/ がうごかん;_; (M-x からなら動くけど)

> この修正に関するcommitはr3859およびr3858かと思いますが、
> 出来ましたら、r3859およびr3859のcommit log およびChangeLog.Meadowに
> "Fixed MW136" といったコメントを加えておいていただけるとよいと思います。

 たしかに逆リンクがないと背景がわかりにくいというのはあります. そ
してこれを意識すると, BTSへの登録を促す効果もありそうです.

> 特に(mw32_message_loop): Set `keyboard_handle' to the signaled
> state.  というコメントは実を記述していないので、

 これは常々思っています. できる限り最上位レベルで何をしようとして
いるのかを書こう思ってはいますが, 関数単位の記述を強制されるとなか
なか思うようにいかないですね.

> また一方、MW136を閉じる際のコメントにも
> "修正しました(r3858)。
> 同様の問題がtrunkにもありましたのでこちらもあわせて修正しました(r3859)。
> "
> のように書いておくのがよいと思います。

 これはあまり問題ないでしょうね.

> あと、GNU ChangeLogの書き方にこだわると、ChangeLogの記載はついつい
> 関数単位での修正内容になりがちなのが心配なので近年はあまり好きでなかったり
> するのですが、実際にはそのChangeSet全体で1つの意味を持つわけですから、

 同意.

> 必ずそうて、といいっているわけではないのですが、今回のようにcommitとfixが
> 直接対応する場合は特に、そうしたほうが、ChangeLogおよびissue/ticketの
> 価値がグンと上がるかと思います。

 話は72度くらい変わるのですが, scarab-mode ってないんですか? (w3m
使えばいいのか)

# 前にそういう話があったような.. とりあえずこれくらいから^^;

(defun meadow-scarab-refer-issue (&optional id)
  (interactive
   (list (read-from-minibuffer "Issue ID: " (thing-at-point 'word))))
  (w3m-goto-url
   (concat "http://www.meadowy.org/scarab/issues/id/" id)))

 あと, テキスト検索動いてなくないですか? > scarab

 w3m/Meaodw でテキストエリアの入力をしようと思ったらどうやら一時ファ
イルの名前が長すぎるらしくて失敗. Sigh..

-- 
ほりぐちきょうたろう