Message: Re: some changes in globals.hh in Geant4.10.0 Not Logged In (login)
 Next-in-Thread Next-in-Thread
 Next-in-Forum Next-in-Forum

News Re: some changes in globals.hh in Geant4.10.0 

Forum: Installation and Configuration
Re: None some changes in globals.hh in Geant4.10.0 (Fada Guan)
Date: 24 Dec, 2013
From: Michael H. Kelsey <Michael H. Kelsey>

On Tue, 24 Dec 2013 20:21:57 GMT, Fada Guan wrote:
> I installed the Geant4.10.0 recently and found some system units cannot
> be recognized even if I included the globals.hh.
> Later I checked this header, and found below lines.
> // Includes Physical Constants and System of Units
> // #include "G4PhysicalConstants.hh"
> // #include "G4SystemOfUnits.hh"
> I could not understand why these lines were commented out.

This was discussed in the Release Notes.

It's bad form (though found throughout older G4 code!) to rely on headers including other headers. It can cause side effects, increases unnecessary dependencies (so that code has to be recompiled even when it shouldn't need to be) and obscures the true dependencies of code.

These headers were removed from globals.hh, so that .cc files which actually do need to make use of either constants or units would explicitly #include whichever of those files they need.

  -- Michael Kelsey

Inline Depth:
 1 1
 All All
Outline Depth:
 1 1
 2 2
 All All
Add message: (add)

1 None: Re: some changes in globals.hh in Geant4.10.0   (Fada Guan - 27 Dec, 2013)
 Add Message Add Message
to: "Re: some changes in globals.hh in Geant4.10.0"

 Subscribe Subscribe

This site runs SLAC HyperNews version 1.11-slac-98, derived from the original HyperNews

[ Geant 4 Home | Geant 4 HyperNews | Search | Request New Forum | Feedback ]