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

CASEClient: Remove unused defines #17726

Merged
merged 1 commit into from
Apr 26, 2022
Merged

CASEClient: Remove unused defines #17726

merged 1 commit into from
Apr 26, 2022

Conversation

kghost
Copy link
Contributor

@kghost kghost commented Apr 26, 2022

Problem

Some unused code forget to remove

Change overview

CASEClient: Remove unused defines

Testing

Not needed

@boring-cyborg boring-cyborg bot added the app label Apr 26, 2022
@github-actions
Copy link

github-actions bot commented Apr 26, 2022 •
edited
Loading

PR #17726: Size comparison from 3a7f03a to 5452f1e

Full report (8 builds for cyw30739, k32w, mbed, nrfconnect, telink)
platform target config section 3a7f03a 5452f1e change % change
cyw30739 light cyw930739m2evb_01 (read/write) 623558 623558 0 0.0
.app_xip_area 526436 526436 0 0.0
.bss 79780 79780 0 0.0
.data 692 692 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 618514 618514 0 0.0
.app_xip_area 522864 522864 0 0.0
.bss 78340 78340 0 0.0
.data 656 656 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 571230 571230 0 0.0
.app_xip_area 465944 465944 0 0.0
.bss 87680 87680 0 0.0
.data 568 568 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
k32w light k32w061+release (read/write) 688820 688820 0 0.0
.bss 81112 81112 0 0.0
.data 2000 2000 0 0.0
.text 599908 599908 0 0.0
lock k32w061+release (read/write) 729712 729712 0 0.0
.bss 81520 81520 0 0.0
.data 1960 1960 0 0.0
.text 640432 640432 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2411676 2411676 0 0.0
.bss 205660 205660 0 0.0
.data 5848 5848 0 0.0
.text 1374276 1374276 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1177475 1177475 0 0.0
bss 141784 141784 0 0.0
rodata 150100 150100 0 0.0
text 806956 806956 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 808020 808020 0 0.0
bss 75216 75216 0 0.0
noinit 40416 40416 0 0.0
text 571792 571792 0 0.0

@github-actions
Copy link

github-actions bot commented Apr 26, 2022 •
edited
Loading

PR #17726: Size comparison from 3a7f03a to b88f0fa

