Re: [edk2-devel] [PATCH] UnitTestFrameworkPkg: Invalid index may be used.

2020-03-24 Thread Bret Barkelew via Groups.Io
: Sean Brogan ; Bret Barkelew Subject: [EXTERNAL] Re: [edk2-devel] [PATCH] UnitTestFrameworkPkg: Invalid index may be used. Reviewed-by: Michael D Kinney Mike > -Original Message- > From: Jiang, Guomin > Sent: Sunday, March 22, 2020 10:56 PM > To: devel@edk2.groups.io; Zha

Re: [edk2-devel] [PATCH] UnitTestFrameworkPkg: Invalid index may be used.

2020-03-23 Thread Michael D Kinney
oft.com > Subject: RE: [edk2-devel] [PATCH] UnitTestFrameworkPkg: > Invalid index may be used. > > Hi Michael, Sean, Bret, > > Could help review the patch? > > Thanks > > -Original Message- > > From: devel@edk2.groups.io > [mailto:devel@edk2.gro

Re: [edk2-devel] [PATCH] UnitTestFrameworkPkg: Invalid index may be used.

2020-03-22 Thread Guomin Jiang
y 20, 2020 9:42 AM > > To: devel@edk2.groups.io > > Cc: GuoMinJ > > Subject: [edk2-devel] [PATCH] UnitTestFrameworkPkg: Invalid index may > > be used. > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2535 > > > > The UINT_TEST_STATUS and FAI

Re: [edk2-devel] [PATCH] UnitTestFrameworkPkg: Invalid index may be used.

2020-03-04 Thread Zhang, Shenglei
Reviewed-by: Shenglei Zhang > -Original Message- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of > GuoMinJ > Sent: Thursday, February 20, 2020 9:42 AM > To: devel@edk2.groups.io > Cc: GuoMinJ > Subject: [edk2-devel] [PATCH] UnitTestFrameworkP

Re: [EXTERNAL] [edk2-devel] [PATCH] UnitTestFrameworkPkg: Invalid index may be used.

2020-02-20 Thread Bret Barkelew via Groups.Io
Reviewed-by: Bret Barkelew - Bret From: devel@edk2.groups.io on behalf of GuoMinJ via Groups.Io Sent: Wednesday, February 19, 2020 5:41:56 PM To: devel@edk2.groups.io Cc: GuoMinJ Subject: [EXTERNAL] [edk2-devel] [PATCH] UnitTestFrameworkPkg: Invalid index

[edk2-devel] [PATCH] UnitTestFrameworkPkg: Invalid index may be used.

2020-02-19 Thread GuoMinJ
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2535 The UINT_TEST_STATUS and FAILURE_TYPE have used 0 as status, so use 0 as unknown is confused, remove it from array enumeration but keep it location in the array. Signed-off-by: GuoMinJ ---