id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
23	MMatrix does not compile	smueller	Thomas	"This is an email from Thomas sent on FACT Online on Sat 2014-Dec-20:

Dear colleagues,

since I always get loud complains if something in the svn doesn't compile, I take the freedom to complain as well ;)

Please fix that (it actually obviously does not comply with the standard which does ONLY allow to instatiate const-static data members in the class header. I am surprised that any compiler compiles that)

In file included from MMatrix.cc:27:0:
MMatrix.h:34:23: sorry, unimplemented: non-static data member initializers
MMatrix.h:34:23: error: ISO C++ forbids in-class initialization of non-const static member ‘fDelimiter’
MMatrix.h:39:21: sorry, unimplemented: non-static data member initializers
MMatrix.h:39:21: error: ISO C++ forbids in-class initialization of non-const static member ‘fComment’
MMatrix.h:43:25: sorry, unimplemented: non-static data member initializers
MMatrix.h:43:25: error: in-class initialization of static data member ‘fFileName’ of non-literal type
MMatrix.h:48:32: sorry, unimplemented: non-static data member initializers
MMatrix.h:48:32: error: ISO C++ forbids in-class initialization of non-const static member ‘fLineNumber’
MMatrix.cc: In member function ‘void MMatrix::ReadFile(TString)’:
MMatrix.cc:93:3: error: ‘fFileName’ was not declared in this scope
MMatrix.cc: In member function ‘double MMatrix::pedantic_strtod(std::string) const’:
MMatrix.cc:146:26: error: ‘fFileName’ was not declared in this scope
MMatrix.cc:170:26: error: ‘fFileName’ was not declared in this scope
MMatrix.cc: In member function ‘void MMatrix::print() const’:
MMatrix.cc:203:25: error: ‘fFileName’ was not declared in this scope
MMatrix.cc: In member function ‘virtual Int_t MMatrix::ReadEnv(const TEnv&, TString, Bool_t)’:
MMatrix.cc:292:46: error: ‘fFileName’ was not declared in this scope

Side remark: You cannot include time offsets in each pixels without respecting them in the calculation of the noise-window as well.

Thanks,
Thomas."	defect	new	major		component1