Fix compiler warning!
authorlgao4 <lgao4@7335b38e-4728-0410-8992-fb3ffe349368>
Wed, 30 Dec 2009 02:25:01 +0000 (02:25 +0000)
committerlgao4 <lgao4@7335b38e-4728-0410-8992-fb3ffe349368>
Wed, 30 Dec 2009 02:25:01 +0000 (02:25 +0000)
git-svn-id: https://buildtools.tianocore.org/svn/buildtools/trunk/BaseTools@1786 7335b38e-4728-0410-8992-fb3ffe349368

Source/C/VfrCompile/VfrCompiler.cpp
Source/C/VfrCompile/VfrError.cpp
Source/C/VfrCompile/VfrFormPkg.cpp
Source/C/VfrCompile/VfrFormPkg.h
Source/C/VfrCompile/VfrSyntax.g
Source/C/VfrCompile/VfrUtilityLib.cpp

index e3c150c..44ce838 100644 (file)
@@ -84,8 +84,6 @@ CVfrCompiler::OptionInitialization (
       mOptions.CreateRecordListFile = TRUE;\r
       gCIfrRecordInfoDB.TurnOn ();\r
     } else if (stricmp(Argv[Index], "-i") == 0) {\r
-      Error (NULL, 0, 1000, "Unknown option", "unrecognized option %s", Argv[Index]);\r
-      goto Fail;\r
       Index++;\r
       if ((Index >= Argc) || (Argv[Index][0] == '-')) {\r
         Error (NULL, 0, 1001, "Missing option", "-i missing path argument"); \r
@@ -115,8 +113,6 @@ CVfrCompiler::OptionInitialization (
     } else if (stricmp(Argv[Index], "-n") == 0 || stricmp(Argv[Index], "--no-pre-processing") == 0 || stricmp(Argv[Index], "-nopp") == 0) {\r
       mOptions.SkipCPreprocessor = TRUE;\r
     } else if (stricmp(Argv[Index], "-f") == 0 || stricmp(Argv[Index], "--pre-processing-flag") == 0 || stricmp(Argv[Index], "-ppflag") == 0) {\r
-      Error (NULL, 0, 1000, "Unknown option", "unrecognized option %s", Argv[Index]);\r
-      goto Fail;\r
       Index++;\r
       if ((Index >= Argc) || (Argv[Index][0] == '-')) {\r
         Error (NULL, 0, 1001, "Missing option", "-od - missing C-preprocessor argument");\r
index 963bd2d..ae5f8c6 100644 (file)
@@ -135,7 +135,6 @@ CVfrErrorHandle::ParseFileScopeRecord (
   IN UINT32    WholeScopeLine\r
   )\r
 {\r
-  CHAR8               *FullPathName = NULL;\r
   SVfrFileScopeRecord *pNode        = NULL;\r
 \r
   if (Record == NULL) {\r
index ffa898b..26f0cf8 100644 (file)
@@ -1340,7 +1340,7 @@ CIFROBJ_DEBUG_PRINT (
 \r
 #endif\r
 \r
-bool gCreateOp = TRUE;\r
+BOOLEAN gCreateOp = TRUE;\r
 \r
 CIfrObj::CIfrObj (\r
   IN  UINT8   OpCode,\r
index 0b84896..52e5597 100644 (file)
@@ -195,11 +195,11 @@ extern CIfrRecordInfoDB gCIfrRecordInfoDB;
 /*\r
  * The definition of CIfrObj\r
  */\r
-extern bool  gCreateOp;\r
+extern BOOLEAN  gCreateOp;\r
 \r
 class CIfrObj {\r
 private:\r
-  bool    mDelayEmit;\r
+  BOOLEAN mDelayEmit;\r
 \r
   CHAR8   *mObjBinBuf;\r
   UINT8   mObjBinLen;\r
@@ -227,7 +227,7 @@ public:
 \r
   inline bool ExpendObjBin (IN UINT8 Size) {\r
     if ((mDelayEmit == TRUE) && ((mObjBinLen + Size) > mObjBinLen)) {\r
-      mObjBinLen += Size;\r
+      mObjBinLen = mObjBinLen + Size;\r
       return TRUE;\r
     } else {\r
       return FALSE;\r
@@ -248,7 +248,7 @@ public:
 \r
   VOID IncLength (UINT8 Size) {\r
     if ((mHeader->Length + Size) > mHeader->Length) {\r
-      mHeader->Length += Size;\r
+      mHeader->Length = mHeader->Length + Size;\r
     }\r
   }\r
 \r
@@ -696,7 +696,7 @@ public:
     UINT8 Len;\r
 \r
     if (Name != NULL) {\r
-      Len = strlen (Name);\r
+      Len = (UINT8) strlen (Name);\r
       if (Len != 0) {\r
         if (ExpendObjBin (Len) == TRUE) {\r
           IncLength (Len);\r
index 09ec691..c1d5f36 100644 (file)
@@ -628,7 +628,7 @@ vfrStatementVarStoreLinear :
                                                        _PCATCH(mCVfrDataStorage.GetVarStoreId(StoreName, &VarStoreId), SN);\r
                                                        VSObj.SetVarStoreId (VarStoreId);\r
                                                        _PCATCH(gCVfrVarDataTypeDB.GetDataTypeSize(TypeName, &Size), LineNum);\r
-                                                       VSObj.SetSize (Size);\r
+                                                       VSObj.SetSize ((UINT16) Size);\r
                                                        VSObj.SetName (SN->getText());\r
                                                     >>\r
   ";"\r
@@ -1268,8 +1268,8 @@ vfrStatementCrossReference :
 vfrStatementGoto :\r
   <<\r
      UINT8               RefType = 1;\r
-     EFI_STRING_ID       DevPath;\r
-     EFI_GUID            FSId;\r
+     EFI_STRING_ID       DevPath = EFI_STRING_ID_INVALID;\r
+     EFI_GUID            FSId = {0, {0,}};\r
      EFI_FORM_ID         FId;\r
      EFI_QUESTION_ID     QId    = EFI_QUESTION_ID_INVALID;\r
      UINT32              BitMask;\r
@@ -2937,8 +2937,8 @@ ideqvallistExp[UINT32 & RootLevel, UINT32 & ExpOpCount] :
 questionref13Exp[UINT32 & RootLevel, UINT32 & ExpOpCount] :\r
   <<\r
      UINT8           Type = 0x1;\r
-     EFI_STRING_ID   DevPath;\r
-     EFI_GUID        Guid;\r
+     EFI_STRING_ID   DevPath = EFI_STRING_ID_INVALID;\r
+     EFI_GUID        Guid = {0, {0, }};\r
      EFI_QUESTION_ID QId = EFI_QUESTION_ID_INVALID;\r
      UINT32          BitMask;\r
      CHAR8           *QName = NULL;\r
@@ -3381,7 +3381,7 @@ EfiVfrParser::_PCATCH (
   IN EFI_VFR_RETURN_CODE ReturnCode\r
   )\r
 {\r
-  mParserStatus += gCVfrErrorHandle.HandleError (ReturnCode);\r
+  mParserStatus = mParserStatus + gCVfrErrorHandle.HandleError (ReturnCode);\r
 }\r
 \r
 VOID\r
@@ -3390,7 +3390,7 @@ EfiVfrParser::_PCATCH (
   IN ANTLRTokenPtr       Tok\r
   )\r
 {\r
-  mParserStatus += gCVfrErrorHandle.HandleError (ReturnCode, Tok->getLine(), Tok->getText());\r
+  mParserStatus = mParserStatus + gCVfrErrorHandle.HandleError (ReturnCode, Tok->getLine(), Tok->getText());\r
 }\r
 \r
 VOID\r
@@ -3399,7 +3399,7 @@ EfiVfrParser::_PCATCH (
   IN UINT32              LineNum\r
   )\r
 {\r
-  mParserStatus += gCVfrErrorHandle.HandleError (ReturnCode, LineNum);\r
+  mParserStatus = mParserStatus + gCVfrErrorHandle.HandleError (ReturnCode, LineNum);\r
 }\r
 \r
 VOID\r
@@ -3409,7 +3409,7 @@ EfiVfrParser::_PCATCH (
   IN CHAR8               *ErrorMsg\r
   )\r
 {\r
-  mParserStatus += gCVfrErrorHandle.HandleError (ReturnCode, LineNum, ErrorMsg);\r
+  mParserStatus = mParserStatus + gCVfrErrorHandle.HandleError (ReturnCode, LineNum, ErrorMsg);\r
 }\r
 \r
 VOID\r
@@ -3686,7 +3686,7 @@ EfiVfrParser::_DeclareDefaultFrameworkVarStore (
     CIfrVarStore      VSObj;\r
     VSObj.SetLineNo (LineNo);\r
     VSObj.SetVarStoreId (0x1); //the first and only one Buffer Var Store\r
-    VSObj.SetSize (TypeSize);\r
+    VSObj.SetSize ((UINT16) TypeSize);\r
     //VSObj.SetName (gCVfrVarDataTypeDB.mFirstNewDataTypeName);\r
     VSObj.SetName ("Setup");\r
     VSObj.SetGuid (&mFormsetGuid);\r
@@ -3703,7 +3703,7 @@ EfiVfrParser::_DeclareDefaultFrameworkVarStore (
         CIfrVarStore      VSObj;\r
         VSObj.SetLineNo (LineNo);\r
         VSObj.SetVarStoreId (pNode->mVarStoreId);\r
-        VSObj.SetSize (pNode->mStorageInfo.mDataType->mTotalSize);\r
+        VSObj.SetSize ((UINT16) pNode->mStorageInfo.mDataType->mTotalSize);\r
         if (FirstNode) {\r
           VSObj.SetName ("Setup");\r
           FirstNode = FALSE;\r
@@ -3764,7 +3764,7 @@ EfiVfrParser::_DeclareDefaultLinearVarStore (
     mCVfrDataStorage.GetVarStoreId(TypeNameList[Index], &VarStoreId);\r
     VSObj.SetVarStoreId (VarStoreId);\r
     gCVfrVarDataTypeDB.GetDataTypeSize(TypeNameList[Index], &Size);\r
-    VSObj.SetSize (Size);\r
+    VSObj.SetSize ((UINT16) Size);\r
     VSObj.SetName (TypeNameList[Index]);\r
     VSObj.SetGuid (&mFormsetGuid);\r
   }\r
@@ -3789,7 +3789,7 @@ EfiVfrParser::_DeclareDefaultLinearVarStore (
     mCVfrDataStorage.GetVarStoreId("Date", &VarStoreId);\r
     VSObj.SetVarStoreId (VarStoreId);\r
     gCVfrVarDataTypeDB.GetDataTypeSize("EFI_HII_DATE", &Size);\r
-    VSObj.SetSize (Size);\r
+    VSObj.SetSize ((UINT16) Size);\r
     VSObj.SetName ("Date");\r
     VSObj.SetGuid (&mFormsetGuid);\r
   }\r
@@ -3810,7 +3810,7 @@ EfiVfrParser::_DeclareDefaultLinearVarStore (
     mCVfrDataStorage.GetVarStoreId("Time", &VarStoreId);\r
     VSObj.SetVarStoreId (VarStoreId);\r
     gCVfrVarDataTypeDB.GetDataTypeSize("EFI_HII_TIME", &Size);\r
-    VSObj.SetSize (Size);\r
+    VSObj.SetSize ((UINT16) Size);\r
     VSObj.SetName ("Time");\r
     VSObj.SetGuid (&mFormsetGuid);\r
   }\r
index f047b2c..e4a34f5 100644 (file)
@@ -287,7 +287,7 @@ CVfrBufferConfig::Write (
   switch (Mode) {\r
   case 'a' : // add\r
     if (Select (Name, Id) != 0) {\r
-      if ((pItem = new SConfigItem (Name, Id, Type, Offset, Width, Value)) == NULL) {\r
+      if ((pItem = new SConfigItem (Name, Id, Type, Offset, (UINT16) Width, Value)) == NULL) {\r
         return 2;\r
       }\r
       if (mItemListHead == NULL) {\r
@@ -1127,7 +1127,7 @@ CVfrVarDataTypeDB::GetDataFieldInfo (
     CHECK_ERROR_RETURN(GetTypeField (FName, pType, pField), VFR_RETURN_SUCCESS);\r
     pType  = pField->mFieldType;\r
     CHECK_ERROR_RETURN(GetFieldOffset (pField, ArrayIdx, Tmp), VFR_RETURN_SUCCESS);\r
-    Offset += Tmp;\r
+    Offset = (UINT16) (Offset + Tmp);\r
     Type   = GetFieldWidth (pField);\r
     Size   = GetFieldSize (pField, ArrayIdx);\r
   }\r
@@ -1386,6 +1386,7 @@ CVfrDataStorage::GetFreeVarStoreId (
   //\r
   // Assign the different ID range for the different type VarStore to support Framework Vfr\r
   //\r
+  Index = 0;\r
   if ((!VfrCompatibleMode) || (VarType == EFI_VFR_VARSTORE_BUFFER)) {\r
     Index = 0;\r
   } else if (VarType == EFI_VFR_VARSTORE_EFI) {\r
@@ -1838,8 +1839,6 @@ CVfrDataStorage::BufferVarStoreRequestElementAdd (
   IN EFI_VARSTORE_INFO &Info\r
   )\r
 {\r
-  CHAR8                 NewReqElt[128] = {'\0',};\r
-  CHAR8                 *OldReqElt = NULL;\r
   SVfrVarStorageNode    *pNode = NULL;\r
   EFI_IFR_TYPE_VALUE    Value = gZeroEfiIfrTypeValue;\r
 \r
@@ -2024,85 +2023,6 @@ CVfrDefaultStore::GetDefaultId (
   return VFR_RETURN_UNDEFINED;\r
 }\r
 \r
-STATIC\r
-EFI_VFR_RETURN_CODE\r
-AltCfgItemPrintToBuffer (\r
-  IN CHAR8              *NewAltCfg, \r
-  IN EFI_VARSTORE_INFO  Info, \r
-  IN UINT8              Type,\r
-  IN EFI_IFR_TYPE_VALUE Value\r
-  )\r
-{\r
-  UINT32 Index;\r
-  UINT8  *BufChar = NULL;\r
-  UINT32 Count    = 0;\r
-\r
-  if (NewAltCfg != NULL) {\r
-    Count = sprintf (\r
-              NewAltCfg,\r
-              "&OFFSET=%x&WIDTH=%x&VALUE=",\r
-              Info.mInfo.mVarOffset,\r
-              Info.mVarTotalSize\r
-              );\r
-    NewAltCfg += Count;\r
-\r
-    switch (Type) {\r
-    case EFI_IFR_TYPE_NUM_SIZE_8 :\r
-      Count = sprintf (NewAltCfg, "%x", Value.u8);\r
-      NewAltCfg += Count;\r
-      break;\r
-    case EFI_IFR_TYPE_NUM_SIZE_16 :\r
-      Count = sprintf (NewAltCfg, "%x", Value.u16);\r
-      NewAltCfg += Count;\r
-      break;\r
-    case EFI_IFR_TYPE_NUM_SIZE_32 :\r
-      Count = sprintf (NewAltCfg, "%x", Value.u32);\r
-      NewAltCfg += Count;\r
-      break;\r
-    case EFI_IFR_TYPE_NUM_SIZE_64 :\r
-      Count = sprintf (NewAltCfg, "%x", Value.u64);\r
-      NewAltCfg += Count;\r
-      break;\r
-    case EFI_IFR_TYPE_BOOLEAN :\r
-      Count = sprintf (NewAltCfg, "%x", Value.b);\r
-      NewAltCfg += Count;\r
-      break;\r
-    case EFI_IFR_TYPE_TIME :\r
-#if 1\r
-      Count = sprintf (NewAltCfg, "%x", *((UINT32 *)(&Value.time)));\r
-      NewAltCfg += Count;\r
-#else\r
-      BufChar = (UINT8 *)&Value.time;\r
-      for (Index = 0; Index < sizeof(EFI_HII_TIME); Index++) {\r
-        Count = sprintf (NewAltCfg, "%02x", (UINT8)BufChar[Index]);\r
-        NewAltCfg += Count;\r
-      }\r
-#endif\r
-      break;\r
-    case EFI_IFR_TYPE_DATE :\r
-#if 1\r
-      Count = sprintf (NewAltCfg, "%x", *((UINT32 *)(&Value.date)));\r
-      NewAltCfg += Count;\r
-#else\r
-      BufChar = (UINT8 *)&Value.date;\r
-      for (Index = 0; Index < sizeof(EFI_HII_DATE); Index++) {\r
-        Count = sprintf (NewAltCfg, "%02x", (UINT8)BufChar[Index]);\r
-        NewAltCfg += Count;\r
-      }\r
-#endif\r
-      break;\r
-    case EFI_IFR_TYPE_STRING :\r
-      Count = sprintf (NewAltCfg, "%x", Value.string);\r
-      NewAltCfg += Count;\r
-      break;\r
-    case EFI_IFR_TYPE_OTHER :\r
-      return VFR_RETURN_UNSUPPORTED;\r
-       }\r
-  }\r
-\r
-  return VFR_RETURN_FATAL_ERROR;\r
-}\r
-\r
 EFI_VFR_RETURN_CODE\r
 CVfrDefaultStore::BufferVarStoreAltConfigAdd (\r
   IN EFI_VARSTORE_ID    DefaultId,\r