Skip to content

Commit

Permalink
ArmPkg: Disable AuditOnly mode for uncrustify
Browse files Browse the repository at this point in the history
Fix up a handful of uncrustify infractions and drop the AuditOnly
override for uncrustify so we get the same uncrustify CI coverage as
other packages.

Signed-off-by: Ard Biesheuvel <[email protected]>
  • Loading branch information
ardbiesheuvel authored and mergify[bot] committed Jul 24, 2024
1 parent d4ae23b commit a9c8c47
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 10 deletions.
5 changes: 0 additions & 5 deletions ArmPkg/ArmPkg.ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -239,10 +239,5 @@
],
"AdditionalIncludePaths": [] # Additional paths to spell check
# (wildcards supported)
},

# options defined in .pytool/Plugin/UncrustifyCheck
"UncrustifyCheck": {
"AuditOnly": True
}
}
2 changes: 1 addition & 1 deletion ArmPkg/Library/ArmMonitorLib/ArmMonitorLib.c
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ ArmMonitorCall (
IN OUT ARM_MONITOR_ARGS *Args
)
{
if (FeaturePcdGet (PcdMonitorConduitHvc)) {
if (FeaturePcdGet (PcdMonitorConduitHvc)) {
ArmCallHvc ((ARM_HVC_ARGS *)Args);
} else {
ArmCallSmc ((ARM_SMC_ARGS *)Args);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ PeCoffLoaderRelocateImageExtraAction (
IN OUT PE_COFF_LOADER_IMAGE_CONTEXT *ImageContext
)
{
#ifdef __GNUC__
#ifdef __GNUC__
if (ImageContext->PdbPointer) {
DEBUG ((
DEBUG_LOAD | DEBUG_INFO,
Expand All @@ -42,7 +42,8 @@ PeCoffLoaderRelocateImageExtraAction (
));
return;
}
#endif

#endif

DEBUG ((
DEBUG_LOAD | DEBUG_INFO,
Expand All @@ -68,7 +69,7 @@ PeCoffLoaderUnloadImageExtraAction (
IN OUT PE_COFF_LOADER_IMAGE_CONTEXT *ImageContext
)
{
#ifdef __GNUC__
#ifdef __GNUC__
if (ImageContext->PdbPointer) {
DEBUG ((
DEBUG_LOAD | DEBUG_INFO,
Expand All @@ -78,7 +79,8 @@ PeCoffLoaderUnloadImageExtraAction (
));
return;
}
#endif

#endif

DEBUG ((
DEBUG_LOAD | DEBUG_INFO,
Expand Down

0 comments on commit a9c8c47

Please sign in to comment.