Thanks for the reply. I understand now that this is not a very proper change. I will talk to the bug reporter in Bugzilla to see if we can keep the current status.
Best Regards, Zhiguang Liu >-----Original Message----- >From: Justen, Jordan L >Sent: Thursday, June 13, 2019 3:47 PM >To: Liu, Zhiguang <zhiguang....@intel.com>; devel@edk2.groups.io >Cc: Andrew Fish <af...@apple.com>; Ni, Ray <ray...@intel.com> >Subject: RE: [Patch V3] EmulatorPkg: don't display the cpu current speed > >On 2019-06-11 22:42:13, Liu, Zhiguang wrote: >> Thanks for the comments. >> I will change the commit message in the next version. >> But I don't think this is a issue worth making a major change. >> Given that the change is consistent with NT32, will you agree with this >change? > >Hmm. You are right that NT32 also does this. I don't agree enough to give you >a Reviewed-by, but if Andrew or Ray give a Reviewed-by, then I'm ok for us to >take the patch. > >-Jordan -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#42400): https://edk2.groups.io/g/devel/message/42400 Mute This Topic: https://groups.io/mt/32013654/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
