ECC tool incorrectly reports that include guards must have prefix and postfix underscores

ECC tool incorrectly reports that include guards must have prefix and postfix underscores

[Rebecca Cran]

I came across what appears to be a bug in the ECC tool. I was running my 
bhyve changes through a personal EDK2 Github build, and ECC failed with 
the ERROR_NAMING_CONVENTION_CHECK_IFNDEF_STATEMENT:


./Source/Python/Ecc/EccToolError.py:164: 
ERROR_NAMING_CONVENTION_CHECK_IFNDEF_STATEMENT : "The #ifndef at the 
start of an include file should use both prefix and postfix underscore 
characters, '_'"


It appears the tool really only checks that there's at least one 
underscore, so it seems it does _allow_ the style documented in the C 
coding standard 
(https://edk2-docs.gitbook.io/edk-ii-c-coding-standards-specification/5_source_files/53_include_files#5-3-5-all-include-file-contents-must-be-protected-by-a-include-guard), 
which has a postfix underscore but which explicitly disallows prefix 
underscore(s).


I've created https://bugzilla.tianocore.org/show_bug.cgi?id=3094 .


-- 
Rebecca Cran