eina: use matrix compose in place of equivalent code in matrix multiply
authorBryce Harrington <bryce@osg.samsung.com>
Wed, 14 Feb 2018 19:56:02 +0000 (11:56 -0800)
committerWonki Kim <wonki_.kim@samsung.com>
Thu, 5 Apr 2018 18:05:30 +0000 (03:05 +0900)
commite1b04fe5cb6815f12307dc213dcca24e4549b207
tree7016d93d2747034472e8c221a09cd65c033746cd
parent3fd4ac72d76cbcb64d093a8a24649a03c34d18c9
eina: use matrix compose in place of equivalent code in matrix multiply

Summary:
The eina_matrix3_compose and eina_matrix3_multiply API's are
mathematically identical (even though the implementations are
reversed... weird), except that the latter also includes a fastpath for
identity matrices.

Having two functionally equivalent APIs is redundant, so ideally one or
the other would be dropped.  But in order avoid API breakage, just have
one routine wrapper the other and eliminate the internal redundancy.

(Note that the parameter signatures of the two routines are different -
eina_matrix3_compose() takes the two input matrices first, and the
output matrix last, while eina_matrix3_multiply() takes the parameters
in the reverse order.  This inconsistency in the API style could result
in accidentally erroneous usage and would be an argument for deprecation
of one of the two APIs.)

Signed-off-by: Bryce Harrington <bryce@osg.samsung.com>
Reviewers: cedric

Reviewed By: cedric

Differential Revision: https://phab.enlightenment.org/D5806

Signed-off-by: Cedric Bail <cedric@osg.samsung.com>
src/lib/eina/eina_matrix.c