Not sure what that indentation was about, as it didn't show in Visual Studio.

For the brushes, while it is read only it's actually a little more complex behind the scenes. This is a screenshot from DotTrace Performance showing what happens when you call Brushes.Black:

![brushes-getblack-calls](https://f.cloud.github.com/assets/1949879/179536/c8f08738-7c03-11e2-9c61-f602beff9bab.png)

This is with the changes I made, so it's only being called once per function. As you can see, every time you get a brush from Brushes, it collects data about the current thread. About two thirds of the time spent in that function is spent collecting that data.

Another (probably better) reason to store the brushes in variables, is if you decide to later adjust a colour slightly, you don't need to hunt through the function to find all uses of it.