eskk.vim icon indicating copy to clipboard operation
eskk.vim copied to clipboard

複数の辞書ファイルを指定できるようにする

Open thinca opened this issue 14 years ago • 14 comments

辞書の優先度を指定できるといいかも。

thinca avatar Jul 08 '10 15:07 thinca

あーそういえばこれずっとTODOでした...どうもです。

tyru avatar Jul 08 '10 17:07 tyru

forkして実装してくれてる人(@voiさん)がいました。 https://github.com/voi/eskk.vim/commit/24f51e536db44ed0785eeb9bc2cf8195e8d564a5

@voi マージしちゃってもよろしいでしょうか?

tyru avatar Jun 10 '12 15:06 tyru

はい、かまいません。よろしくおねがいします。

voi avatar Jun 11 '12 14:06 voi

了解です。ありがとうございます。 マージがいつになるかはまだちょっと分からないですが、そのうちやろうと思います。

tyru avatar Jun 11 '12 16:06 tyru

この機能については私も興味があるので、私がマージを行います。

問題点がいくつか。

  1. ドキュメントがない
  2. skkserver 対応機能とコンフリクトする
  3. 外部インタフェースは互換性のためにこのままでもよいが、内部的にはシステム辞書と統合するべき

優先度の問題については、与えるリストの順番にすれば良いと思います。

skkserver 対応が master にマージされた後に手を付ける予定です。

Shougo avatar Jun 17 '14 11:06 Shougo

優先度の問題については、与えるリストの順番にすれば良いと思います。

ですね。

ぶっちゃけ本件はもうマージというより @voi さんの実装を参考にして実装する感じになると思います…

あと検索する辞書が増えると get_candidates() 関数のオーバーヘッドが気になってきます… それぞれの辞書を遅延して必要になった時に初めて検索するようにすればよさそうですが面倒そうです… get_candidates() のインターフェースも変わって来るので呼び出し側も変更しないといけなくなりそう。

まぁ検索を遅延させる件は後でissue上げておきます…

2014/06/17 20:57 "Shougo" [email protected]:

この機能については私も興味があるので、私がマージを行います。

問題点がいくつか。

ドキュメントがない server 機能とコンフリクトする 外部インタフェースは互換性のためにこのままでもよいが、内部的にはシステム辞書と統合するべき

優先度の問題については、与えるリストの順番にすれば良いと思います。

server が master にマージされた後に手を付ける予定です。

— Reply to this email directly or view it on GitHub.

tyru avatar Jun 26 '14 17:06 tyru

  1. ドキュメントがない
  2. skkserver 対応機能とコンフリクトする

今更で申し訳ないですが、ローカルでは最新とマージしていましたのでpushしました。 また、ドキュメントも追加しました。

よろしくお願いします。

voi avatar Jul 02 '14 13:07 voi

ありがとうございます。

Shougo avatar Jul 02 '14 19:07 Shougo

最新へのマージありがとうございます。

tyru avatar Jul 03 '14 21:07 tyru

age

tyru avatar Jan 11 '16 17:01 tyru

私のような馬鹿が意見して心苦しいのですが, 変数eskk#large_dictionaryの"sorted"という項目の値を適宜変更すれば, eskk.vimに複数辞書の順位付け参照をさせることができるという理解でよろしいでしょうか? というか,それだとうまくいきませんでした。

cmplstofB avatar Mar 16 '17 04:03 cmplstofB

いえ、g:eskk#large_dictionary の "sorted" の項目は辞書の各行がソート済みであることを示すためのものです。 まだこの件はマージされていませんので、複数辞書(=3つ以上の辞書ファイル)には対応していません。 現状対応しているのは、「システム辞書」と「ユーザ辞書」だけです。

tyru avatar Mar 16 '17 06:03 tyru

そうですか,ありがとうございます。そして勘違いしてすいません。 当分は人名辞書等を基本辞書に結合,内容を調整してつかっていきます。

cmplstofB avatar Mar 23 '17 06:03 cmplstofB

いえいえ、全然構わないです。 すみませんが、よろしくお願いします。

tyru avatar Mar 23 '17 09:03 tyru