Full report (32 builds for cc13x2_26x2, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, p6, telink)
platform target config section 3a7f03a b88f0fa change % change
cc13x2_26x2 all-clusters-app LP_CC2652R7 (read only) 692195 692195 0 0.0
(read/write) 159828 159828 0 0.0
.bss 75008 75008 0 0.0
.data 3396 3396 0 0.0
.rodata 105995 105995 0 0.0
.text 585720 585720 0 0.0
lock-ftd LP_CC2652R7 (read only) 678719 678719 0 0.0
(read/write) 164616 164616 0 0.0
.bss 73376 73376 0 0.0
.data 3220 3220 0 0.0
.rodata 98831 98831 0 0.0
.text 579404 579404 0 0.0
lock-mtd LP_CC2652R7 (read only) 627463 627463 0 0.0
(read/write) 146172 146172 0 0.0
.bss 69096 69096 0 0.0
.data 3220 3220 0 0.0
.rodata 98711 98711 0 0.0
.text 528260 528260 0 0.0
pump-app LP_CC2652R7 (read only) 651195 651195 0 0.0
(read/write) 151684 151684 0 0.0
.bss 73552 73552 0 0.0
.data 3252 3252 0 0.0
.rodata 75723 75723 0 0.0
.text 574984 574984 0 0.0
pump-controller-app LP_CC2652R7 (read only) 644247 644247 0 0.0
(read/write) 151440 151440 0 0.0
.bss 73600 73600 0 0.0
.data 3216 3216 0 0.0
.rodata 78839 78839 0 0.0
.text 564920 564920 0 0.0
cyw30739 light cyw930739m2evb_01 (read/write) 623558 623558 0 0.0
.app_xip_area 526436 526436 0 0.0
.bss 79780 79780 0 0.0
.data 692 692 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
lock cyw930739m2evb_01 (read/write) 618514 618514 0 0.0
.app_xip_area 522864 522864 0 0.0
.bss 78340 78340 0 0.0
.data 656 656 0 0.0
.rodata 0 0 0 0.0
.text 0 0 0 0.0
ota-requestor-no-progress-logging cyw930739m2evb_01 (read/write) 571230 571230 0 0.0
.app_xip_area 465944 465944 0 0.0
.bss 87680 87680 0 0.0
.data 568 568 0 0.0
.rodata 0 0 0 0.0
.text 112 112 0 0.0
efr32 lighting-app BRD4161A (read only) 907776 907776 0 0.0
(read/write) 134912 134912 0 0.0
.bss 132864 132864 0 0.0
.data 2044 2044 0 0.0
.text 907768 907768 0 0.0
BRD4161A+rpc (read only) 942192 942192 0 0.0
(read/write) 151592 151592 0 0.0
.bss 149344 149344 0 0.0
.data 2248 2248 0 0.0
.text 942184 942184 0 0.0
window-app BRD4161A (read only) 845160 845160 0 0.0
(read/write) 133000 133000 0 0.0
.bss 131040 131040 0 0.0
.data 1956 1956 0 0.0
.text 845152 845152 0 0.0
esp32 all-clusters-app c3devkit (read only) 999482 999482 0 0.0
(read/write) 1473666 1473666 0 0.0
.dram0.bss 68176 68176 0 0.0
.dram0.data 14428 14428 0 0.0
.flash.rodata 206616 206616 0 0.0
.flash.text 999482 999482 0 0.0
.iram0.text 62020 62020 0 0.0
m5stack (read only) 1053791 1053791 0 0.0
(read/write) 475656 475656 0 0.0
.dram0.bss 73696 73696 0 0.0
.dram0.data 34168 34168 0 0.0
.flash.rodata 235956 235956 0 0.0
.flash.text 1048407 1048407 0 0.0
.iram0.text 123107 123107 0 0.0
k32w light k32w061+release (read/write) 688820 688820 0 0.0
.bss 81112 81112 0 0.0
.data 2000 2000 0 0.0
.text 599908 599908 0 0.0
lock k32w061+release (read/write) 729712 729712 0 0.0
.bss 81520 81520 0 0.0
.data 1960 1960 0 0.0
.text 640432 640432 0 0.0
linux all-clusters-app debug (read only) 2735377 2735377 0 0.0
(read/write) 172872 172872 0 0.0
.bss 83168 83168 0 0.0
.data 1984 1984 0 0.0
.data.rel.ro 81640 81640 0 0.0
.dynamic 608 608 0 0.0
.got 4464 4464 0 0.0
.init 27 27 0 0.0
.init_array 992 992 0 0.0
.rodata 233861 233861 0 0.0
.text 2326674 2326674 0 0.0
bridge-app debug+rpc (read only) 1891361 1891361 0 0.0
(read/write) 120304 120304 0 0.0
.bss 71168 71168 0 0.0
.data 3424 3424 0 0.0
.data.rel.ro 40392 40392 0 0.0
.dynamic 592 592 0 0.0
.got 4032 4032 0 0.0
.init 27 27 0 0.0
.init_array 680 680 0 0.0
.rodata 159633 159633 0 0.0
.text 1608946 1608946 0 0.0
chip-tool debug (read only) 10945293 10945293 0 0.0
(read/write) 375792 375792 0 0.0
.bss 22592 22592 0 0.0
.data 1136 1136 0 0.0
.data.rel.ro 345808 345808 0 0.0
.dynamic 624 624 0 0.0
.got 4952 4952 0 0.0
.init 27 27 0 0.0
.init_array 648 648 0 0.0
.rodata 547381 547381 0 0.0
.text 9557957 9557957 0 0.0
chip-tool-no-interactive-ipv6only arm64 (read only) 10556732 10556732 0 0.0
(read/write) 497697 497697 0 0.0
.bss 40897 40897 0 0.0
.data 1192 1192 0 0.0
.data.rel.ro 393104 393104 0 0.0
.dynamic 560 560 0 0.0
.got 58680 58680 0 0.0
.init 24 24 0 0.0
.init_array 184 184 0 0.0
.rodata 521228 521228 0 0.0
.text 8917652 8917652 0 0.0
lighting-app debug+rpc (read only) 2326817 2326817 0 0.0
(read/write) 151256 151256 0 0.0
.bss 73216 73216 0 0.0
.data 1984 1984 0 0.0
.data.rel.ro 70328 70328 0 0.0
.dynamic 608 608 0 0.0
.got 4320 4320 0 0.0
.init 27 27 0 0.0
.init_array 784 784 0 0.0
.rodata 183729 183729 0 0.0
.text 1976450 1976450 0 0.0
lock-app debug (read only) 2207169 2207169 0 0.0
(read/write) 145640 145640 0 0.0
.bss 71808 71808 0 0.0
.data 1504 1504 0 0.0
.data.rel.ro 66664 66664 0 0.0
.dynamic 592 592 0 0.0
.got 4312 4312 0 0.0
.init 27 27 0 0.0
.init_array 736 736 0 0.0
.rodata 192193 192193 0 0.0
.text 1854978 1854978 0 0.0
ota-provider-app debug (read only) 2059889 2059889 0 0.0
(read/write) 138888 138888 0 0.0
.bss 71232 71232 0 0.0
.data 1672 1672 0 0.0
.data.rel.ro 60216 60216 0 0.0
.dynamic 608 608 0 0.0
.got 4480 4480 0 0.0
.init 27 27 0 0.0
.init_array 640 640 0 0.0
.rodata 174235 174235 0 0.0
.text 1728146 1728146 0 0.0
ota-requestor-app debug (read only) 2091137 2091137 0 0.0
(read/write) 141696 141696 0 0.0
.bss 71872 71872 0 0.0
.data 1928 1928 0 0.0
.data.rel.ro 62264 62264 0 0.0
.dynamic 592 592 0 0.0
.got 4320 4320 0 0.0
.init 27 27 0 0.0
.init_array 664 664 0 0.0
.rodata 170588 170588 0 0.0
.text 1761554 1761554 0 0.0
shell debug (read only) 2561209 2561209 0 0.0
(read/write) 196840 196840 0 0.0
.bss 113768 113768 0 0.0
.data 1360 1360 0 0.0
.data.rel.ro 75992 75992 0 0.0
.dynamic 592 592 0 0.0
.got 4184 4184 0 0.0
.init 27 27 0 0.0
.init_array 920 920 0 0.0
.rodata 215602 215602 0 0.0
.text 2183698 2183698 0 0.0
thermostat-no-ble arm64 (read only) 2368284 2368284 0 0.0
(read/write) 174369 174369 0 0.0
.bss 86097 86097 0 0.0
.data 1488 1488 0 0.0
.data.rel.ro 79008 79008 0 0.0
.dynamic 560 560 0 0.0
.got 4744 4744 0 0.0
.init 24 24 0 0.0
.init_array 368 368 0 0.0
.rodata 145500 145500 0 0.0
.text 1993680 1993680 0 0.0
tv-app debug (read only) 2845017 2845017 0 0.0
(read/write) 276536 276536 0 0.0
.bss 189112 189112 0 0.0
.data 4640 4640 0 0.0
.data.rel.ro 76568 76568 0 0.0
.dynamic 592 592 0 0.0
.got 4696 4696 0 0.0
.init 27 27 0 0.0
.init_array 920 920 0 0.0
.rodata 216011 216011 0 0.0
.text 2446546 2446546 0 0.0
mbed lock-app CY8CPROTO_062_4343W+release (read only) 6224 6224 0 0.0
(read/write) 2411676 2411676 0 0.0
.bss 205660 205660 0 0.0
.data 5848 5848 0 0.0
.text 1374276 1374276 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 (read/write) 1177475 1177475 0 0.0
bss 141784 141784 0 0.0
rodata 150100 150100 0 0.0
text 806956 806956 0 0.0
p6 all-clusters-app default (read/write) 2527096 2527096 0 0.0
.bss 139040 139040 0 0.0
.data 2784 2784 0 0.0
.text 1485360 1485360 0 0.0
light-app default (read/write) 2418848 2418848 0 0.0
.bss 132520 132520 0 0.0
.data 2584 2584 0 0.0
.text 1377112 1377112 0 0.0
lock-app default (read/write) 2423040 2423040 0 0.0
.bss 132320 132320 0 0.0
.data 2544 2544 0 0.0
.text 1381304 1381304 0 0.0
telink lighting-app tlsr9518adk80d (read/write) 808020 808020 0 0.0
bss 75216 75216 0 0.0
noinit 40416 40416 0 0.0
text 571792 571792 0 0.0

