Simplify the viewport calculation

It was calculating vpX0/vpY0 twice.  Also use more readable names than vpXa and vpXb.

Also, we should really keep mixed newlines out of the project imho.

-[Unknown]