FINEARCHS

Results 259 comments of FINEARCHS

(そもそも通常の引用構文に包含されるような形ではなくなるなら全部そうではある)

私がmfm-jsにあんまり手を加えたくない一番大きな理由は「あんまり見てもらえないイメージが有る」なので、構文が確定できれば、あとはしゅいろさんがmfm-js側のPRをちゃんと見てくれれば問題なくはあります

> eslintのコーディング規約チェック通ってないですね 修正ありがとうございました。 > `Error: 55:2 error 'onMounted' is not defined no-undef`が出ているとなると手元でも正常に動作してないと思われるのですがPR提出前にこちら動作確認されていますか? 一応動作確認はしましたが、畳み条件を「実際の表示サイズに基づく」に設定した際の被引用ノートの表示の確認に漏れがあったかもしれません。改めて確認しました。

@kakkokari-gtyih 旧式を残すのは経過措置です。新方式2つにバグがあった時などのために念の為 将来的には消してもいいと思います。

できればLexical Windowにしたい 私自身関数内で外部変数を参照する使い方をよくするので

どのような環境で実行しましたか?私の手元の環境では再現できないのですが…

確認できました。開発版の方のパーサーの問題ですね。 本来ならパーサーエラーで弾くべきですが、普通は書かない文であり、放っておいても問題はなさそうな気もします。

ちなみに、どのような状況でこのバグを発見したかを聞いてもいいですか?

確認した所、`readToken()`の記号トークンのswitch caseの一部に変数`token`を更新しない場合があるものがあり、それが今回のバグに繋がっているようですね。 修正は容易そうですし、今後の安全性のために直したほうがよさそうですね。