Skip to content

Commit

Permalink
Change from int to StarMagField::EBField type (#605)
Browse files Browse the repository at this point in the history
Resolves a run-time error seen in tests of compiling software for ROOT
6.16.00. I also tested this fix in ROOT 5.34.38 to be sure it doesn't
break it.

-Gene
____________________

    root4star -b -q -l 'bfc.C(-1)'

...

QA :INFO - ProcessLine if (!StarMagField::Instance()) new StarMagField(
2, 1.000000, kTRUE);
input_line_668:2:37: error: no matching constructor for initialization
of 'StarMagField'
 if (!StarMagField::Instance()) new StarMagField( 2, 1.000000, kTRUE);
                                    ^             ~~~~~~~~~~~~~~~~~~

/afs/rhic.bnl.gov/star/packages/SL23x/.sl73_x8664_gcc485/include/StarMagField.h:155:3:
note: candidate constructor not viable: no known conversion from 'int'
to 'StarMagField::EBField' for 1st argument
  StarMagField ( EBField map     = kMapped, Float_t Factor  =      1, 
  ^

/afs/rhic.bnl.gov/star/packages/SL23x/.sl73_x8664_gcc485/include/StarMagField.h:91:7:
note: candidate constructor (the implicit copy constructor) not viable:
requires 1 argument, but 3 were provided
class StarMagField : public TVirtualMagField 
      ^
BFC:ERROR - StBFChain::ProcessLine command:if
(!StarMagField::Instance()) new StarMagField( 2, 1.000000, kTRUE); has
failed. Quit job.
  • Loading branch information
genevb authored Oct 13, 2023
1 parent 07909ba commit ae51606
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion StRoot/StBFChain/StBFChain.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -1918,7 +1918,7 @@ void StBFChain::SetDbOptions(StMaker *mk){
<< "\",\"MagFactor\")" << endm;
if ( gClassTable->GetID("StarMagField") >= 0) {
TString cmd =
Form("if (!StarMagField::Instance()) new StarMagField( 2, %f, kTRUE);",
Form("if (!StarMagField::Instance()) new StarMagField( StarMagField::EBField::kMapped, %f, kTRUE);",
FieldOptions[k].scale);
ProcessLine(cmd);
}
Expand Down

0 comments on commit ae51606

Please sign in to comment.