kiwanami / emacs-window-manager

Customizable window manager for emacs
230 stars 28 forks source link

Fix: blank buffer appears when opening a file #46

Closed tkf closed 11 years ago

tkf commented 11 years ago

バージョン管理下にあるファイルを開くとメインのウィンドウに *e2wm:blank* が表示されてしまうバグの修正です。 バッファを開く時に vc-follow-link が、一度新しく開いたバッファを kill しているのが原因のようです。

この時、 e2wm:kill-buffer-hook 内では BUFFERS と WINS が両方とも nil になります。 新しく開いているバッファは履歴に未登録なのと(BUFFERS が nil)、ウィンドウにも表示されていない(WINS が nil)からです。この場合、 COND の中身は実行しないのが正しい選択だと思うので、 *e2wm:blank* を表示する前に、 WINS が nil かどうかのチェックを入れておきました。

tkf commented 11 years ago

ちなみに、 #42 と衝突するので #42 を先に pull する予定があれば、このコミットを cherry-pick するのが良いと思います: https://github.com/tkf/emacs-window-manager/commit/edff0d2f6b0f0776a21817331708834779a33c74

超トリビアルな違いですが...

tkf commented 11 years ago

書き忘れていましたが、影響を確認するには find-file-other-window などを使う必要があります。 find-file だと出てきた blank buffer が上書きされてしまうので、バグに気づかないと思います。

kiwanami commented 11 years ago

こっちのほうの取り込みが滞っていてすみません。 なんとかまとめて取り込みます!

tkf commented 11 years ago

Rebase したのでそのまま取り込めるかと思います。