diff options
author | Laszlo Ersek <lersek@redhat.com> | 2020-04-17 17:37:48 +0200 |
---|---|---|
committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2020-04-22 19:42:32 +0000 |
commit | 92958abf7a0c87c6be3854bc1a5df2dfa35b876c (patch) | |
tree | 3ab184cb3d15ae076a7f1d2b6316eee339e433c1 /OvmfPkg | |
parent | 002f38a44a774648e88bc559f2b739e777a49017 (diff) | |
download | edk2-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.c | 6 |
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;
}
}
|