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.

Sign up for GitHub

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

Jump to bottom

filterExtendedのmaskがうまく動作していないため修正 #2

Open

MizukiAsano-C wants to merge 1 commit into master from change-mask

MizukiAsano-C commented Nov 24, 2023

fork元にプルリクを発行しようと思いましたが、すでに発行されていてしかもガンスルーされてたのでこちらだけで。

元の状態だとこんな感じ

mask &= ~(mask & 0x1FFFFFFF);
mask & ~(mask & FFFF)  // 思考簡略化のためFFFFに短縮、代入処理を&に展開
mask & (~mask + 0000)  // 反転をカッコ内に代入　ド・モルガンの法則により&から+に
mask & ~mask + mask & 0000  // 掛け算はカッコ内の各々に行うことができる
0000  // A & ~Aは0、x & 0も0

&=から&を取るだけで大丈夫です。
一応maskについては以下のような仕組みらしいっす…

https://www.espressif.com/sites/default/files/documentation/esp32_technical_reference_manual_en.pdf
↑これの556ページ


          filterExtendedのmaskがうまく動作していないため試験的コミット

80fdb52

MizukiAsano-C requested a review from kouuta

November 24, 2023 00:44

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet