The cases under ResetSystem() currently mix "break"s with "return"s for no good reason. Use "break" consistently.
(The inconsistency was introduced in commit 84c0b80de716, "OvmfPkg/ResetSystemLib: Add new API ResetSystem", 2019-04-28.) Cc: Ard Biesheuvel <[email protected]> Cc: Jordan Justen <[email protected]> Cc: Philippe Mathieu-Daudé <[email protected]> Cc: Rebecca Cran <[email protected]> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2675 Signed-off-by: Laszlo Ersek <[email protected]> --- OvmfPkg/Library/ResetSystemLib/ResetSystemLib.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/OvmfPkg/Library/ResetSystemLib/ResetSystemLib.c b/OvmfPkg/Library/ResetSystemLib/ResetSystemLib.c index 23816183a953..0fc142325ece 100644 --- a/OvmfPkg/Library/ResetSystemLib/ResetSystemLib.c +++ b/OvmfPkg/Library/ResetSystemLib/ResetSystemLib.c @@ -157,13 +157,13 @@ ResetSystem ( case EfiResetShutdown: ResetShutdown (); - return; + break; case EfiResetPlatformSpecific: ResetPlatformSpecific (DataSize, ResetData); - return; + break; default: - return; + break; } } -- 2.19.1.3.g30247aa5d201 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#57517): https://edk2.groups.io/g/devel/message/57517 Mute This Topic: https://groups.io/mt/73085502/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
