Re: [ROOT] base <-> tree recursive dependency

From: Fons Rademakers (Fons.Rademakers@cern.ch)
Date: Wed Mar 19 2003 - 15:02:35 MET


Done. Also in TTree and TChain.

-- Fons


On Wed, 2003-03-19 at 14:30, Axel Naumann wrote:
> Hi Rene,
> Thanks. I just realized this change also contains inefficient code
> ("illegal c++" :-]). Could you please replace
> 
> base/inc/TDSet.h:   virtual Int_t Draw(const char *varexp, TCut selection,
> by
> virtual Int_t Draw(const char *varexp, const TCut& selection,
> 
> Cheers, Axel.
> 
> Some time ago, Rene Brun decided to write:
> > Hi Axel,
> >
> > This problem is now fixed in CVS.
> > The class TDSet has been moved to the directory "tree"
> >
> > Thanks for reporting the problem.
> >
> > Rene Brun
> >
> > Axel Naumann wrote:
> >>
> >> Hi,
> >>
> >> since today's cvs there is a recursive dependency between the base and
> >> tree modules. This is not essential for most platforms (which allow
> >> undefined symbols for shared libs), but it shows up on win (I tested
> >> on the unpublished "win32gcc").
> >>
> >> $ grep TCut base/inc/*.h # [modified]
> >> base/inc/TDSet.h:#include "TCut.h"
> >> base/inc/TDSet.h:   virtual Int_t Draw(const char *varexp, TCut
> >> selection,
> >>
> >> The linker needs to resolve TCut symbols to be able to build libCore,
> >> which it can't, because libTree is not built. But libTree can only be
> >> built if libCore exists.
> >>
> >> Cheers, Axel.
-- 
Org:    CERN, European Laboratory for Particle Physics.
Mail:   1211 Geneve 23, Switzerland
E-Mail: Fons.Rademakers@cern.ch              Phone: +41 22 7679248
WWW:    http://root.cern.ch/~rdm/            Fax:   +41 22 7679480



This archive was generated by hypermail 2b29 : Thu Jan 01 2004 - 17:50:10 MET