Uninitialized data member in test code

[jwakely]

oneTBB/test/conformance/conformance_flowgraph.h

Lines 335 to 351 in 1f52f50

	template<typename OutputType, typename InputType = int>
	struct copy_counting_object {
	std::size_t copy_count;/*increases on every new copied object*/
	mutable std::size_t copies_count;/*count number of objects copied from this object*/
	std::size_t assign_count;
	bool is_copy;
	copy_counting_object():
	copy_count(0), copies_count(0), assign_count(0), is_copy(false) {}
	copy_counting_object(int):
	copy_count(0), copies_count(0), assign_count(0), is_copy(false) {}
	copy_counting_object( const copy_counting_object<OutputType, InputType>& other ):
	copy_count(other.copy_count + 1), is_copy(true) {
	++other.copies_count;
	}

If I'm reading this correctly, the copy constructor does not initialize copies_count. That means if you make a copy of a copy, the new object will do ++other.copies_count; on an uninitialized variable.

Is there any reason not to initialize copies_count to zero here?

Using default member initializers for all four data members seems sensible.