Message: Re: the problem of SetRandomNumberStoreDir function in G4RunManager.hh Not Logged In (login)
 Next-in-Thread Next-in-Thread
 Next-in-Forum Next-in-Forum

More Re: the problem of SetRandomNumberStoreDir function in G4RunManager.hh 

Forum: Run Management
Re: None the problem of SetRandomNumberStoreDir function in G4RunManager.hh (Fada Guan)
Re: None Re: the problem of SetRandomNumberStoreDir function in G4RunManager.hh (Michael H. Kelsey)
Re: None Re: the problem of SetRandomNumberStoreDir function in G4RunManager.hh (Fada Guan)
Date: 14 Oct, 2010
From: Michael H. Kelsey <Michael H. Kelsey>

On Tue, 12 Oct 2010 15:21:18 GMT, Fada Guan wrote: [ Regarding the system() call in G4RunManager: ] > 2) Hope it will be fixed.

Hello again. I just realized that there is no actual bug in that code. You had reported:

> 2)system(shellCmd);
> 
> shellCmd is a string, but not a const char*, which is the type of the argument > in the system function shown below:
>
> int system ( const char * command ); the reference is:
> http://www.cplusplus.com/reference/clibrary/cstdlib/system/

I had responded to you about the string vs. char* issue. In fact shellCmd is of type G4String, which provides a large number of additional interface functions on top of its std::string base class. In particular, it provides "operator const char*()", which will do an automatic cast for the system() call you mention above.

The compiler warning you mentioned,

> /geant4/geant4.9.3.p02/source/run/include/G4RunManager.hh: In member function 
> ‘void G4RunManager::SetRandomNumberStoreDir(const G4String&)’: 
> geant4/geant4.9.3.p02/source/run/include/G4RunManager.hh:357: warning: 
> ignoring return value of ‘int system(const char*)’, declared with attribute 
> warn_unused_result

is caused by (as it says) disregaring the status-return from system().

Could you tell us which GCC version you are using?

In the latest versions, a number of "stylistic" warnings are generated, which rather exceed the standards of both the C and C++ languages (wanting extra parentheses with boolean operators, wanting empty braces on for-loops, and so on). The warning above sounds like more of the same.

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

1 None: Re: the problem of SetRandomNumberStoreDir function in G4RunManager.hh   (Fada Guan - 14 Oct, 2010)
 Add Message Add Message
to: "Re: the problem of SetRandomNumberStoreDir function in G4RunManager.hh"

 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 ]