summaryrefslogtreecommitdiff
path: root/OvmfPkg
diff options
context:
space:
mode:
authorLaszlo Ersek <lersek@redhat.com>2020-04-17 17:37:48 +0200
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>2020-04-22 19:42:32 +0000
commit92958abf7a0c87c6be3854bc1a5df2dfa35b876c (patch)
tree3ab184cb3d15ae076a7f1d2b6316eee339e433c1 /OvmfPkg
parent002f38a44a774648e88bc559f2b739e777a49017 (diff)
downloadedk2-92958abf7a0c87c6be3854bc1a5df2dfa35b876c.zip
edk2-92958abf7a0c87c6be3854bc1a5df2dfa35b876c.tar.gz
edk2-92958abf7a0c87c6be3854bc1a5df2dfa35b876c.tar.bz2
OvmfPkg/ResetSystemLib: improve coding style in ResetSystem()
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 <ard.biesheuvel@arm.com> Cc: Jordan Justen <jordan.l.justen@intel.com> Cc: Philippe Mathieu-Daudé <philmd@redhat.com> Cc: Rebecca Cran <rebecca@bsdio.com> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2675 Signed-off-by: Laszlo Ersek <lersek@redhat.com> Message-Id: <20200417153751.7110-4-lersek@redhat.com> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> Reviewed-by: Ard Biesheuvel <ard.biesheuvel@arm.com> Reviewed-by: Rebecca Cran <rebecca@bsdio.com>
Diffstat (limited to 'OvmfPkg')
-rw-r--r--OvmfPkg/Library/ResetSystemLib/ResetSystemLib.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/OvmfPkg/Library/ResetSystemLib/ResetSystemLib.c b/OvmfPkg/Library/ResetSystemLib/ResetSystemLib.c
index 2381618..0fc1423 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;
}
}