https://bugzilla.tianocore.org/show_bug.cgi?id=4292

--- Comment #3 from Veeresh Sangolli <veeresh.sangolli@...> ---
Created attachment 1314
--> https://bugzilla.tianocore.org/attachment.cgi?id=1314&action=edit
Proposed patch to resolve the MISSING_BREAK issue pointed by Coverity for
MdeModulePkg

The function ReadString has a switch-case on Key.ScanCode in which

case CHAR_BACKSPACE: falls through to
default:

While this may be intentional, it is not evident to any general code
reader as well as any static analyzer tool. Just adding

// No break; here as this is an intentional fallthrough.

as comment in between makes any reader as well as Coverity happy.

--
You are receiving this mail because:
You are on the CC list for the bug.