skk-dev / ddskk

Daredevil SKK (Simple Kana to Kanji conversion program)

GNU General Public License v3.0

195 stars 42 forks source link

Closed tkita closed 4 years ago

tkita commented 4 years ago

117 手元では、 warning 表示されず、make install 後も普通に使えている様子です。

tkita commented 4 years ago

落ち着いて考えてみると、これを merge すると、元々の議論に戻っちゃうんですよね。「各ファイルがskk.elをrequireする」＝「インストーラがskk.elをrequireしちゃっている」やはり、どの関数をどのファイルに配置するのかを慎重に見直すのが本質なんですよね．．．

tkita commented 4 years ago

なるほど、MELPA 版で warning が消えるのですね。

conao3 commented 4 years ago

なるほど、MELPA 版で warning が消えるのですね。

それは分からないですね。 MELPAから降ったときにどうなるのかは実際にやってみないと分からないので。。

tkita commented 4 years ago

ありがとうございます。merge しちゃいました。次は、無用な autoload マジックコメントを削除します。

tats commented 4 years ago

まだskk.elにskk-autoloadsのrequireが残っていますので、skk-autoloads 無しのMELPAでの警告はなくならないのでは?

skk.el内で直接必要な宣言をすれば解消するものと思います。

tkita commented 4 years ago

了解です。