Made Quarternion's operator overloads use their respective methods

[aevitas]

Operator overloads of Quaternion now utilize their respective methods, and removed redundant "this" qualifiers in Quaternion constructor.

[nguerrera]

Thanks for this.

Merely commenting here to make it easier to recognize that it's been acknowledged. The good discussion is going on in #41.

(I came looking for that discussion based on email notification and was confused when it wasn't here.)

[mmitche]

Can one of the admins verify this patch?

[mellinoe]

Thanks for the submission, @aevitas . As per the discussion in #41, I don't think we have enough information to say that this is a safe change, with regards to expectations around inlining. I would imagine that this change wouldn't make a significant difference in most cases, and especially on newer versions of our JIT, but we would be sacrificing perf in cases where inlining fails, and even in the best case this would not increase performance. The code duplication is not desirable, but given that the methods are quite small, it is not a big problem to maintain at this time.