Code Review for Job #20055

Code Review for Job #20055 - Workitem available at https://www.worklist.net/20055
_defaultHeadModelPosition was added to the SkeletonModel class but it is only initialized in buildShapes(), which is not necessarily called for all SkeletonModel instances.

True, the new variable is only used for the SkeletonModel instance that happens to belong to MyAvatar for which buildShapes() is almost always called, but if the fallback behavior is to init defaultHeadModelPosition to (0,0,0) for the case where there is no valid headJointIndex, then I think the variable should be initialized to the same value in the SkeletonModel ctor.  That initialization may spare some future dev a minor headache if they ever try to use defaultHeadModelPosition for a non-MyAvatar SkeletonModel instance.