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

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

Open
thinca opened this issue Jul 8, 2010 · 14 comments
Open

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

thinca opened this issue Jul 8, 2010 · 14 comments
Assignees
Labels

Comments

@thinca
Copy link
Member

thinca commented Jul 8, 2010

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

@tyru
Copy link
Collaborator

tyru commented Jul 8, 2010

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

@tyru
Copy link
Collaborator

tyru commented Jun 10, 2012

forkして実装してくれてる人(@voiさん)がいました。
voi@24f51e5

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

@voi
Copy link

voi commented Jun 11, 2012

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

@tyru
Copy link
Collaborator

tyru commented Jun 11, 2012

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

@Shougo Shougo self-assigned this Jun 17, 2014
@Shougo
Copy link
Collaborator

Shougo commented Jun 17, 2014

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

問題点がいくつか。

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

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

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

@tyru
Copy link
Collaborator

tyru commented Jun 26, 2014

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

ですね。

ぶっちゃけ本件はもうマージというより @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.

@voi
Copy link

voi commented Jul 2, 2014

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

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

よろしくお願いします。

@Shougo
Copy link
Collaborator

Shougo commented Jul 2, 2014

ありがとうございます。

@tyru
Copy link
Collaborator

tyru commented Jul 3, 2014

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

@tyru
Copy link
Collaborator

tyru commented Jan 11, 2016

age

@cmplstofB
Copy link

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

@tyru
Copy link
Collaborator

tyru commented Mar 16, 2017

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

@cmplstofB
Copy link

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

@tyru
Copy link
Collaborator

tyru commented Mar 23, 2017

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

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

No branches or pull requests

5 participants