ciniml / rust-dap

CMSIS-DAP Rust implementation
Apache License 2.0
88 stars 10 forks source link

PIOで減算オーバーフローが発生していたので修正 #38

Closed elfmimi closed 2 years ago

elfmimi commented 2 years ago

@tnishinaga さんの指摘で気付きました。

PIO で panic が発生する箇所があったので修正しました。

気付いていなかったのは、最適化ありの release ビルドしか試していなかったからです。

最適化によって 軽微な?整合性チェックの panic は、消えてしまうんですね。

マージ順は、後回しでよいと思います。

elfmimi commented 2 years ago

追記:

release では、デフォルトで overflow-checks = false なので、 overflow-checks = true とすれば release でも再現する。(最適化は関係しない)

再現テストは、openocd などで接続中に、SWD の接続を物理的に切断すればよい。

ciniml commented 2 years ago

ありがとうございます。効果が確認できたのでマージします。