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

From: Axel Naumann (axel@fnal.gov)
Date: Wed Mar 19 2003 - 14:30:56 MET


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.



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