Hi all,
I bumped the revision, and tagged CVS:
Version: 1.15/4
Tag: BRAT-1-15-4
Changes:
* Added method BrDigDC::Compare(const TObject*) const for ROOT 3.
* Added Classes BrSiCent, BrSiCentCalModule, BrSiCentModule, completly
parallel to the similar classes for TMA.
* Renamed classes BrCent, BrCentCalibration and BrCentModule to
BrMultCent, BrMultCentCalibration, BrMultCentModule, as suggested in
Steve's previous mail. Changed table name of BrMultCent
accordingly.
* Added a histogram to BrMultCentModule.
* In BrTileRdoModule, changed konstant kNBICorrection to
kHitCorrection, since it's more descriptive.
* Added kHitCorrection to BrSiRdoModule.
* BrModule::Init(), BrModule::Begin(), BrModule::End(), and
BrModule::Finish() now sets the stat appropiatly.
* Turned on automatic streamer generation for BrMRSTrack. Appreantly
Flemming took out the hand-written streamer and forgot to put in the
automatic one.
* Updated BrCentFilter to also filter on Si and Mult centralities.
Some coding isssues that I've mention a trizilion times before but no
one seems to listen to:
* You should not put a semi-colon after an inline method. That is
class BrFoo {
private:
Int_t fFoo;
public:
Int_t GetFoo() const { return fFoo; };
};
^
|
---------------------------------------+
should be
class BrFoo {
private:
Int_t fFoo;
public:
Int_t GetFoo() const { return fFoo; }
};
^
|
---------------------------------------+
Just like you don't put ';' after a method defintion.
* Please remember to put $Id$ in the begning of a file, and $Log$ at
the end of a file
// -*- mode: c++ -*- Tell Emacs this is C++ mode, not C mode
//
// $Id$
//
#ifndef BRAT_BrFoo
#define BRAT_BrFoo
#ifndef ROOT_TObject
#include <TObject.h> // Use "<...>" in headers! Rational, see GCC doc
#endif
class BrFoo : public TObject
{
private:
Int_t fFoo;
public:
BrFoo(Int_t foo);
Int_t GetFoo() const { return fFoo; }
ClassDef(BrFoo, 0) // Foo class i.e., nothing at all
};
#endif
//
// $Log$
//
In implmentation files, you should also add $Author$, $Date$ and
$Copyright$ for the HTML documntation
//____________________________________________________________________
//
// This class is only for illustration purposes. In this part of
// the file you _must_ docuemtent your class!
// Really, you _must_! It's bloody annoying not to have the
// documentation avaliable!
//
//
// $Id$
// $Author$
// $Date$
// $Copyright: (C) 2001 BRAHMS Collaboration <brahmlib@rhic.bnl.gov>
//
#ifndef BRAT_BrFoo
#include "BrFoo.h"
#endif
//____________________________________________________________________
ClassImp(BrFoo); // ';' not needed but recommended
//____________________________________________________________________
BrFoo::BrFoo(Int_t foo)
{
// CTOR of this really dumb class. Argument foo is the value of
// this object.
// Just set the internal variable
fFoo = foo;
}
//
// $Log$
//
* In modules, having a few (no more then say 20) histograms for quick
checks, etc. is a very good idea.
Yours,
Christian -----------------------------------------------------------
Holm Christensen Phone: (+45) 35 35 96 91
Sankt Hansgade 23, 1. th. Office: (+45) 353 25 305
DK-2200 Copenhagen N Web: www.nbi.dk/~cholm
Denmark Email: cholm@nbi.dk
This archive was generated by hypermail 2b29 : Thu Apr 19 2001 - 11:57:17 EDT