nothingmn/AGENT.Contrib — pull request #4 by ErikNoren (2013-06-26 22:47:09+00:00)
Title: Converted Drawing/Point.cs to struct of shorts

Point has no methods and is used as a composite value so it makes more sense as a struct. Also since Point is used to represent a point on the screen, short made a little more sense (could conceivably use byte but there might be times that a point off the screen is desirable).

I removed the auto-properties as a consequence and had to alter one watch face to use the constructor instead of the properties. I left the construction of ints to reduce the number of code lines that need to be changed but added range checks to ensure we aren't accidentally truncating the value. Purely convenience.

Finally - I've never done a fork and pull before so I wanted to see what it's like.