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

Bug: 空のエントリー 「foo //」や無効なエントリーなどがあるとパースに失敗してIMEごと落ちる #233

Closed
LumaKernel opened this issue Oct 24, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@LumaKernel
Copy link

https://github.com/tani/skk-jisyo-latex/blob/master/SKK-JISYO.latex.utf8 を読み込もうとしたら、次のような行を無効化しないと読み込めず、また、単に「Parse Error」と安全にエラーになる場合もあれば、IMEごと落ちるケースもありました。
IMEごと落ちた場合は、IMEの一覧から一度削除し、再度登録する、といったことをすれば正常に再起動してくれました。

;; 略
mathslash ///
;; 略
semicolon /;/
;; 略
lbrack /[/
;; 略
bullet //
smblkcircle //
;; 略
blockuphalf //

macSKKで動かないものをコメントアウトした完全版がこちらに一応あります: https://github.com/LumaKernel/dotfiles/blob/master/skk-dict/SKK-JISYO.latex.utf8.txt

@LumaKernel
Copy link
Author

すみません、これはちょっとだけ古いバージョンでやりました。またのちほど最新バージョンでも確認してみます…。

@mtgto mtgto added the bug Something isn't working label Oct 25, 2024
@mtgto
Copy link
Owner

mtgto commented Oct 25, 2024

報告ありがとうございます。
試してみたところ semicolon /;/ のエントリが正しく処理できず最新バージョンでもクラッシュするようでした。
クラッシュは想定外なので早急に修正します。
また空文字列への変換は「読み込み失敗」とは出さないように早急に修正します。

ddskkでの挙動を見てみると、空文字列への変換ができるのがよさそうなのですがすぐにできるかどうかわからないので確認してみます。

@mtgto
Copy link
Owner

mtgto commented Oct 27, 2024

v1.4.0でクラッシュする問題とエラーだと言われてしまう問題に対応しました。
v1.4.0では空文字列への変換候補は存在しないものとしていますが、今後問題ないことが確認できたら空文字列への変換にも対応する予定です。
問題の報告いただきありがとうございました。
https://github.com/mtgto/macSKK/releases/tag/1.4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants