Skip to content

[v1.4] fix: missing field initializer build error (#38069)#38385

Merged
andy31415 merged 2 commits intoproject-chip:v1.4-branchfrom
shubhamdp:fix/data-model-provider.build.warn-1-4
Apr 14, 2025
Merged

[v1.4] fix: missing field initializer build error (#38069)#38385
andy31415 merged 2 commits intoproject-chip:v1.4-branchfrom
shubhamdp:fix/data-model-provider.build.warn-1-4

Conversation

@shubhamdp
Copy link
Contributor

@shubhamdp shubhamdp commented Apr 14, 2025 •
edited
Loading

Backporting #38069

Testing

  • build without change using idf.py build
/Users/shubhampatil/esp/connectedhomeip/src/app/data-model-provider/Provider.h:119:50: warning: missing initializer for member 'chip::app::DataModel::InteractionModelContext::dataModelChangeListener' [-Wmissing-field-initializers]
  119 |     InteractionModelContext mContext = { nullptr };
      |                                                  ^
  • After fix this warning goes off

@shubhamdp shubhamdp requested a review from andy31415 April 14, 2025 11:37
@github-actions github-actions bot added the app label Apr 14, 2025
@shubhamdp
Copy link
Contributor Author

shubhamdp commented Apr 14, 2025

@j0tunn PTAL!

@SuGlider SuGlider mentioned this pull request Apr 14, 2025
Closed
1 task
tcarmelveilleux
tcarmelveilleux reviewed Apr 14, 2025
View reviewed changes
@shubhamdp @tcarmelveilleux
Update src/app/data-model-provider/Provider.h
bfae047 
Co-authored-by: Tennessee Carmel-Veilleux <tennessee.carmelveilleux@gmail.com>
@andy31415 andy31415 merged commit 2a68f6d into project-chip:v1.4-branch Apr 14, 2025
66 of 69 checks passed
@shubhamdp shubhamdp deleted the fix/data-model-provider.build.warn-1-4 branch April 14, 2025 13:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tcarmelveilleux tcarmelveilleux tcarmelveilleux approved these changes

@andy31415 andy31415 Awaiting requested review from andy31415

Assignees

No one assigned

Labels

app fast track

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@shubhamdp @tcarmelveilleux @andy31415 @j0tunn