Hi Michael, I would prefer to have the debug macro checks enabled in DynamicTablesPkg to catch any issues.
Please do not disable this feature for this package. Regards, Sami Mujawar On 13/09/2023, 18:08, "mikub...@linux.microsoft.com <mailto:mikub...@linux.microsoft.com>" <mikub...@linux.microsoft.com <mailto:mikub...@linux.microsoft.com>> wrote: From: Michael Kubacki <michael.kuba...@microsoft.com <mailto:michael.kuba...@microsoft.com>> Adds a CI YAML entry to acknowledge a case where custom strings contain print specifiers for a single debug macro. Cc: Sami Mujawar <sami.muja...@arm.com <mailto:sami.muja...@arm.com>> Cc: Alexei Fedorov <alexei.fedo...@arm.com <mailto:alexei.fedo...@arm.com>> Cc: Pierre Gondois <pierre.gond...@arm.com <mailto:pierre.gond...@arm.com>> Signed-off-by: Michael Kubacki <michael.kuba...@microsoft.com <mailto:michael.kuba...@microsoft.com>> --- DynamicTablesPkg/DynamicTablesPkg.ci.yaml | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/DynamicTablesPkg/DynamicTablesPkg.ci.yaml b/DynamicTablesPkg/DynamicTablesPkg.ci.yaml index 5addf8626841..1d41d44bbf33 100644 --- a/DynamicTablesPkg/DynamicTablesPkg.ci.yaml +++ b/DynamicTablesPkg/DynamicTablesPkg.ci.yaml @@ -130,5 +130,13 @@ # should be ignore "AdditionalIncludePaths": [] # Additional paths to spell check # (wildcards supported) + }, + + "DebugMacroCheck": { + "StringSubstitutions": { + # DynamicTablesPkg/Library/Common/TableHelperLib/ConfigurationManagerObjectParser.c + # Reason: Debug format strings are dynamically set. + "Parser[Index].Format": "%d" + } } } -- 2.42.0.windows.2 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#108642): https://edk2.groups.io/g/devel/message/108642 Mute This Topic: https://groups.io/mt/101341656/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
