t-sin / inquisitor

Encoding/end-of-line detection and external-format abstraction for Common Lisp
34 stars 3 forks source link

Muffle warning #39

Closed cxxxr closed 7 years ago

cxxxr commented 7 years ago

警告が出ていたところを出ないようにしました それからmetabang-bindに関する警告も出ていて、無くても問題無く書けたので消しました

話は少しずれますがpullreqがマージされた部分で筋が悪いと思ったところがあれば残さずに消してくれていいです

t-sin commented 7 years ago

あ。diff見ようとしてマージボタン押してしまった…。

t-sin commented 7 years ago

Diff確認しましたが LGTM でした! ありがとうございます!

話は少しずれますがpullreqがマージされた部分で筋が悪いと思ったところがあれば残さずに消してくれていいです

了解です!

t-sin commented 7 years ago

あ、ちなみに手元の環境ではマージ前のコードで警告を確認できなかったのですが、どのようにロードしたときに警告でました? (ぼくは単に(ql:quickload :inquisitor)しました)

cxxxr commented 7 years ago

(ql:quickload :inquisitor)ではコンパイラの出力が隠されるので (asdf:load-system :inquisitor)でロードしました slimeのC-c C-kcompile-fileでファイル単位でのコンパイルでも確認出来るはずです

t-sin commented 7 years ago

おおお、なるほど。こうしてみると結構警告でてました。ありがとうございます!