[edk2-devel] [patch] MdeModulePkg/DisplayEngine: Add Debug message to show mismatch menu info

2020-06-17 Thread Dandan Bi
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2326 Currently when meet mismatch case for one-of and ordered-list menu, just show a popup window to indicate mismatch, no more info for debugging. This patch is to add more debug message about mismatch menu info which is helpful to debug. Cc: L

Re: [edk2-devel] [patch] MdeModulePkg/DisplayEngine: Add Debug message to show mismatch menu info

2020-06-17 Thread Dong, Eric
Hi Dandan, It's good to show more info about the mismatch error. Current end user doesn't know which question need to check if this error appears. Can you post an example about the error message for one of and for ordered list? We need to make sure the message is easy to be understood and the e

Re: [edk2-devel] [patch] MdeModulePkg/DisplayEngine: Add Debug message to show mismatch menu info

2020-06-18 Thread Dandan Bi
Hi Eric, For example: For one of, if its current value is 4, but there is no option with value 4, then it will pop up mismatch window and the debug message like below. [DisplayEngine]: Mismatch Formset: Formset Guid = 9E0C30BC-3F06-4BA6-8288-09179B855DBE. [DisplayEngine]: Mismatch Form

Re: [edk2-devel] [patch] MdeModulePkg/DisplayEngine: Add Debug message to show mismatch menu info

2020-06-18 Thread Laszlo Ersek
On 06/18/20 05:24, Dandan Bi wrote: > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2326 > > Currently when meet mismatch case for one-of and ordered-list > menu, just show a popup window to indicate mismatch, no more > info for debugging. This patch is to add more debug message > about mism

Re: [edk2-devel] [patch] MdeModulePkg/DisplayEngine: Add Debug message to show mismatch menu info

2020-06-18 Thread Dong, Eric
Hi Dandan, Thanks for your detail explanation, add my comments below. Thanks, Eric From: Bi, Dandan Sent: Thursday, June 18, 2020 4:39 PM To: Dong, Eric ; devel@edk2.groups.io Cc: Gao, Liming Subject: RE: [patch] MdeModulePkg/DisplayEngine: Add Debug message to show mismatch menu info Hi Eric