Skip to content
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

Clean up group invoke requests #18683

Merged
merged 2 commits into from
May 23, 2022
Merged

Clean up group invoke requests #18683

merged 2 commits into from
May 23, 2022

Conversation

mkardous-silabs
Copy link
Contributor

Problem

  • function clean up was doing "tell the auto-ptr to forget the pointer just so we can manually delete it"

Change overview

  • Simplify function and let the auto-ptr do the clean up

Testing

  • automated tests to validate regression

@restyled-io restyled-io bot mentioned this pull request May 20, 2022
Closed
@github-actions
Copy link

github-actions bot commented May 20, 2022 •
edited
Loading

PR #18683: Size comparison from 9490a5f to dedb491

Decreases (1 build for telink)
platform target config section 9490a5f dedb491 change % change
telink light-switch-app tlsr9518adk80d (read/write) 782392 782368 -24 -0.0
text 553434 553412 -22 -0.0
Full report (28 builds for cc13x2_26x2, cyw30739, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 9490a5f dedb491 change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 666671 666671 0 0.0
(read/write) 184144 184144 0 0.0
.bss 73796 73796 0 0.0
.data 3404 3404 0 0.0
.rodata 99631 99631 0 0.0
.text 566812 566812 0 0.0
lock-ftd LP_CC2652R7 (read only) 675919 675919 0 0.0
(read/write) 166864 166864 0 0.0
.bss 72820 72820 0 0.0
.data 3236 3236 0 0.0
.rodata 94407 94407 0 0.0
.text 581028 581028 0 0.0
lock-mtd LP_CC2652R7 (read only) 625327 625327 0 0.0
(read/write) 145652 145652 0 0.0
.bss 68556 68556 0 0.0
.data 3236 3236 0 0.0
.rodata 94287 94287 0 0.0
.text 530548 530548 0 0.0
pump-app LP_CC2652R7 (read only) 659475 659475 0 0.0
(read/write) 184588 184588 0 0.0
.bss 73076 73076 0 0.0
.data 3268 3268 0 0.0
.rodata 79571 79571 0 0.0
.text 579424 579424 0 0.0
pump-controller-app LP_CC2652R7 (read only) 653963 653963 0 0.0
(read/write) 189908 189908 0 0.0
.bss 73140 73140 0 0.0
.data 3232 3232 0 0.0
.rodata 83675 83675 0 0.0
.text 569808 569808 0 0.0
shell LP_CC2652R7 (read only) 659750 659750 0 0.0
(read/write) 186624 186624 0 0.0
.bss 76156 76156 0 0.0
.data 3408 3408 0 0.0
.rodata 96598 96598 0 0.0
.text 562928 562928 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 624150 624150 0 0.0
.app_xip_area 527508 527508 0 0.0
.bss 79284 79284 0 0.0
.data 708 708 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 626962 626962 0 0.0
.app_xip_area 531776 531776 0 0.0
.bss 77860 77860 0 0.0
.data 672 672 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 570882 570882 0 0.0
.app_xip_area 465980 465980 0 0.0
.bss 87280 87280 0 0.0
.data 584 584 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
k32w light k32w061+release (read/write) 682488 682488 0 0.0
.bss 80352 80352 0 0.0
.data 2016 2016 0 0.0
.text 598416 598416 0 0.0
lock k32w061+release (read/write) 728636 728636 0 0.0
.bss 80784 80784 0 0.0
.data 1976 1976 0 0.0
.text 644172 644172 0 0.0
linux all-clusters-app debug (read only) 2751841 2751841 0 0.0
(read/write) 176320 176320 0 0.0
.bss 85408 85408 0 0.0
.data 2064 2064 0 0.0
.data.rel.ro 82680 82680 0 0.0
.dynamic 608 608 0 0.0
.got 4496 4496 0 0.0
.init 27 27 0 0.0
.init_array 1016 1016 0 0.0
.rodata 240733 240733 0 0.0
.text 2336722 2336722 0 0.0
bridge-app debug+rpc (read only) 2035481 2035481 0 0.0
(read/write) 148024 148024 0 0.0
.bss 73120 73120 0 0.0
.data 3936 3936 0 0.0
.data.rel.ro 65384 65384 0 0.0
.dynamic 592 592 0 0.0
.got 4272 4272 0 0.0
.init 27 27 0 0.0
.init_array 688 688 0 0.0
.rodata 169065 169065 0 0.0
.text 1709298 1709298 0 0.0
chip-tool debug (read only) 9451541 9451541 0 0.0
(read/write) 579192 579192 0 0.0
.bss 23936 23936 0 0.0
.data 1152 1152 0 0.0
.data.rel.ro 547800 547800 0 0.0
.dynamic 624 624 0 0.0
.got 5000 5000 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 481821 481821 0 0.0
.text 7642789 7642789 0 0.0
lighting-app debug+rpc (read only) 2327273 2327273 0 0.0
(read/write) 153440 153440 0 0.0
.bss 74912 74912 0 0.0
.data 2048 2048 0 0.0
.data.rel.ro 70728 70728 0 0.0
.dynamic 608 608 0 0.0
.got 4344 4344 0 0.0
.init 27 27 0 0.0
.init_array 792 792 0 0.0
.rodata 188073 188073 0 0.0
.text 1972850 1972850 0 0.0
lock-app debug (read only) 2240769 2240769 0 0.0
(read/write) 148152 148152 0 0.0
.bss 73600 73600 0 0.0
.data 1568 1568 0 0.0
.data.rel.ro 67272 67272 0 0.0
.dynamic 592 592 0 0.0
.got 4336 4336 0 0.0
.init 27 27 0 0.0
.init_array 752 752 0 0.0
.rodata 198713 198713 0 0.0
.text 1882098 1882098 0 0.0
ota-provider-app debug (read only) 2065977 2065977 0 0.0
(read/write) 141232 141232 0 0.0
.bss 73056 73056 0 0.0
.data 1768 1768 0 0.0
.data.rel.ro 60600 60600 0 0.0
.dynamic 608 608 0 0.0
.got 4504 4504 0 0.0
.init 27 27 0 0.0
.init_array 648 648 0 0.0
.rodata 179872 179872 0 0.0
.text 1727538 1727538 0 0.0
ota-requestor-app debug (read only) 2094929 2094929 0 0.0
(read/write) 144040 144040 0 0.0
.bss 73728 73728 0 0.0
.data 1992 1992 0 0.0
.data.rel.ro 62664 62664 0 0.0
.dynamic 592 592 0 0.0
.got 4344 4344 0 0.0
.init 27 27 0 0.0
.init_array 672 672 0 0.0
.rodata 175840 175840 0 0.0
.text 1759058 1759058 0 0.0
shell debug (read only) 2568065 2568065 0 0.0
(read/write) 200344 200344 0 0.0
.bss 116328 116328 0 0.0
.data 1376 1376 0 0.0
.data.rel.ro 76880 76880 0 0.0
.dynamic 608 608 0 0.0
.got 4192 4192 0 0.0
.init 27 27 0 0.0
.init_array 936 936 0 0.0
.rodata 221746 221746 0 0.0
.text 2185314 2185314 0 0.0
tv-app debug (read only) 2856913 2856913 0 0.0
(read/write) 279072 279072 0 0.0
.bss 191032 191032 0 0.0
.data 4672 4672 0 0.0
.data.rel.ro 77120 77120 0 0.0
.dynamic 592 592 0 0.0
.got 4720 4720 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 221137 221137 0 0.0
.text 2453602 2453602 0 0.0
tv-casting-app debug (read only) 5424873 5424873 0 0.0
(read/write) 225344 225344 0 0.0
.bss 78616 78616 0 0.0
.data 2400 2400 0 0.0
.data.rel.ro 138120 138120 0 0.0
.dynamic 608 608 0 0.0
.got 4728 4728 0 0.0
.init 27 27 0 0.0
.init_array 864 864 0 0.0
.rodata 339937 339937 0 0.0
.text 4725442 4725442 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2418936 2418936 0 0.0
.bss 202796 202796 0 0.0
.data 5872 5872 0 0.0
.text 1381580 1381580 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1180715 1180715 0 0.0
bss 138528 138528 0 0.0
rodata 152532 152532 0 0.0
text 810808 810808 0 0.0
p6 all-clusters-app default (read/write) 2538336 2538336 0 0.0
.bss 136328 136328 0 0.0
.data 2808 2808 0 0.0
.text 1496600 1496600 0 0.0
light-app default (read/write) 2424552 2424552 0 0.0
.bss 129616 129616 0 0.0
.data 2608 2608 0 0.0
.text 1382816 1382816 0 0.0
lock-app default (read/write) 2435256 2435256 0 0.0
.bss 129432 129432 0 0.0
.data 2568 2568 0 0.0
.text 1393520 1393520 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 782392 782368 -24 -0.0
bss 70744 70744 0 0.0
noinit 40416 40416 0 0.0
text 553434 553412 -22 -0.0
lighting-app tlsr9518adk80d (read/write) 802412 802412 0 0.0
bss 71000 71000 0 0.0
noinit 40416 40416 0 0.0
text 570146 570146 0 0.0

@github-actions
Copy link

github-actions bot commented May 20, 2022 •
edited
Loading

PR #18683: Size comparison from b4bbe98 to 20d9561

Decreases (1 build for telink)
platform target config section b4bbe98 20d9561 change % change
telink light-switch-app tlsr9518adk80d (read/write) 782400 782376 -24 -0.0
text 553442 553420 -22 -0.0
Full report (37 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section b4bbe98 20d9561 change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 668311 668311 0 0.0
(read/write) 182520 182520 0 0.0
.bss 73812 73812 0 0.0
.data 3404 3404 0 0.0
.rodata 99911 99911 0 0.0
.text 568176 568176 0 0.0
lock-ftd LP_CC2652R7 (read only) 675919 675919 0 0.0
(read/write) 166864 166864 0 0.0
.bss 72820 72820 0 0.0
.data 3236 3236 0 0.0
.rodata 94407 94407 0 0.0
.text 581028 581028 0 0.0
lock-mtd LP_CC2652R7 (read only) 625327 625327 0 0.0
(read/write) 145652 145652 0 0.0
.bss 68556 68556 0 0.0
.data 3236 3236 0 0.0
.rodata 94287 94287 0 0.0
.text 530548 530548 0 0.0
pump-app LP_CC2652R7 (read only) 675859 675859 0 0.0
(read/write) 168228 168228 0 0.0
.bss 73100 73100 0 0.0
.data 3272 3272 0 0.0
.rodata 88899 88899 0 0.0
.text 586476 586476 0 0.0
pump-controller-app LP_CC2652R7 (read only) 653963 653963 0 0.0
(read/write) 189908 189908 0 0.0
.bss 73140 73140 0 0.0
.data 3232 3232 0 0.0
.rodata 83675 83675 0 0.0
.text 569808 569808 0 0.0
shell LP_CC2652R7 (read only) 661414 661414 0 0.0
(read/write) 184976 184976 0 0.0
.bss 76172 76172 0 0.0
.data 3408 3408 0 0.0
.rodata 96878 96878 0 0.0
.text 564308 564308 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 624150 624150 0 0.0
.app_xip_area 527508 527508 0 0.0
.bss 79284 79284 0 0.0
.data 708 708 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 626962 626962 0 0.0
.app_xip_area 531776 531776 0 0.0
.bss 77860 77860 0 0.0
.data 672 672 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 570882 570882 0 0.0
.app_xip_area 465980 465980 0 0.0
.bss 87280 87280 0 0.0
.data 584 584 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 915988 915988 0 0.0
(read/write) 133372 133372 0 0.0
.bss 131312 131312 0 0.0
.data 2060 2060 0 0.0
.text 915980 915980 0 0.0
BRD4161A+rpc (read only) 950160 950160 0 0.0
(read/write) 150056 150056 0 0.0
.bss 147792 147792 0 0.0
.data 2264 2264 0 0.0
.text 950152 950152 0 0.0
BRD4161A+rs911x (read only) 790548 790548 0 0.0
(read/write) 129640 129640 0 0.0
.bss 127572 127572 0 0.0
.data 2068 2068 0 0.0
.text 790540 790540 0 0.0
lock-app BRD4161A+wf200 (read only) 946760 946760 0 0.0
(read/write) 124124 124124 0 0.0
.bss 122100 122100 0 0.0
.data 2024 2024 0 0.0
.text 946752 946752 0 0.0
window-app BRD4161A (read only) 897260 897260 0 0.0
(read/write) 133432 133432 0 0.0
.bss 131384 131384 0 0.0
.data 2048 2048 0 0.0
.text 897252 897252 0 0.0
esp32 all-clusters-app c3devkit (read only) 1002630 1002630 0 0.0
(read/write) 1478058 1478058 0 0.0
.dram0.bss 68376 68376 0 0.0
.dram0.data 14624 14624 0 0.0
.flash.rodata 209624 209624 0 0.0
.flash.text 1002630 1002630 0 0.0
.iram0.text 62954 62954 0 0.0
m5stack (read only) 1057411 1057411 0 0.0
(read/write) 480064 480064 0 0.0
.dram0.bss 73896 73896 0 0.0
.dram0.data 34200 34200 0 0.0
.flash.rodata 239972 239972 0 0.0
.flash.text 1052027 1052027 0 0.0
.iram0.text 123267 123267 0 0.0
k32w light k32w061+release (read/write) 682488 682488 0 0.0
.bss 80352 80352 0 0.0
.data 2016 2016 0 0.0
.text 598416 598416 0 0.0
lock k32w061+release (read/write) 728636 728636 0 0.0
.bss 80784 80784 0 0.0
.data 1976 1976 0 0.0
.text 644172 644172 0 0.0
linux all-clusters-app debug (read only) 2758633 2758633 0 0.0
(read/write) 176384 176384 0 0.0
.bss 85440 85440 0 0.0
.data 2064 2064 0 0.0
.data.rel.ro 82712 82712 0 0.0
.dynamic 608 608 0 0.0
.got 4496 4496 0 0.0
.init 27 27 0 0.0
.init_array 1016 1016 0 0.0
.rodata 241373 241373 0 0.0
.text 2342706 2342706 0 0.0
bridge-app debug+rpc (read only) 2035481 2035481 0 0.0
(read/write) 148024 148024 0 0.0
.bss 73120 73120 0 0.0
.data 3936 3936 0 0.0
.data.rel.ro 65384 65384 0 0.0
.dynamic 592 592 0 0.0
.got 4272 4272 0 0.0
.init 27 27 0 0.0
.init_array 688 688 0 0.0
.rodata 169065 169065 0 0.0
.text 1709298 1709298 0 0.0
chip-tool debug (read only) 9451797 9451797 0 0.0
(read/write) 579192 579192 0 0.0
.bss 23936 23936 0 0.0
.data 1152 1152 0 0.0
.data.rel.ro 547800 547800 0 0.0
.dynamic 624 624 0 0.0
.got 5000 5000 0 0.0
.init 27 27 0 0.0
.init_array 656 656 0 0.0
.rodata 481821 481821 0 0.0
.text 7643045 7643045 0 0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 9199652 9199652 0 0.0
(read/write) 645329 645329 0 0.0
.bss 42225 42225 0 0.0
.data 1192 1192 0 0.0
.data.rel.ro 583088 583088 0 0.0
.dynamic 560 560 0 0.0
.got 14984 14984 0 0.0
.init 24 24 0 0.0
.init_array 184 184 0 0.0
.rodata 446260 446260 0 0.0
.text 7291364 7291364 0 0.0
lighting-app debug+rpc (read only) 2327273 2327273 0 0.0
(read/write) 153440 153440 0 0.0
.bss 74912 74912 0 0.0
.data 2048 2048 0 0.0
.data.rel.ro 70728 70728 0 0.0
.dynamic 608 608 0 0.0
.got 4344 4344 0 0.0
.init 27 27 0 0.0
.init_array 792 792 0 0.0
.rodata 188073 188073 0 0.0
.text 1972850 1972850 0 0.0
lock-app debug (read only) 2240769 2240769 0 0.0
(read/write) 148152 148152 0 0.0
.bss 73600 73600 0 0.0
.data 1568 1568 0 0.0
.data.rel.ro 67272 67272 0 0.0
.dynamic 592 592 0 0.0
.got 4336 4336 0 0.0
.init 27 27 0 0.0
.init_array 752 752 0 0.0
.rodata 198713 198713 0 0.0
.text 1882098 1882098 0 0.0
ota-provider-app debug (read only) 2065977 2065977 0 0.0
(read/write) 141232 141232 0 0.0
.bss 73056 73056 0 0.0
.data 1768 1768 0 0.0
.data.rel.ro 60600 60600 0 0.0
.dynamic 608 608 0 0.0
.got 4504 4504 0 0.0
.init 27 27 0 0.0
.init_array 648 648 0 0.0
.rodata 179872 179872 0 0.0
.text 1727538 1727538 0 0.0
ota-requestor-app debug (read only) 2094929 2094929 0 0.0
(read/write) 144040 144040 0 0.0
.bss 73728 73728 0 0.0
.data 1992 1992 0 0.0
.data.rel.ro 62664 62664 0 0.0
.dynamic 592 592 0 0.0
.got 4344 4344 0 0.0
.init 27 27 0 0.0
.init_array 672 672 0 0.0
.rodata 175840 175840 0 0.0
.text 1759058 1759058 0 0.0
shell debug (read only) 2574841 2574841 0 0.0
(read/write) 200408 200408 0 0.0
.bss 116360 116360 0 0.0
.data 1376 1376 0 0.0
.data.rel.ro 76912 76912 0 0.0
.dynamic 608 608 0 0.0
.got 4192 4192 0 0.0
.init 27 27 0 0.0
.init_array 936 936 0 0.0
.rodata 222354 222354 0 0.0
.text 2191314 2191314 0 0.0
thermostat-no-ble arm64 (read only) 2356292 2356292 0 0.0
(read/write) 176609 176609 0 0.0
.bss 87873 87873 0 0.0
.data 1520 1520 0 0.0
.data.rel.ro 79416 79416 0 0.0
.dynamic 560 560 0 0.0
.got 4768 4768 0 0.0
.init 24 24 0 0.0
.init_array 376 376 0 0.0
.rodata 147556 147556 0 0.0
.text 1979760 1979760 0 0.0
tv-app debug (read only) 2856913 2856913 0 0.0
(read/write) 279072 279072 0 0.0
.bss 191032 191032 0 0.0
.data 4672 4672 0 0.0
.data.rel.ro 77120 77120 0 0.0
.dynamic 592 592 0 0.0
.got 4720 4720 0 0.0
.init 27 27 0 0.0
.init_array 928 928 0 0.0
.rodata 221137 221137 0 0.0
.text 2453602 2453602 0 0.0
tv-casting-app debug (read only) 5425049 5425049 0 0.0
(read/write) 225344 225344 0 0.0
.bss 78616 78616 0 0.0
.data 2400 2400 0 0.0
.data.rel.ro 138120 138120 0 0.0
.dynamic 608 608 0 0.0
.got 4728 4728 0 0.0
.init 27 27 0 0.0
.init_array 864 864 0 0.0
.rodata 339937 339937 0 0.0
.text 4725618 4725618 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2418936 2418936 0 0.0
.bss 202796 202796 0 0.0
.data 5872 5872 0 0.0
.text 1381580 1381580 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1182367 1182367 0 0.0
bss 138540 138540 0 0.0
rodata 152808 152808 0 0.0
text 812176 812176 0 0.0
p6 all-clusters-app default (read/write) 2540520 2540520 0 0.0
.bss 136344 136344 0 0.0
.data 2808 2808 0 0.0
.text 1498784 1498784 0 0.0
light-app default (read/write) 2424552 2424552 0 0.0
.bss 129616 129616 0 0.0
.data 2608 2608 0 0.0
.text 1382816 1382816 0 0.0
lock-app default (read/write) 2435256 2435256 0 0.0
.bss 129432 129432 0 0.0
.data 2568 2568 0 0.0
.text 1393520 1393520 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 782400 782376 -24 -0.0
bss 70744 70744 0 0.0
noinit 40416 40416 0 0.0
text 553442 553420 -22 -0.0
lighting-app tlsr9518adk80d (read/write) 802420 802420 0 0.0
bss 71000 71000 0 0.0
noinit 40416 40416 0 0.0
text 570154 570154 0 0.0

@bzbarsky-apple bzbarsky-apple merged commit 0d5c917 into project-chip:master May 23, 2022
@mkardous-silabs mkardous-silabs deleted the feature/code_cleanup branch May 30, 2022 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andy31415 andy31415 andy31415 approved these changes

@anush-apple anush-apple Awaiting requested review from anush-apple

@arkq arkq Awaiting requested review from arkq

@Byungjoo-Lee Byungjoo-Lee Awaiting requested review from Byungjoo-Lee

@carol-apple carol-apple Awaiting requested review from carol-apple

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@Damian-Nordic Damian-Nordic Awaiting requested review from Damian-Nordic

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@franck-apple franck-apple Awaiting requested review from franck-apple

@gjc13 gjc13 Awaiting requested review from gjc13

@hawk248 hawk248 Awaiting requested review from hawk248

@harsha-rajendran harsha-rajendran Awaiting requested review from harsha-rajendran

@isiu-apple isiu-apple Awaiting requested review from isiu-apple

@jelderton jelderton Awaiting requested review from jelderton

@jepenven-silabs jepenven-silabs Awaiting requested review from jepenven-silabs

@jmartinez-silabs jmartinez-silabs Awaiting requested review from jmartinez-silabs

@jtung-apple jtung-apple Awaiting requested review from jtung-apple

@kghost kghost Awaiting requested review from kghost

@kpschoedel kpschoedel Awaiting requested review from kpschoedel

@lazarkov lazarkov Awaiting requested review from lazarkov

@LuDuda LuDuda Awaiting requested review from LuDuda

@mlepage-google mlepage-google Awaiting requested review from mlepage-google

@msandstedt msandstedt Awaiting requested review from msandstedt

@mspang mspang Awaiting requested review from mspang

@rgoliver rgoliver Awaiting requested review from rgoliver

@robszewczyk robszewczyk Awaiting requested review from robszewczyk

@sagar-apple sagar-apple Awaiting requested review from sagar-apple

@saurabhst saurabhst Awaiting requested review from saurabhst

@selissia selissia Awaiting requested review from selissia

@tecimovic tecimovic Awaiting requested review from tecimovic

@turon turon Awaiting requested review from turon

@vijs vijs Awaiting requested review from vijs

@vivien-apple vivien-apple Awaiting requested review from vivien-apple

@wbschiller wbschiller Awaiting requested review from wbschiller

@bzbarsky-apple bzbarsky-apple Awaiting requested review from bzbarsky-apple

@woody-apple woody-apple Awaiting requested review from woody-apple

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@yufengwangca yufengwangca Awaiting requested review from yufengwangca

@mrjerryjohns mrjerryjohns Awaiting requested review from mrjerryjohns

@tcarmelveilleux tcarmelveilleux Awaiting requested review from tcarmelveilleux

Assignees
No one assigned
Labels
controller review - approved
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@mkardous-silabs @tcarmelveilleux @andy31415 @mrjerryjohns @bzbarsky-apple @restyled-commits