connectedhomeip icon indicating copy to clipboard operation
connectedhomeip copied to clipboard

[Tooling] Update code generation MATTER_DM_PLUGIN_

Open lpbeliveau-silabs opened this issue 8 months ago • 0 comments

Reproduction steps / Feature

Add an #ifdef MATTER_DM_PLUGIN_ without #include <app/util/config.h> defined previously in the path.

Platform

core (please add to version below)

Platform Version(s)

1.3

Type

Manually tested with SDK

(Optional) If manually tested please explain why this is only manually tested

Any use of #ifdef MATTER_DM_PLUGIN_ relies on #include <app/util/config.h> to be present somewhere before the ifdef to function properly.

We should update our code generation to generate #define MATTER_DM_PLUGIN_ 1 when the cluster is enabled and 0 otherwise so we can use #if MATTER_DM_PLUGIN_ instead, which will fail in the event #include <app/util/config.h> is not on the path as expected.

Anything else?

No response

lpbeliveau-silabs avatar May 29 '24 13:05 lpbeliveau-silabs