-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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.
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
Use RAII for group session iteration #33122
Conversation
* Use RAII for iterator management * Restyle * Fix typo * Fix naming a bit now that I made this a template * Make it clear that class member is initialized
PR #33122: Size comparison from 4431f18 to 2c45ebe Increases (71 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink)
Decreases (3 builds for efr32, linux)
Full report (72 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, stm32, telink)
|
Everything now is 1.3... We shouldn't cherry pick stuff in at this point. Will bring this up at the next SW standup (which will have to move times, sorry!) |
There are known issues in master right now for pairing issues, and failures, we should be encouraging (like all releases), master vs 1.3-branch. |
Based on comments from various members, it seems that people would like stable branches. While I agree that master is likely to have more fixes than a fixed branch in time, we cannot assume everyone uses master. |
…oject-chip#33122)" This reverts commit 29ed2f9. Signed-off-by: Adrian Gielniewski <[email protected]>
This cherrypicks #32970 into the 1.3 branch.