Leif: How about submit one BZ to catch it first? Then, revisit it later.
Thanks Liming > -----Original Message----- > From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of Leif > Lindholm > Sent: Monday, April 15, 2019 5:02 PM > To: Wu, Hao A <hao.a...@intel.com> > Cc: Ard Biesheuvel <ard.biesheu...@linaro.org>; devel@edk2.groups.io; Kinney, > Michael D <michael.d.kin...@intel.com>; Andrew Fish > <af...@apple.com>; Zeng, Star <star.z...@intel.com>; Dong, Eric > <eric.d...@intel.com>; Ni, Ray <ray...@intel.com>; Gao, Liming > <liming....@intel.com>; Carsey, Jaben <jaben.car...@intel.com>; Shi, Steven > <steven....@intel.com>; Wang, Jian J > <jian.j.w...@intel.com> > Subject: Re: [edk2-devel] [PATCH v6 1/7] MdeModulePkg: introduce PE/COFF > image emulator protocol > > On Mon, Apr 15, 2019 at 01:23:11AM +0000, Wu, Hao A wrote: > > > + This program and the accompanying materials are licensed and made > > > available > > > + under the terms and conditions of the BSD License which accompanies > > > this > > > + distribution. The full text of the license may be found at > > > + http://opensource.org/licenses/bsd-license.php > > > + > > > + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" > > > BASIS, > > > + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER > > > EXPRESS OR IMPLIED. > > > + > > > +**/ > > > + > > > +#ifndef PECOFF_IMAGE_EMULATOR_PROTOCOL_GUID_H > > > +#define PECOFF_IMAGE_EMULATOR_PROTOCOL_GUID_H > > > > Hello Ard, > > > > Sorry for the delayed response. > > > > Could you help to use: > > _PECOFF_IMAGE_EMULATOR_PROTOCOL_GUID_H_ > > > > here to please the Ecc checker when you push the series? > > Hmm... > > Not a major issue, but I'll mention it anyway (which I mentioned to > Andrew/Mike at Linaro Connect in Vancouver last year): > clang has a warning, enabled by -Wreserved-id-macro, which complains > about this. > > The Coding Style matches this, suggesting > MACROS_SHOULD_BE_WRITTEN_THUS_. So do we need to fix Ecc? > > Regards, > > Leif > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#39089): https://edk2.groups.io/g/devel/message/39089 Mute This Topic: https://groups.io/mt/31133140/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
