Compilation warnings

[GoogleCodeExporter]

Centos 5.x, g++44, -Wall gives the following:

NaiveAlignmentSet.cpp: In member function ‘bool 
CNaiveAlignmentSet::Add(Alignment&)’:
NaiveAlignmentSet.cpp:82: warning: converting to ‘unsigned int’ from 
‘float’
NaiveAlignmentSet.cpp:92: warning: converting to ‘unsigned int’ from 
‘float’

ArchiveMerge.cpp: In member function ‘void 
CArchiveMerge::WriteAlignment(Mosaik::AlignedRead&)’:
ArchiveMerge.cpp:475: warning: converting to ‘unsigned int’ from ‘float’
ArchiveMerge.cpp:477: warning: converting to ‘unsigned int’ from ‘float’
ArchiveMerge.cpp:499: warning: converting to ‘unsigned int’ from ‘float’
ArchiveMerge.cpp:501: warning: converting to ‘unsigned int’ from ‘float’

QualityNeuralNetwork.cpp: In function ‘unsigned char float2phred(float)’:
QualityNeuralNetwork.cpp:17: warning: converting to ‘unsigned char’ from 
‘double’
QualityNeuralNetwork.cpp: In member function ‘unsigned char 
QualityNeuralNetwork::GetQualityPe(const QualityNeuralNetwork::FannInputs&, 
const QualityNeuralNetwork::FannInputs&, const int&)’:
QualityNeuralNetwork.cpp:70: warning: converting to ‘int’ from ‘float’

StatisticsMaps.cpp: In member function ‘void CStatisticsMaps::PrintMaps(const 
char*, const std::vector<MosaikReadFormat::ReadGroup, 
std::allocator<MosaikReadFormat::ReadGroup> >&, unsigned char)’:
StatisticsMaps.cpp:468: warning: comparison is always false due to limited 
range of data type
StatisticsMaps.cpp:481: warning: comparison is always false due to limited 
range of data type

AlignerMain.cpp: In function ‘int main(int, char**)’:
AlignerMain.cpp:452: warning: converting to ‘unsigned int’ from ‘double’
AlignerMain.cpp:456: warning: converting to ‘unsigned int’ from ‘double’
AlignerMain.cpp:504: warning: comparison is always false due to limited range 
of data type
AlignerMain.cpp:509: warning: comparison is always false due to limited range 
of data type

MosaikAligner.cpp: In member function ‘void 
CMosaikAligner::AlignReadArchiveLowMemory()’:
MosaikAligner.cpp:391: warning: converting to ‘unsigned int’ from 
‘double’
MosaikAligner.cpp:405: warning: converting to ‘unsigned int’ from 
‘double’
MosaikAligner.cpp: In member function ‘void 
CMosaikAligner::GetHashStatistics(std::vector<unsigned int, 
std::allocator<unsigned int> >&, std::vector<unsigned int, 
std::allocator<unsigned int> >&, uint64_t&, const 
std::vector<ReferenceSequence, std::allocator<ReferenceSequence> >&)’:
MosaikAligner.cpp:573: warning: passing ‘double’ for argument 14 to 
‘CJumpDnaHash::CJumpDnaHash(unsigned char, const std::string&, short unsigned 
int, bool, bool, unsigned int, unsigned int, unsigned int, unsigned int, 
unsigned int, bool, bool, uint64_t, unsigned int)’
MosaikAligner.cpp:591: warning: passing ‘double’ for argument 6 to ‘void 
CJumpDnaHash::GetHashStatistics(const std::vector<std::pair<unsigned int, 
unsigned int>, std::allocator<std::pair<unsigned int, unsigned int> > >&, 
std::vector<unsigned int, std::allocator<unsigned int> >&, std::vector<unsigned 
int, std::allocator<unsigned int> >&, const bool&, const unsigned int&, const 
unsigned int&)’
MosaikAligner.cpp: In member function ‘void 
CMosaikAligner::InitializeHashTables(unsigned char, unsigned int, unsigned int, 
unsigned int, bool, unsigned int, bool)’:
MosaikAligner.cpp:1426: warning: passing ‘double’ for argument 14 to 
‘CJumpDnaHash::CJumpDnaHash(unsigned char, const std::string&, short unsigned 
int, bool, bool, unsigned int, unsigned int, unsigned int, unsigned int, 
unsigned int, bool, bool, uint64_t, unsigned int)’
MosaikAligner.cpp:1431: warning: passing ‘double’ for argument 14 to 
‘CJumpDnaHash::CJumpDnaHash(unsigned char, const std::string&, short unsigned 
int, bool, bool, unsigned int, unsigned int, unsigned int, unsigned int, 
unsigned int, bool, bool, uint64_t, unsigned int)’
MosaikAligner.cpp:1436: warning: passing ‘double’ for argument 14 to 
‘CJumpDnaHash::CJumpDnaHash(unsigned char, const std::string&, short unsigned 
int, bool, bool, unsigned int, unsigned int, unsigned int, unsigned int, 
unsigned int, bool, bool, uint64_t, unsigned int)’

Original issue reported on code.google.com by torsten....@gmail.com on 9 Nov 2011 at 2:19

[GoogleCodeExporter]

Thanks for reporting these. In the newest, I tried to use more cruel options 
for compiling and reduce most of warnings.

Original comment by WanPing....@gmail.com on 22 Nov 2011 at 3:48

[GoogleCodeExporter]

Original comment by WanPing....@gmail.com on 22 Nov 2011 at 10:01

• Changed state: Done