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 promise prefix in examples #17

Closed
wants to merge 3 commits into from
Closed

Conversation

conao3
Copy link
Contributor

@conao3 conao3 commented Nov 24, 2019

promise使い初めのころ、バッファ全体を評価して *scratch* で評価したかったのですが、prefixがなかったので安易に評価できず大変でした。。

また、promise:xml-retrieveと同じものが別名で定義されているのは本家の関数を使うようにしました。

これでより実践的な例を提供できると思います。

一応動作確認をしましたが、example16については #16 で報告したとおり手元では動いてません。。

@chuntaro
Copy link
Owner

jp(日本)、ja(日本語)だけど、APIが日本国内で意味のあるものなのでjpでいいのではないでしょうか?
後、exampleはMELPAで配布されないファイルで基本的には参照用でもあるので、厳密にpackage-lintに従う必要は無いと思ってます。
なので、プリフィックスを付けて関数名が長くなり過ぎて可読性が悪くなるよりは、-Qで別Emacsを起動してそれで動けば良しとしたいです。

@conao3
Copy link
Contributor Author

conao3 commented Nov 26, 2019

なるほど、分かりました!では、こちらは取り下げます。

@conao3 conao3 closed this Nov 26, 2019
@conao3 conao3 deleted the fix-example branch November 26, 2019 07:38
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