- static bool first = true ; - if( first ){ - std::cout << " WARNING >>>>>>> MCParticleImpl::getNumberOfParents() is deprecated " - << " - please use MCParticleImpl::getParents().size() ! " << std::endl ; - first = false ; - }

+ //static bool first = true ; + //if( first ){ + // std::cout << " WARNING >>>>>>> MCParticleImpl::getNumberOfParents() is deprecated " + // << " - please use MCParticleImpl::getParents().size() ! " << std::endl ; + // first = false ; + //} + + UTIL::LCWarning::getInstance().printWarning( "MCPARTICLE_DEPRECATED_GETNUMBEROFPARENTS" ) ; +

- static bool first = true ; - if( first ){ - std::cout << " WARNING >>>>>>> MCParticleImpl::getParent(i) is deprecated " - << " - please use MCParticleImpl::getParents()[i] ! " << std::endl ; - first = false ; - }

+ //static bool first = true ; + //if( first ){ + // std::cout << " WARNING >>>>>>> MCParticleImpl::getParent(i) is deprecated " + // << " - please use MCParticleImpl::getParents()[i] ! " << std::endl ; + // first = false ; + //} + + UTIL::LCWarning::getInstance().printWarning( "MCPARTICLE_DEPRECATED_GETPARENT" ) ; +

- static bool first = true ; - if( first ){ - std::cout << " WARNING >>>>>>> MCParticleImpl::getNumberOfDaughters() is deprecated " - << " - please use MCParticleImpl::getDaughters().size() ! " << std::endl ; - first = false ; - }

+ //static bool first = true ; + //if( first ){ + // std::cout << " WARNING >>>>>>> MCParticleImpl::getNumberOfDaughters() is deprecated " + // << " - please use MCParticleImpl::getDaughters().size() ! " << std::endl ; + // first = false ; + //} + + UTIL::LCWarning::getInstance().printWarning( "MCPARTICLE_DEPRECATED_GETNUMBEROFDAUGHTERS" ) ; +

- static bool first = true ; - if( first ){ - std::cout << " WARNING >>>>>>> MCParticleImpl::getDaughter(i) is deprecated " - << " - please use MCParticleImpl::getDaughters()[i] ! " << std::endl ; - first = false ; - }

+ //static bool first = true ; + //if( first ){ + // std::cout << " WARNING >>>>>>> MCParticleImpl::getDaughter(i) is deprecated " + // << " - please use MCParticleImpl::getDaughters()[i] ! " << std::endl ; + // first = false ; + //} + + UTIL::LCWarning::getInstance().printWarning( "MCPARTICLE_DEPRECATED_GETDAUGHTER" ) ; +

- static bool first = true ; - if( first ){ - std::cout << " WARNING >>>>>>> SimCalorimeterHitImpl::getNMCParticles() is deprecated " - << " - please use SimCalorimeterHitImpl::getNMCContributions() ! " << std::endl ; - first = false ; - }

+ //static bool first = true ; + //if( first ){ + // std::cout << " WARNING >>>>>>> SimCalorimeterHitImpl::getNMCParticles() is deprecated " + // << " - please use SimCalorimeterHitImpl::getNMCContributions() ! " << std::endl ; + // first = false ; + //} + + UTIL::LCWarning::getInstance().printWarning( "SIMCALORIMETERHIT_DEPRECATED_GETNMCPARTICLES" ) ;

// remove float converstion and check what happens ;) MYTEST( trkHit->getEDepError() , float((i+j)*.3) , "EDepError" ) ; //MYTEST( trkHit->getEDepError() , (i+j)*.3 , "EDepError" ) ;

lcio/src/cpp/src/UTIL

LCWarning.cc added at 1.1

diff -N LCWarning.cc
--- /dev/null	1 Jan 1970 00:00:00 -0000
+++ LCWarning.cc	17 Jun 2010 12:29:00 -0000	1.1
@@ -0,0 +1,167 @@

