Skip to content
This repository was archived by the owner on Dec 20, 2023. It is now read-only.

Commit 220b97b

Browse files
authored
Merge pull request #361 from robszewczyk/bug/clang9-warning-fixes
Clang 9 warning fix.
2 parents 68913e6 + 161279e commit 220b97b

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

src/lib/profiles/data-management/Current/UpdateClient.h

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,16 @@ struct UpdateClient::InEventParam
122122
struct UpdateClient::OutEventParam
123123
{
124124
bool DefaultHandlerCalled;
125-
union
126-
{
127-
};
125+
126+
// additional parameters required by application shall be added via structs
127+
// added to an anonymous union, following the pattern for InEventParam
128+
// above. As some compilers emit warnings for anonymous empty unions, we
129+
// leave the union declaration commented out below.
130+
131+
// union
132+
// {
133+
// // Parameters for specific purposes should be embedded here as structs
134+
// };
128135
void Clear() { memset(this, 0, sizeof(*this)); }
129136
};
130137
}; // namespace WeaveMakeManagedNamespaceIdentifier(DataManagement, kWeaveManagedNamespaceDesignation_Current)

0 commit comments

Comments
 (0)