-
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
update eventDataIB and add eventReport, eventReports #11383
update eventDataIB and add eventReport, eventReports #11383
Conversation
Update IsUrgentconnectedhomeip/src/app/EventManagement.cpp Lines 319 to 329 in ed24591
This comment was generated by todo based on a
|
Update statusIBconnectedhomeip/src/app/EventManagement.cpp Lines 320 to 330 in ed24591
This comment was generated by todo based on a
|
ed24591
to
50fe96e
Compare
PR #11383: Size comparison from fe62374 to 50fe96e Increases (3 builds for k32w)
Full report (3 builds for k32w)
|
50fe96e
to
27566a0
Compare
Update IsUrgent, issue 11386connectedhomeip/src/app/EventManagement.cpp Lines 319 to 329 in 27566a0
This comment was generated by todo based on a
|
Update statusIB, issue 11388connectedhomeip/src/app/EventManagement.cpp Lines 320 to 330 in 27566a0
This comment was generated by todo based on a
|
PR #11383: Size comparison from fe62374 to 27566a0 Increases above 0.2%:
Increases (31 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
27566a0
to
69f14e2
Compare
PR #11383: Size comparison from 38fe11e to 69f14e2 Increases above 0.2%:
Increases (31 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
Full report (38 builds for efr32, esp32, k32w, linux, mbed, nrfconnect, p6, qpg, telink)
|
Problem
Update EventDataIB/EventReportIB/EventReports encoding per latest IM spec and update the corresponding EventManagement.
Change overview
See above
Testing
Add unit tests for EventDataIB/EventReportIB/EventReports