Apologies if my post is not quite according to the standards of this sub-forum but.. Would anyone want to take the time to do a code-review of my c++ code. Not a standard time intensive review, just review the main/only header and cpp files and review format, c++11 standards, idioms and any big no-no's that stand out? The application is for my github portfolio.

The main application code is found in the Components folder. The bulk of the code resides in another repository (the CBA API) which please take a look at also although this hasn't been refactored in a while.

https://github.com/sazr/WindowTiler

I had a quick browse through the App code without actually reading things, and noticed a total lack of comments for the methods. Reverse engineering what a function does, with its entry and exit values and conditions won't work over a longer period of time, or when you add more lines of code.

The other thing was your switch at line 489, https://github.com/sazr/WindowTiler/blob/master/Components/App.cpp#L489

Why is everything at the same indent and no empty lines? It's impossible to find your 'case' lines this way.

	int yPos = 0;

	switch (barData.uEdge)
	{
	case ABE_LEFT:
	case ABE_RIGHT:
	{
		yPos = btnDim.top - lbH - 10;
	}
	break;
	case ABE_TOP:
	{
		yPos = btnDim.bottom + 10;
	}
	break;
	//case ABE_BOTTOM:
	default:
	{
		yPos = btnDim.top - lbH - 10;
	}
	break;
	}

	int yPos = (barData.uEdge == ABE_TOP) ? btnDim.bottom + 10 : btnDim.top - lbH - 10;