iterators are verbose; range-based for loops are not

[ReiquelApplegate]

DataStructs.cpp could use some range-based for loops

[wilsonpe66]

Agreeed

see the following:

FunctionData::FunctionData(Program& program)
	: program(program),
	  globalData(program.getScope().getLocalData())
{
	for (const auto it : globalData)
	{
		Datum& datum = *it;
		if (!datum.getCompileTimeValue())
		{
			globalVariables.push_back(&datum);
		}
	}

	for (const auto script : program.scripts)
	{
		const vector<Datum*> data = script->getScope().getLocalData();
		globalVariables.insert(globalVariables.end(),
		                       data.begin(), data.end());
	}
	
	for (const auto it : program.namespaces)
	{
		const auto & data = it->getScope().getLocalData();
		globalVariables.insert(globalVariables.end(),
		                       data.begin(), data.end());
	}
}

[wilsonpe66]

see the following pr #31