From Paul Martz, "Per the discussion in osg-users, this change adds code comments to the Matrix* headers to document assumptions in the getRotate() method."

include/osg/Matrixd

@@ -311,6 +311,10 @@ class OSG_EXPORT Matrixd
 #endif
 
         void setRotate(const Quat& q);
+        /** Get the matrix rotation as a Quat. Note that this function
+          * assumes a non-scaled matrix and will return incorrect results
+          * for scaled matrixces. Consider decompose() instead.
+          */
         Quat getRotate() const;
 
         void setTrans( value_type tx, value_type ty, value_type tz );

include/osg/Matrixf

@@ -312,6 +312,10 @@ class OSG_EXPORT Matrixf
 #endif
 
         void setRotate(const Quat& q);
+        /** Get the matrix rotation as a Quat. Note that this function
+          * assumes a non-scaled matrix and will return incorrect results
+          * for scaled matrixces. Consider decompose() instead.
+          */
         Quat getRotate() const;