Review: Approve Tested merge: OK (no conflicts) Tested compilation: problems
Compiling idtelescopes/KVIDSiCorrCsI_e503.cpp... idtelescopes/KVIDSiCorrCsI_e503.cpp: In member function ‘virtual Double_t KVIDSiCorrCsI_e503::GetIDMapX(Option_t*)’: idtelescopes/KVIDSiCorrCsI_e503.cpp:98:15: warning: variable ‘tmp’ set but not used [-Wunused-but-set-variable] idtelescopes/KVIDSiCorrCsI_e503.cpp: In member function ‘virtual Double_t KVIDSiCorrCsI_e503::GetIDMapY(Option_t*)’: idtelescopes/KVIDSiCorrCsI_e503.cpp:109:15: warning: variable ‘tmp’ set but not used [-Wunused-but-set-variable] idtelescopes/KVIDSiCorrCsI_e503.cpp: In member function ‘virtual Bool_t KVIDSiCorrCsI_e503::Identify(KVIdentificationResult*, Double_t, Double_t)’: idtelescopes/KVIDSiCorrCsI_e503.cpp:147:14: warning: variable ‘xVar’ set but not used [-Wunused-but-set-variable] idtelescopes/KVIDSiCorrCsI_e503.cpp:148:14: warning: variable ‘yVar’ set but not used [-Wunused-but-set-variable] idtelescopes/KVIDSiCorrCsI_e503.cpp: In member function ‘virtual Bool_t KVIDSiCorrCsI_e503::SetIdentificationParameters(const KVMultiDetArray*)’: idtelescopes/KVIDSiCorrCsI_e503.cpp:314:5: error: ‘ifstream’ was not declared in this scope idtelescopes/KVIDSiCorrCsI_e503.cpp:314:5: note: suggested alternative: /usr/include/c++/4.6/iosfwd:159:34: note: ‘std::ifstream’ idtelescopes/KVIDSiCorrCsI_e503.cpp:314:14: error: expected ‘;’ before ‘datfile’ idtelescopes/KVIDSiCorrCsI_e503.cpp:315:5: error: ‘datfile’ was not declared in this scope idtelescopes/KVIDSiCorrCsI_e503.cpp:339:13: error: ‘stringstream’ was not declared in this scope idtelescopes/KVIDSiCorrCsI_e503.cpp:339:13: note: suggested alternative: /usr/include/c++/4.6/iosfwd:153:37: note: ‘std::stringstream’ idtelescopes/KVIDSiCorrCsI_e503.cpp:339:26: error: expected ‘;’ before ‘name’ idtelescopes/KVIDSiCorrCsI_e503.cpp:340:13: error: ‘name’ was not declared in this scope idtelescopes/KVIDSiCorrCsI_e503.cpp:340:45: error: ‘setfill’ was not declared in this scope idtelescopes/KVIDSiCorrCsI_e503.cpp:340:45: note: suggested alternative: /usr/include/c++/4.6/iomanip:164:5: note: ‘std::setfill’ idtelescopes/KVIDSiCorrCsI_e503.cpp:340:56: error: ‘setw’ was not declared in this scope idtelescopes/KVIDSiCorrCsI_e503.cpp:340:56: note: suggested alternative: /usr/include/c++/4.6/iomanip:224:3: note: ‘std::setw’ idtelescopes/KVIDSiCorrCsI_e503.cpp:362:17: error: ‘string’ was not declared in this scope idtelescopes/KVIDSiCorrCsI_e503.cpp:362:17: note: suggested alternative: /usr/include/c++/4.6/bits/stringfwd.h:65:33: note: ‘std::string’ idtelescopes/KVIDSiCorrCsI_e503.cpp:362:24: error: expected ‘;’ before ‘thisLine’ idtelescopes/KVIDSiCorrCsI_e503.cpp:364:38: error: ‘thisLine’ was not declared in this scope make[1]: *** [idtelescopes/KVIDSiCorrCsI_e503.o] Error 1 make[1]: Leaving directory `/home/john/software/sources/kaliveda/merge_wigg_indraid/KVIndra' make: *** [Indra] Error 2 I added the missing "using namespace std;" declaration in KVIDSiCorrCsI_e503.cpp, you'll need to do the same if you want to compile with the latest versions of ROOT. -- https://code.launchpad.net/~peter-wigg-314159/kaliveda/kaliveda-indraid/+merge/129454 Your team KaliVeda Development Team is subscribed to branch lp:kaliveda. _______________________________________________ Mailing list: https://launchpad.net/~kaliveda-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~kaliveda-dev More help : https://help.launchpad.net/ListHelp
