Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use "switch to buffer" step from espuds (7 failed scenarios) #61

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tkf
Copy link
Contributor

@tkf tkf commented Feb 28, 2013

As of ecukes/espuds#11, "I switch to buffer" supports buffer
starts with spaces.

The step defined in espuds uses keyboard macro rather than calling
switch-to-buffer function directly and therefore more close to
the real use. Currently there are 7 failed scenarios.


switch-to-buffer を呼ぶ代わりにキーボードマクロで "C-x b BUFFER-NAME" を叩くようにテストを変更した結果、7つのシナリオが通らなくなりました。テストのバグなのか e2wm 自体のバグなのかわかりませんが、interactiveに試しても同様の問題が発生することから、 e2wm 自体にバグがあるんじゃないかと思っています。

すぐに解決出来るか分からないので、 issue 登録の意味も含めて PR しておきます。

As of ecukes/espuds#11, "I switch to buffer" supports buffer
starts with space.

The step defined in espuds uses keyboard macro rather than calling
`switch-to-buffer` function directly and therefore more close to
the real use.  Currently there are 7 failed scenarios.
@kiwanami
Copy link
Owner

ありがとうございます!
テスト結果を見ると、historyの内容が期待と違うという事と、twoでのバッファ切り替え不良という感じでしょうか。手元で使ってる限りはそんなに違和感を感じてなかったのですが、もうちょっと見てみます。

@tkf
Copy link
Contributor Author

tkf commented Feb 28, 2013

もうちょっと調べてみました。

  (setq e2wm:c-recordable-buffer-p
       (lambda (buf)
         (e2wm:aand (buffer-name buf)
                    (string-prefix-p "recordable" it t))))

を評価して C-x b recordable-1 RET としても、 history に表示されません。ただ、もう一度同じコマンドを実行すると表示されます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants