libAliHLTITS added to HLT build system; minor compilation warnings removed
authorrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 6 Mar 2008 10:26:44 +0000 (10:26 +0000)
committerrichterm <richterm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 6 Mar 2008 10:26:44 +0000 (10:26 +0000)
18 files changed:
HLT/ITS/AliHLTITSLinkDef.h [deleted file]
HLT/ITS/AliHLTITSVertexerZ.cxx
HLT/ITS/AliHLTITSVertexerZ.h
HLT/ITS/AliHLTITSclusterer.cxx
HLT/ITS/AliHLTITSclusterer.h
HLT/ITS/AliHLTITStrack.cxx
HLT/ITS/AliHLTITStrack.h
HLT/ITS/AliHLTITStracker.cxx
HLT/ITS/AliHLTITStracker.h
HLT/ITS/Makefile.am [new file with mode: 0644]
HLT/Makefile.am
HLT/SampleLib/AliHLTSampleMonitoringComponent.cxx
HLT/SampleLib/AliHLTSamplePreprocessor.cxx
HLT/TPCLib/AliHLTTPCConfMapper.cxx
HLT/TPCLib/AliHLTTPCConfMapper.h
HLT/configure.ac
HLT/hlt.conf
HLT/libAliHLTITS.pkg

diff --git a/HLT/ITS/AliHLTITSLinkDef.h b/HLT/ITS/AliHLTITSLinkDef.h
deleted file mode 100644 (file)
index 26b0ba9..0000000
+++ /dev/null
@@ -1,15 +0,0 @@
-
-#ifdef __CINT__
-#pragma link off all globals;
-#pragma link off all classes;
-#pragma link off all functions;
-
-#ifdef use_aliroot
-#pragma link C++ class AliHLTITStrack; 
-#pragma link C++ class AliHLTITStracker; 
-#pragma link C++ class AliHLTITSVertexerZ;
-#pragma link C++ class AliHLTITSclusterer;
-#endif
-
-#endif
index 3901443..466ab8e 100644 (file)
@@ -1,3 +1,5 @@
+// $Id$
+
 /**************************************************************************
  * Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
  *                                                                        *
index 0dfde31..7375db7 100644 (file)
@@ -1,5 +1,8 @@
-#ifndef ALIL3ITSVERTEXERZ_H
-#define ALIL3ITSVERTEXERZ_H
+//-*- Mode: C++ -*-
+// $Id$
+
+#ifndef ALIHLTITSVERTEXERZ_H
+#define ALIHLTITSVERTEXERZ_H
 /* Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
  * See cxx source for full Copyright notice                               */
 
index 54e08d8..3cb408b 100644 (file)
@@ -1,3 +1,5 @@
+// $Id$
+
 /**************************************************************************
  * Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
  *                                                                        *
 
 ClassImp(AliHLTITSclusterer)
 
-AliHLTITSclusterer::AliHLTITSclusterer(const Char_t *geom):AliITSclustererV2(geom)
+AliHLTITSclusterer::AliHLTITSclusterer(const Char_t *geom)
+  :
+  AliITSclustererV2(geom),
+  fNModule(AliITSgeomTGeo::GetNModules())  
 {
-fNModule = AliITSgeomTGeo::GetNModules();
 }
 
 void AliHLTITSclusterer::Digits2Clusters(AliRawReader* rawReader,TTree *cTree)
index d3a56d7..0a4c47a 100644 (file)
@@ -1,5 +1,8 @@
-#ifndef ALIL3ITSCLUSTERER_H
-#define ALIL3ITSCLUSTERER_H
+//-*- Mode: C++ -*-
+// $Id$
+
+#ifndef ALIHLTITSCLUSTERER_H
+#define ALIHLTITSCLUSTERER_H
 /* Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
  * See cxx source for full Copyright notice                               */
 
