Yes, I suggest we fix it.

> -----Original Message-----
> From: Song, BinX
> Sent: Tuesday, November 7, 2017 1:43 PM
> To: Yao, Jiewen <jiewen....@intel.com>; edk2-devel@lists.01.org
> Cc: Gao, Liming <liming....@intel.com>
> Subject: RE: [PATCH 05/14] IntelFsp2WrapperPkg: Enable MSFT C4255 warning
> 
> Hi Jiewen,
> 
> Do you mean we fix the problem after enable MSFT C4255 warning?
> If yes, I have fix them in related patch, such as MdeModulePkg.
> 
> Best Regards,
> Bell Song
> 
> 
> > -----Original Message-----
> > From: Yao, Jiewen
> > Sent: Tuesday, November 7, 2017 1:39 PM
> > To: Song, BinX <binx.s...@intel.com>; edk2-devel@lists.01.org
> > Cc: Gao, Liming <liming....@intel.com>
> > Subject: RE: [PATCH 05/14] IntelFsp2WrapperPkg: Enable MSFT C4255
> > warning
> >
> > Hi
> > I suggest we fix the problem.
> >
> > > -----Original Message-----
> > > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> > Song,
> > > BinX
> > > Sent: Tuesday, November 7, 2017 1:35 PM
> > > To: edk2-devel@lists.01.org
> > > Cc: Gao, Liming <liming....@intel.com>
> > > Subject: [edk2] [PATCH 05/14] IntelFsp2WrapperPkg: Enable MSFT C4255
> > warning
> > >
> > > Enable MSFT C4255 warning
> > >
> > > From MSDN:
> > > Compiler Warning (level 4) C4255
> > > function' : no function prototype given: converting '()' to '(void)'
> > > The compiler did not find an explicit list of arguments to a function.
> > > This warning is for the C compiler only.
> > >
> > > Cc: Liming Gao <liming....@intel.com>
> > > Contributed-under: TianoCore Contribution Agreement 1.1
> > > Signed-off-by: Bell Song <binx.s...@intel.com>
> > > ---
> > >  IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dsc | 1 +
> > >  1 file changed, 1 insertion(+)
> > >
> > > diff --git a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dsc
> > > b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dsc
> > > index 6496dad..4b4d5b2 100644
> > > --- a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dsc
> > > +++ b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dsc
> > > @@ -95,3 +95,4 @@
> > >
> > >  [BuildOptions]
> > >    *_*_*_CC_FLAGS = -D DISABLE_NEW_DEPRECATED_INTERFACES
> > > +  MSFT:*_*_*_CC_FLAGS = /we4255
> > > --
> > > 2.10.2.windows.1
> > >
> > > _______________________________________________
> > > edk2-devel mailing list
> > > edk2-devel@lists.01.org
> > > https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to