/*
 * Copyright 2011-2013 Blender Foundation
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

#ifndef __UTIL_TIME_H__
#define __UTIL_TIME_H__

CCL_NAMESPACE_BEGIN

/* Give current time in seconds in double precision, with good accuracy. */

double time_dt();

/* Sleep for the specified number of seconds */

void time_sleep(double t);

class scoped_timer {
public:
	scoped_timer(double *value) : value_(value)
	{
		if(value_ != NULL) {
			time_start_ = time_dt();
		}
	}

	~scoped_timer()
	{
		if(value_ != NULL) {
			*value_ = time_dt() - time_start_;
		}
	}
protected:
	double *value_;
	double time_start_;
};

CCL_NAMESPACE_END

#endif


/*
 * Copyright 2011-2013 Blender Foundation
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

#ifndef __UTIL_TIME_H__
#define __UTIL_TIME_H__

CCL_NAMESPACE_BEGIN

/* Give current time in seconds in double precision, with good accuracy. */

double time_dt();

/* Sleep for the specified number of seconds */

void time_sleep(double t);

class scoped_timer {
public:
	scoped_timer(double *value) : value_(value)
	{
		time_start_ = time_dt();
	}

	~scoped_timer()
	{
		if(value_ != NULL) {
			*value_ = time_dt() - time_start_;
		}
	}
protected:
	double *value_;
	double time_start_;
};

CCL_NAMESPACE_END

#endif


Fix Uninitialized Value compiler warning in the scoped_timer
Fix Uninitialized Value compiler warning in the scoped_timer

Although the code made it impossible to use time_start_ uninitialized, at least GCC did
still produce multiple warnings about it.
Since time_dt() is an extremely cheap operation and functionality does not change in any way when
removing the check in the constructor, this commit removes the check and therefore the warning.
