From 80873def607ba8056e5aab40eaa3be07e9a5ec19 Mon Sep 17 00:00:00 2001 From: Eric Dong Date: Thu, 26 Dec 2013 04:19:37 +0000 Subject: Keep consistent about the return value between the caller and callee. Signed-off-by: Eric Dong Reviewed-by: Liming Gao git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@15021 6f19259b-4bc3-4df7-8a09-765794883524 --- MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c | 6 +++--- MdeModulePkg/Universal/SetupBrowserDxe/Setup.c | 2 ++ 2 files changed, 5 insertions(+), 3 deletions(-) (limited to 'MdeModulePkg/Universal/SetupBrowserDxe') diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c b/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c index f8b1d50..788a63f 100644 --- a/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c +++ b/MdeModulePkg/Universal/SetupBrowserDxe/Presentation.c @@ -1263,15 +1263,15 @@ ProcessChangedData ( RetValue = TRUE; switch (mFormDisplay->ConfirmDataChange()) { - case BROWSER_ACTION_DISCARD: + case BROWSER_DISCARD_CHANGES: DiscardForm (Selection->FormSet, Selection->Form, Scope); break; - case BROWSER_ACTION_SUBMIT: + case BROWSER_SAVE_CHANGES: SubmitForm (Selection->FormSet, Selection->Form, Scope); break; - case BROWSER_ACTION_NONE: + case BROWSER_NO_CHANGES: RetValue = FALSE; break; diff --git a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c index c7c7f9a..3671c0e 100644 --- a/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c +++ b/MdeModulePkg/Universal/SetupBrowserDxe/Setup.c @@ -5062,6 +5062,8 @@ SaveReminder ( } else if (DataSavedAction == BROWSER_DISCARD_CHANGES) { DiscardForm (NULL, NULL, SystemLevel); break; + } else if (DataSavedAction == BROWSER_NO_CHANGES) { + break; } } while (1); -- cgit v1.1