search

ueno / libskk

Japanese SKK input method library
GNU General Public License v3.0
78 stars 27 forks source link

Fix KZIK rules #35

Closed ohac closed 9 years ago

ohac commented 9 years ago

KZIKでうまく動かないところがあったので、修正しました。 AZIKも同じだと思うのですが、そちらはとりあえずそのままにしています。

coveralls commented 9 years ago

Coverage Status

Coverage remained the same at 82.16% when pulling 3bd59faf7e7f315cf323b3ece689fc04a570409d on ohac:master into 2ecb72dc9680fa760b18859dfd5553e59c141b25 on ueno:master.

ueno commented 9 years ago

あまり凝ったキーマップを使っていないので的外れだとは思いますが、 うまく動かなかった点と、default/default との差分の意図を commit ログに書いていただけると助かります。

あと、この辺を使って自動でバリデーションをできるようにすると今後の手間が減るかもしれません。 http://people.collabora.com/~pwith/walbottle/

ohac commented 9 years ago

すいません、default/defaultについては私が勘違いしているのかもしれません。 単に副作用があるといやだなぁと思って外してしまいました。

ちょっと教えていただきたいのですが、単にdefaultと書くと同じルールのdefault.jsonをincludeすることになるのでしょうか? あと、default/defaultと書けばdefault/keymap/default.jsonをincludeするのでしょうか?

もう少し理解してから書き直そうかと思います。

ueno commented 9 years ago

はい、そのように動作するはずです： https://github.com/ueno/libskk/blob/master/rules/README.rules#L67 変更点を見たら、いくつか外されたキーがあったので、何か理由があるのかと思った次第です。

ohac commented 9 years ago

よく調べてみると、最新のazik側はいろいろと修正されているようなので、これらの修正を移植したブランチを別途作成して、改めてプルリクエストを行いたいと思います。