@@ -20,7 +23,7 @@ class AliRawReader;
 //-------------------------------------------------------------------------
 class AliHLTITSclusterer : public AliITSclustererV2 {
 public:
-  AliHLTITSclusterer():AliITSclustererV2(){fNModule = 0;}
+  AliHLTITSclusterer():AliITSclustererV2(), fNModule(0) {}
   AliHLTITSclusterer(const Char_t *geom);
 
   void Digits2Clusters(AliRawReader* rawReader,TTree *cTree);
index ac0b195..94f8b15 100644 (file)
@@ -1,3 +1,5 @@
+// $Id$
+
 /**************************************************************************
  * Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
  *                                                                        *
index 2a29c72..22c14c9 100644 (file)
@@ -1,5 +1,8 @@
-#ifndef ALIL3ITSTRACK_H
-#define ALIL3ITSTRACK_H
+//-*- Mode: C++ -*-
+// $Id$
+
+#ifndef ALIHLTITSTRACK_H
+#define ALIHLTITSTRACK_H
 /* Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
  * See cxx source for full Copyright notice                               */
 
index 363c6e0..711ef0f 100644 (file)
@@ -1,3 +1,5 @@
+// $Id$
+
 /**************************************************************************
  * Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
  *                                                                        *
index edd4f08..e920ce1 100644 (file)
@@ -1,5 +1,8 @@
-#ifndef ALIL3ITSTRACKER_H
-#define ALIL3ITSTRACKER_H
+//-*- Mode: C++ -*-
+// $Id$
+
+#ifndef ALIHLTITSTRACKER_H
+#define ALIHLTITSTRACKER_H
 /* Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
  * See cxx source for full Copyright notice                               */
 
diff --git a/HLT/ITS/Makefile.am b/HLT/ITS/Makefile.am
new file mode 100644 (file)
index 0000000..20bb3b6
--- /dev/null
@@ -0,0 +1,65 @@
+# $Id$
+# Makefile template for the AliHLTITS library.
+
+# be aware of the two different meanings of 'MODULE'
+# 1. AliRoot classifies each detector and main sub-package as a module
+#    so for all HLT library packages MODULE is equal HLT
+# 2. The HLT stand-alone build system knows about sub-modules
+#    e.g. for libAliHLTITS, MODULE=AliHLTITS
+MODULE                         = AliHLTITS
+
+SUBDIRS                        =
+
+EXTRA_DIST                     =
+
+# library definition
+lib_LTLIBRARIES                        =  libAliHLTITS.la
+
+# version info for the library
+LIBRARY_VERSION                        = '0:0:0'
+
+# MODDIR is set by the AliRoot build system and denotes the topdir
+# of the module, we must set it since the package definition libAliHLTITS.pkg
+# includes another common configuration file
+MODDIR                         = $(top_srcdir)
+PKGDEF                         = $(MODDIR)/libAliHLTITS.pkg
+include $(top_srcdir)/libAliHLTITS.pkg
+
+# compiler flags
+AM_CPPFLAGS                    = -DMODULE=$(MODULE)         \
+                                 $(foreach i, $(EINCLUDE),  \
+                                       $(shell echo $(i) | sed -e "/HLT\//!d" -e "s|HLT/|-I$(top_srcdir)/|")) \
+                                 $(foreach i, $(EINCLUDE),  \
+                                       $(shell echo $(i) | sed -e "/HLT\//d" -e "s|^|-I$(ALICE_ROOT)/|"))
+
+# library sources
+libAliHLTITS_la_SOURCES        = $(MODULE_SRCS)
+
+# library headers
+pkginclude_HEADERS             = $(MODULE_HDRS)
+
+# linker flags
+libAliHLTITS_la_LDFLAGS                = -L@ROOTLIBDIR@ \
+                                 @ROOTLIBS@ \
+                                 @ALIROOT_LDFLAGS@ \
+                                 @ALIROOT_LIBS@ \
+                                 -version-info $(LIBRARY_VERSION)
+
+# automatic generation of data and time of library build
+COMPILE_INFO                   =  AliHLTITSCompileInfo.cxx
+
+# set the file name for the generated root dictionary
+DICTCPP                                =  AliHLTITS-DICT.cxx
+nodist_libAliHLTITS_la_SOURCES =  $(COMPILE_INFO) \
+                                  $(DICTCPP)
+
+CLEANFILES                     =  $(COMPILE_INFO)
+
+include $(top_srcdir)/make.dict
+
+$(COMPILE_INFO): $(libAliHLTITS_la_SOURCES) $(pkginclude_HEADERS) $(noinst_HEADERS) Makefile.am
+       @echo '//automatically generated compilation info' > $@
+       @echo '//!!! DO NOT EDIT THIS FILE !!!' >> $@
+       @echo '//add changes in Makefile.am' >> $@
+       @echo 'extern "C" void CompileInfo( char*& date, char*& time)' >> $@
+       @echo '{date=__DATE__; time=__TIME__; return;}' >> $@
index 9232e6c..f2a8408 100644 (file)
@@ -30,9 +30,13 @@ if EN_HLT_COMP
 COMP_DIR=comp
 endif
 
-#if EN_HLT_TRIGGER
+if EN_HLT_TRIGGER
 TRIGGER_DIR=trigger
-#endif
+endif
+
+if EN_HLT_ITS
+ITS_DIR=ITS
+endif
 
 SUBDIRS                = BASE \
                          sim \
@@ -46,6 +50,7 @@ SUBDIRS               = BASE \
                          $(MUON_DIR) \
                          $(COMP_DIR) \
                           $(TRIGGER_DIR) \
+                          $(ITS_DIR) \
                          doc
 
 EXTRA_DIST             = libHLTbase.pkg \
index 812ce4c..789d677 100644 (file)
@@ -143,7 +143,7 @@ int AliHLTSampleMonitoringComponent::DoDeinit()
   return 0;
 }
 