@andy31415 andy31415 merged commit b16892f into project-chip:master Apr 26, 2022
AntonGrey pushed a commit to AntonGrey/connectedhomeip that referenced this pull request Apr 26, 2022
@kghost @AntonGrey
CASEClient: Remove unused defines (project-chip#17726)
b7ffce3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

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

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

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

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

@cecille cecille Awaiting requested review from cecille

@chrisdecenzo chrisdecenzo Awaiting requested review from chrisdecenzo

@chshu chshu Awaiting requested review from chshu

@chulspro chulspro Awaiting requested review from chulspro

@dhrishi dhrishi Awaiting requested review from dhrishi

@electrocucaracha electrocucaracha Awaiting requested review from electrocucaracha

@erjiaqing erjiaqing Awaiting requested review from erjiaqing

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

@gjc13 gjc13 Awaiting requested review from gjc13

@hawk248 hawk248 Awaiting requested review from hawk248

@holbrookt holbrookt Awaiting requested review from holbrookt

@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

@kpschoedel kpschoedel Awaiting requested review from kpschoedel

@lazarkov lazarkov Awaiting requested review from lazarkov

@LuDuda LuDuda Awaiting requested review from LuDuda

@lzgrablic02 lzgrablic02 Awaiting requested review from lzgrablic02

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

@msandstedt msandstedt Awaiting requested review from msandstedt

@mspang mspang Awaiting requested review from mspang

@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

@vijs vijs Awaiting requested review from vijs

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

@wbschiller wbschiller Awaiting requested review from wbschiller

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

@xylophone21 xylophone21 Awaiting requested review from xylophone21

@andy31415 andy31415 Awaiting requested review from andy31415

@yunhanw-google yunhanw-google Awaiting requested review from yunhanw-google

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

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

Successfully merging this pull request may close these issues.
3 participants
@kghost @andy31415 @Damian-Nordic