In "player.h" you declare "int friendOrFoo[8];" as a private member on line 71.

In "player.cpp" you initialize it as "this->friendOrFoo[8] ={0};" on line 15. This doesn't do what you think it does because C++ only fills the array on declaration with this syntax. What this actually does is set the 8th element (which is a overflow) to 0 leaving all the other indices with whatever garabage is in RAM.

A number of options are available to fix this:

• In player.h do "int friendOrFoo[8] = {0};"
• In player.cpp use std::fill, memset, a for loop, or manually set each array index to 0.

Describe the bug
A single click outside the gamemap creates a selection box

In actors.cpp line 1603 I'm getting crashes when selecting troops and having them do things. I'm trying to determine how to replicate this easily. For reference, this is line 1603:

void actors::doNextStackedCommand()
{
    if (!this->listOfOrders.empty()) {
        switch (this->listOfOrders.front().orderType) { //crash here, maybe front is invalid?
...
}

The last return statement in the implementation of "int distanceToResource(int, cords)," currently line 135, has the potential to crash as it may dereference the front() of an empty list.

int distanceToResource(int kind, cords from)
{
	std::list <nearestBuildingTile> listOfResourceLocations;

	if (!listOfObjects.empty()) { ... }

	if (!listOfResourceLocations.empty()) { ... }

	return listOfResourceLocations.front().deltaDistance;
        //Crash here if both of the above if statements are false since listOfResourceLocations is empty
}

Vector index out of range error on line 1767. Worker's buildingId is -1

Steps to reproduce the behavior:

1. Set workers to gather resources with just Town Center
2. Wait for workers to try to return to drop off

Describe the bug
Double click of command to actors makes them teleport

In "randomMapGenerator.cpp" the while loop on line 120 might never break if the return statement condition is never met I think?

	bool foodFound = false;

	while (!foodFound) {
		int possibleFood = roll(0, static_cast<int>(listOfObjects.size()));
		if (listOfObjects[possibleFood].getTypeOfResource() == 1) {
			bool foodSourceOccupied = false;
			for (const int& foodOwned : occupiedFoodSources)
			{
				if (foodOwned == possibleFood) {
					foodSourceOccupied = true;
				}
			}
			if (!foodSourceOccupied) {
				return  listOfObjects[possibleFood].getLocation();
			}
		}
	}

In "main.cpp" line 111 I think this is meant to be a comparison and not assignment operator?
if (endProjectile = listOfProjectiles.size()) { ... }

I spent some time playing today and noticed a few things that can be addressed:

• If you box in the spawn area of a town hall the units will never be able to move. you can select them but never redirect them.
  
• When you select a large number of units the images in the bottom bar have odd orientation, sometimes outside of the bounds of the middle wooden rectangle texture (see bottom image).
• The edge scrolling continues to happen when the mouse leaves the game window.
• Sometimes the selection rectangle is... janky? It catches and misses some troops.
• There are times where the game stutters or freezes really badly for no apparent reason. It may be related to pathfinding. I'll run a profiler and see if I can figure this out soon.
• Pathfinding is sometimes unpredicatable. Troops may walk very long paths around obstacles that don't make much sense. I wonder if it's because troops refuse to walk on each other? I'll look more into this too.

This was fun though!