I will summarize some points which g++ warned.
(1) I fixed following points.
*Made destructor of ROMEConfig and ROMEDataBase virtual.
*Removed or comment out unused variables.
int i in ROMEPATH::~ROMEPath()
char* defaultRow in ROMESQL::ReadField
char* c in ROMEString::SetFormatted
int istart,iend,iValue,jValue in ROMEXMLDataBase::Read
int istep,iValue,jValue in ROMEXMLDataBase::Write
ROMEStrArray *array in ROMEXMLDataBase::Write
(2) Following points are not fixed yet.
* There can be unused "int i" in several codes which builder generates. (ex. XXAnalyzer::InitMidasBanks
)
This is not a problem, but a little bit annoying. If it is very easy, please fix it.
* Please make sure this is not a problem.
in ROMEAnalyzer:: ResponseFunction
while (obj = iterFolders->Next()) {
ROMEString str;
str.SetFormatted("%s",obj->GetName());
names->Add(new TObjString(str.Data()));
}
*ROMEString.cpp:74: warning: `int numberOfDigits' might be used uninitialized in this function
*ROMEXMLDataBase.cpp:80: warning: `ROMEXML*xml' might be used uninitialized in this function
(3) This is not warning from g++, but request from me.
If it is very easy, please make ROMEString::SetFormatted,AppendFormatted to check that number of
arguments is correct. |