-int AliHLTSampleMonitoringComponent::DoEvent( const AliHLTComponentEventData& evtData, AliHLTComponentTriggerData& /*trigData*/)
+int AliHLTSampleMonitoringComponent::DoEvent( const AliHLTComponentEventData& /*evtData*/, AliHLTComponentTriggerData& /*trigData*/)
 {
   // see header file for class documentation
 
@@ -226,7 +226,7 @@ int AliHLTSampleMonitoringComponent::Configure(const char* arguments)
   return iResult;
 }
 
-int AliHLTSampleMonitoringComponent::Reconfigure(const char* cdbEntry, const char* chainId)
+int AliHLTSampleMonitoringComponent::Reconfigure(const char* cdbEntry, const char* /*chainId*/)
 {
   // see header file for class documentation
   int iResult=0;
index 90714ef..48b1ef8 100644 (file)
@@ -58,7 +58,7 @@ void AliHLTSamplePreprocessor::Initialize(Int_t /*run*/, UInt_t /*startTime*/,
 }
 
 
-UInt_t AliHLTSamplePreprocessor::Process(TMap* dcsAliasMap)
+UInt_t AliHLTSamplePreprocessor::Process(TMap* /*dcsAliasMap*/)
 {
   // see header file for function documentation
   UInt_t retVal = 0;
index 417ddd4..0e10dc6 100644 (file)
@@ -1,16 +1,27 @@
 // @(#) $Id$
 // Original: AliHLTConfMapper.cxx,v 1.26 2005/06/14 10:55:21 cvetan Exp $
 
-/** \class AliHLTTPCConfMapper
-<pre>
-//_____________________________________________________________
-// AliHLTTPCConfMapper
-//
-// Conformal mapping base class
-//
-// Author: Anders Vestbo <mailto:vestbo@fi.uib.no>
-// Copyright &copy ALICE HLT Group
-</pre>
+//**************************************************************************
+//* This file is property of and copyright by the ALICE HLT Project        * 
+//* ALICE Experiment at CERN, All rights reserved.                         *
+//*                                                                        *
+//* Primary Authors: Anders Vestbo, maintained by
+//*                  Matthias Richter <Matthias.Richter@ift.uib.no>        *
+//*                  for The ALICE HLT Project.                            *
+//*                                                                        *
+//* Permission to use, copy, modify and distribute this software and its   *
+//* documentation strictly for non-commercial purposes is hereby granted   *
+//* without fee, provided that the above copyright notice appears in all   *
+//* copies and that both the copyright notice and this permission notice   *
+//* appear in the supporting documentation. The authors make no claims     *
+//* about the suitability of this software for any purpose. It is          *
+//* provided "as is" without express or implied warranty.                  *
+//**************************************************************************
+
+/** @file   AliHLTAltroEncoder.cxx
+    @author Anders Vestbo, Matthias Richter
+    @date   Conformal mapping base class.
+    @brief  
 */
 
 #include <cassert>
@@ -829,9 +840,9 @@ Double_t AliHLTTPCConfMapper::TrackletAngle(AliHLTTPCConfMapTrack *track,Int_t n
   if(n<3)
     return 0;
   
-  Double_t x1[2];
-  Double_t x2[2];
-  Double_t x3[2];
+  Double_t x1[2]={0,0};
+  Double_t x2[2]={0,0};
+  Double_t x3[2]={0,0};
   Double_t angle1,angle2;
   Int_t counter=0;
   for(track->StartLoop(); track->LoopDone(); track->GetNextHit())
index a4e7104..70e16cc 100644 (file)
@@ -3,12 +3,15 @@
 
 #ifndef ALIHLTTPCCONFMAPPER_H
 #define ALIHLTTPCCONFMAPPER_H
+//* This file is property of and copyright by the ALICE HLT Project        * 
+//* ALICE Experiment at CERN, All rights reserved.                         *
+//* See cxx source for full Copyright notice                               *
 
-//
-//Conformal mapping base class
-//
-// Author: Anders Vestbo <mailto:vestbo@fi.uib.no>
-//*-- Copyright &copy ALICE HLT Group
+/** @file   AliHLTAltroEncoder.h
+    @author Anders Vestbo, Matthias Richter
+    @date   Conformal mapping base class.
+    @brief  
+*/
 
 #include <vector>
 
@@ -19,6 +22,10 @@ class AliHLTTPCTrackArray;
 class AliHLTTPCSpacePointData;
 
 
+/** 
+ * @class AliHLTTPCConfMapper
+ * Conformal mapping base class.
+ */
 class AliHLTTPCConfMapper {
 
  public:
index cf79e0d..3fbb639 100644 (file)
@@ -449,7 +449,7 @@ AC_ARG_ENABLE(strict,
       [disable coding convention checks ])],
   [],[enable_strict=yes])
 if test "x$enable_strict" = "xyes" ; then
-   CPPFLAGS="$CPPFLAGS -Weffc++ -Wall"
+   CPPFLAGS="$CPPFLAGS -W -Weffc++ -Wall"
 fi
 AC_MSG_RESULT([$enable_strict])
 
@@ -519,7 +519,7 @@ if test "x$enable_tpc" = "xyes" ; then
   AC_DEFINE(HLT_TPC)
 fi
 AM_CONDITIONAL(EN_HLT_TPC, test x$enable_tpc = xyes)
-AC_MSG_CHECKING([whether to compile tpc library])
+AC_MSG_CHECKING([whether to compile TPC library])
 AC_MSG_RESULT([$enable_module])
 AC_SUBST([ALITPC_LIBS])
 
@@ -565,7 +565,7 @@ if test "x$enable_rcu" = "xyes" ; then
   AC_DEFINE(HLT_RCU)
 fi
 AM_CONDITIONAL(EN_HLT_RCU, test x$enable_rcu = xyes)
-AC_MSG_CHECKING([whether to compile rcu library])
+AC_MSG_CHECKING([whether to compile RCU library])
 AC_MSG_RESULT([$enable_module])
 AC_SUBST([ALIRCU_LIBS])
 
@@ -615,7 +615,7 @@ if test "x$enable_phos" = "xyes" ; then
   ALIPHOS_LIBS=$ALIHLTMODULE_LIBS
 fi
 AM_CONDITIONAL(EN_HLT_PHOS, test x$enable_phos = xyes)
-AC_MSG_CHECKING([whether to compile phos library])
+AC_MSG_CHECKING([whether to compile PHOS library])
 AC_MSG_RESULT([$enable_module])
 AC_SUBST([ALIPHOS_LIBS])
 
@@ -657,7 +657,7 @@ if test "x$enable_trd" = "xyes" ; then
   ALITRD_LIBS=$ALIHLTMODULE_LIBS
 fi
 AM_CONDITIONAL(EN_HLT_TRD, test x$enable_trd = xyes)
-AC_MSG_CHECKING([whether to compile trd library])
+AC_MSG_CHECKING([whether to compile TRD library])
 AC_MSG_RESULT([$enable_module])
 AC_SUBST([ALITRD_LIBS])
 
@@ -686,11 +686,67 @@ if test "x$enable_muon" = "xyes" ; then
   ALIMUON_LIBS=$ALIHLTMODULE_LIBS
 fi
 AM_CONDITIONAL(EN_HLT_MUON, test x$enable_muon = xyes)
-AC_MSG_CHECKING([whether to compile muon library])
+AC_MSG_CHECKING([whether to compile MUON library])
 AC_MSG_RESULT([$enable_module])
 AC_SUBST([ALIMUON_LIBS])
 
 dnl ------------------------------------------------------------------
+AH_TEMPLATE([HLT_TRIGGER],[hlt trigger library])
+
+ALITRIGGER_LIBS=
+CHECK_HLTMODULE([trigger],
+               [], [$ALIROOT_CPPFLAGS],
+               [], [-L$ROOTLIBDIR], [$ROOTLIBS $ADD_ROOTLIBS],
+               [], [-L$ROOTLIBDIR $ALIROOT_LDFLAGS], 
+               [$ROOTLIBS $ADD_ROOTLIBS $ALIROOT_LIBS])
+
+if test "x$enable_module" = "xmissheader"; then
+  enable_module="no...header.missing"
+  enable_trigger="no...header.missing"
+elif test "x$enable_module" = "xforce"; then
+  enable_trigger="yes"
+else
+  enable_trigger=$enable_module
+fi
+
+if test "x$enable_trigger" = "xyes" ; then 
+  AC_DEFINE(HLT_TRIGGER)
+  ALITRIGGER_LIBS=$ALIHLTMODULE_LIBS
+fi
+AM_CONDITIONAL(EN_HLT_TRIGGER, test x$enable_trigger = xyes)
+AC_MSG_CHECKING([whether to compile Trigger library])
+AC_MSG_RESULT([$enable_module])
+AC_SUBST([ALITRIGGER_LIBS])
+
+dnl ------------------------------------------------------------------
+AH_TEMPLATE([HLT_ITS],[hlt its library])
+
+ALIITS_LIBS=
+CHECK_HLTMODULE([its],
+               [], [$ALIROOT_CPPFLAGS],
+               [], [-L$ROOTLIBDIR], [$ROOTLIBS $ADD_ROOTLIBS],
+               [], [-L$ROOTLIBDIR $ALIROOT_LDFLAGS], 
+               [$ROOTLIBS $ADD_ROOTLIBS $ALIROOT_LIBS])
+
+if test "x$enable_module" = "xmissheader"; then
+  enable_module="no...header.missing"
+  enable_its="no...header.missing"
+elif test "x$enable_module" = "xforce"; then
+  enable_its="yes"
+else
+  enable_its=$enable_module
+fi
+
+if test "x$enable_its" = "xyes" ; then 
+  AC_DEFINE(HLT_ITS)
+  ALIITS_LIBS=$ALIHLTMODULE_LIBS
+fi
+AM_CONDITIONAL(EN_HLT_ITS, test x$enable_its = xyes)
+AC_MSG_CHECKING([whether to compile ITS library])
+AC_MSG_RESULT([$enable_module])
+AC_SUBST([ALIITS_LIBS])
+
+dnl ------------------------------------------------------------------
 AH_TEMPLATE([HLT_COMP],[hlt comp library])
 
 ALICOMP_LIBS=
@@ -829,6 +885,7 @@ AC_CONFIG_FILES([Makefile
                 TRD/Makefile
                 MUON/Makefile
                 trigger/Makefile
+                ITS/Makefile
                 comp/Makefile
                 doc/Makefile
                 doc/doxygen.conf
index dfe4e7d..36f5f05 100644 (file)
@@ -7,9 +7,9 @@ HLTDEFS     := -Duse_aliroot -Duse_root \
                -DALIROOTVERSION=\"unchecked\" \
               -D__ROOT__ -DUSE_ALILOG -DLINUX
 
-HLTCXXFLAGS := $(filter-out -ansi,$(filter-out -pedantic-errors, $(CXXFLAGS)))
-HLTCFLAGS   := $(filter-out -ansi, $(filter-out -pedantic-errors, $(CFLAGS)))
-HLTDCXXFLAGS:= $(filter-out -pedantic-errors, $(CXXFLAGS))
+HLTCXXFLAGS := $(filter-out -ansi, $(filter-out -pedantic-errors, $(filter-out -Woverloaded-virtual, $(CXXFLAGS))))
+HLTCFLAGS   := $(filter-out -ansi, $(filter-out -pedantic-errors, $(filter-out -Woverloaded-virtual, $(CFLAGS))))
+HLTDCXXFLAGS:= $(filter-out -pedantic-errors, $(filter-out -Woverloaded-virtual, $(CXXFLAGS)))
 
 ifdef ALIDEBUG
 HLTDEFS += -DDEBUG
index 505df7b..058e28e 100644 (file)
@@ -1,16 +1,28 @@
 #-*- Mode: Makefile -*-
+# $Id$
 
-include HLT/hlt.conf
+include $(MODDIR)/hlt.conf
 
-SRCS:= ITS/AliHLTITStrack.cxx ITS/AliHLTITStracker.cxx ITS/AliHLTITSVertexerZ.cxx ITS/AliHLTITSclusterer.cxx
+CLASS_HDRS:=   AliHLTITStrack.h \
+               AliHLTITStracker.h \
+               AliHLTITSVertexerZ.h \
+               AliHLTITSclusterer.h
 
-HDRS:= $(SRCS:.cxx=.h) $(HLTEHDRS)
+MODULE_SRCS=   $(CLASS_HDRS:.h=.cxx)
 
-DHDR:= ITS/AliHLTITSLinkDef.h
+MODULE_HDRS:=  $(CLASS_HDRS)
+
+MODULE_DHDR:=          
+
+SRCS:=$(patsubst %,ITS/%,$(MODULE_SRCS))
+CINTHDRS:=$(patsubst %,ITS/%,$(CLASS_HDRS))
+HDRS:=$(patsubst %,ITS/%,$(MODULE_HDRS))
+DHDR:=$(patsubst %,ITS/%,$(MODULE_DHDR))
+CINTAUTOLINK:= $(shell test "x$(MODULE_DHDR)" = "x" && echo 1)
 
 EDEFINE      := ${HLTDEFS}
 PACKCXXFLAGS := ${HLTCXXFLAGS}
 PACKCFLAGS   := ${HLTCLFAGS}
 PACKDCXXFLAGS:= ${HLTDCXXFLAGS}
 
-EINCLUDE += HLT/src HLT/hough HLT/comp HLT/misc HLT/ITS HLT/MUON/src TPC ITS MUON STEER RAW
+EINCLUDE := HLT/BASE HLT/ITS HLT/MUON/src TPC ITS MUON STEER RAW