+#include "UTIL/LCWarning.h"
+
+#include <iostream>
+#include "Exceptions.h"
+
+using namespace std;
+
+namespace UTIL{
+
+
+///// add new warnings here ===================================================
+LCWarning::LCWarning( ostream& outstream ) : _outstream(outstream) {
+
+
+    // ----- Warnings for SimTrackerHit ---------------------------------------
+    registerWarning(
+        "SIMTRACKERHIT_DEPRECATED_GETDEDX",
+        "used DEPRECATED method SimTrackerHitImpl::getdEdx()\n"
+        "please change your code to use SimTrackerHitImpl::getEDep() instead"
+    );
+
+    registerWarning(
+        "SIMTRACKERHIT_DEPRECATED_SETDEDX",
+        "used DEPRECATED method SimTrackerHitImpl::setdEdx()\n"
+        "please change your code to use SimTrackerHitImpl::setEDep() instead"
+    );
+    // ------------------------------------------------------------------------
+
+
+
+    // ----- Warnings for TrackerHit ------------------------------------------
+    registerWarning(
+        "TRACKERHIT_DEPRECATED_GETDEDX",
+        "used DEPRECATED method TrackerHitImpl::getdEdx()\n"
+        "please change your code to use TrackerHitImpl::getEDep()"
+    );
+
+    registerWarning(
+        "TRACKERHIT_DEPRECATED_SETDEDX",
+        "used DEPRECATED method TrackerHitImpl::setdEdx()\n"
+        "please change your code to use TrackerHitImpl::setEDep() instead."
+    );
+    // ------------------------------------------------------------------------
+
+
+
+    // ----- Warnings for SimCalorimeterHit ---------------------------------------
+    registerWarning(
+        "SIMCALORIMETERHIT_DEPRECATED_GETNMCPARTICLES",
+        "used DEPRECATED method SimCalorimeterHitImpl::getNMCParticles()\n"
+        "please change your code to use SimCalorimeterHitImpl::getNMCContributions() instead"
+    );
+    // ------------------------------------------------------------------------
+
+
+
+    // ----- Warnings for MCParticle ------------------------------------------
+    registerWarning(
+        "MCPARTICLE_DEPRECATED_GETNUMBEROFPARENTS",
+        "used DEPRECATED method MCParticleImpl::getNumberOfParents()\n"
+        "please change your code to use MCParticleImpl::getParents().size() instead."
+    );
+
+    registerWarning(
+        "MCPARTICLE_DEPRECATED_GETPARENT",
+        "used DEPRECATED method MCParticleImpl::getParent(i)\n"
+        "please change your code to use MCParticleImpl::getParents()[i] instead."
+    );
+
+    registerWarning(
+        "MCPARTICLE_DEPRECATED_GETNUMBEROFDAUGHTERS",
+        "used DEPRECATED method MCParticleImpl::getNumberOfDaughters()\n"
+        "please change your code to use MCParticleImpl::getDaughters().size() instead."
+    );
+
+    registerWarning(
+        "MCPARTICLE_DEPRECATED_GETDAUGHTER",
+        "used DEPRECATED method MCParticleImpl::getDaughter(i)\n"
+        "please change your code to use MCParticleImpl::getDaughters()[i] instead."
+    );
+    // ------------------------------------------------------------------------
+
+
+}
+///// =========================================================================
+
+
+
+void LCWarning::registerWarning( const string id, const string txt, int max ) {
+
+    if( _warning_cfg.find( id ) != _warning_cfg.end() ){
+        throw EVENT::Exception( std::string( "Warning [ "+id+" ] was already registered") ); 
+    }
+
+    _warning_cfg[ id ].txt = txt ;
+    _warning_cfg[ id ].max = max ;
+    _warning_cfg[ id ].counter = max ;
+
+    // _outstream << "registered warning " << id << "with max limit: " << max << endl;
+}
+
+
+
+void LCWarning::printWarning( const string id ) {
+
+    // if max limit has been reached for displaying the warning, just return and do nothing.
+    if( _warning_cfg[ id ].counter <= 0 ){ return ; }
+
+    _outstream << endl << endl
+         << endl << "=========== WARNING ========================================================="
+         << endl << _warning_cfg[ id ].txt;
+
+    if(  _warning_cfg[ id ].counter > 1 ){
+        _outstream << endl << "*** this warning will be printed (at most) "
+            << _warning_cfg[ id ].max << " time(s). [ "
+            << _warning_cfg[ id ].counter-1 << " time(s) left ]" ;
+    }
+    // max warnings have been reached.
+    else{
+        _outstream << endl << "*** the maximum amount of times for printing this warning has been reached."
+            << endl << "*** this warning will be printed one last time before the application exits." ;
+
+    }
+
+    _outstream << endl << "============================================================================="
+         << endl << endl ;
+
+    // decrease counter
+    _warning_cfg[ id ].counter-- ;
+}
+
+
+
+// destructor shows warnings one very last time before program ends
+LCWarning::~LCWarning(){
+
+    bool first_warning = true;
+
+    for( _warning_cfg_it = _warning_cfg.begin(); _warning_cfg_it != _warning_cfg.end(); _warning_cfg_it++ ){
+        if( (*_warning_cfg_it).second.max != (*_warning_cfg_it).second.counter ){
+            if( first_warning ){
+                _outstream << endl << endl
+                    << endl << "+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++"
+                    << endl << "+++ FOLLOWING WARNINGS WERE FOUND:"
+                    << endl << "+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++"
+                ;
+                first_warning = false ;
+            }
+            _outstream << endl << endl << _warning_cfg[ (*_warning_cfg_it).first ].txt ;
+        }
+    }
+    _outstream << endl << endl;
+}
+
+
+
+// singleton
+LCWarning& LCWarning::getInstance()
+{
+  static LCWarning instance;
+  return instance;
+}
+
+
+
+} // namespace UTIL
+

Commit in `lcio/src` on MAIN
`aid/EVENT/SimTrackerHit.aid`	+3	-2	1.10 -> 1.11
`/TrackerHit.aid`	+3	-2	1.15 -> 1.16
`cpp/CMakeLists.txt`	+1		1.28 -> 1.29
`cpp/include/IMPL/SimTrackerHitImpl.h`	+4	-2	1.13 -> 1.14
`/TrackerHitImpl.h`	+3	-2	1.14 -> 1.15
`cpp/src/EXAMPLE/simjob.cc`	+2	-1	1.58 -> 1.59
`cpp/src/IMPL/MCParticleImpl.cc`	+37	-24	1.29 -> 1.30
`/SimCalorimeterHitImpl.cc`	+9	-6	1.16 -> 1.17
`/SimTrackerHitImpl.cc`	+7		1.13 -> 1.14
`/TrackerHitImpl.cc`	+7		1.13 -> 1.14
`cpp/src/TESTS/test_trackerhit.cc`	+4	-2	1.2 -> 1.3
`cpp/src/UTIL/LCWarning.cc`	+167		added 1.1
	+247	-41