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.

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

repl tests xml parsing: Ignore global commands parsing from xml #24676

Closed

Conversation

andy31415
Copy link
Contributor

Without this, we will get warnings that global::commands::* are not handled.

Matter does not seem to have a concept of global commands, so we skip these when parsing.

Also logged the name in case log level is reduced.

@andy31415
Copy link
Contributor Author

Closing in favour of #24672 ... better to remove than ignore.

@github-actions
Copy link

PR #24676: Size comparison from 86b064d to 4d2c921

Increases (3 builds for bl602, bl702)
platform target config section 86b064d 4d2c921 change % change
bl602 lighting-app bl602 .text 1022518 1022520 2 0.0
bl702 lighting-app bl702 (read/write) 1184935 1184951 16 0.0
.debug_info 40190977 40190979 2 0.0
.text 952352 952356 4 0.0
bl702+rpc .debug_info 44587269 44587271 2 0.0
.text 1029340 1029344 4 0.0
Full report (4 builds for bl602, bl702)
platform target config section 86b064d 4d2c921 change % change
bl602 lighting-app bl602 (read/write) 1345482 1345482 0 0.0
.bss 94858 94858 0 0.0
.data 9736 9736 0 0.0
.text 1022518 1022520 2 0.0
bl602+rpc (read/write) 1390930 1390930 0 0.0
.bss 102906 102906 0 0.0
.data 10128 10128 0 0.0
.text 1053452 1053452 0 0.0
bl702 lighting-app bl702 0 0 0 0.0
(read only) 3358 3358 0 0.0
(read/write) 1184935 1184951 16 0.0
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 292 292 0 0.0
.bss 70701 70701 0 0.0
.bss_psram 30048 30048 0 0.0
.comment 48 48 0 0.0
.data 4056 4056 0 0.0
.debug_abbrev 1549734 1549734 0 0.0
.debug_aranges 134056 134056 0 0.0
.debug_frame 491000 491000 0 0.0
.debug_info 40190977 40190979 2 0.0
.debug_line 5266095 5266095 0 0.0
.debug_loc 3401794 3401794 0 0.0
.debug_ranges 371904 371904 0 0.0
.debug_str 3535090 3535090 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 144 144 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 106704 106704 0 0.0
.rsvd 2960 2960 0 0.0
.sha_ocram 72 72 0 0.0
.shstrtab 304 304 0 0.0
.stack 2048 2048 0 0.0
.strtab 571662 571662 0 0.0
.symtab 173168 173168 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3358 3358 0 0.0
.text 952352 952356 4 0.0
bl702+rpc 0 0 0 0.0
(read only) 3358 3358 0 0.0
(read/write) 1277579 1277579 0 0.0
.bleromro 6342 6342 0 0.0
.bleromrw 124 124 0 0.0
.boot2 292 292 0 0.0
.bss 78749 78749 0 0.0
.bss_psram 30304 30304 0 0.0
.comment 48 48 0 0.0
.data 4608 4608 0 0.0
.debug_abbrev 1698172 1698172 0 0.0
.debug_aranges 142280 142280 0 0.0
.debug_frame 518704 518704 0 0.0
.debug_info 44587269 44587271 2 0.0
.debug_line 5664420 5664420 0 0.0
.debug_loc 3597931 3597931 0 0.0
.debug_ranges 395608 395608 0 0.0
.debug_str 3938399 3938399 0 0.0
.hbn 536 536 0 0.0
.hbn_noinit 260 260 0 0.0
.init 342 342 0 0.0
.init_array 160 160 0 0.0
.psram 0 0 0 0.0
.riscv.attributes 47 47 0 0.0
.rodata 121232 121232 0 0.0
.rsvd 2960 2960 0 0.0
.sha_ocram 72 72 0 0.0
.shstrtab 304 304 0 0.0
.stack 2048 2048 0 0.0
.strtab 632259 632259 0 0.0
.symtab 191520 191520 0 0.0
.tcm_data 36 36 0 0.0
.tcmcode 3358 3358 0 0.0
.text 1029340 1029344 4 0.0

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

Successfully merging this pull request may close these issues.

1 participant