On Wed May 17 09:07:24 2023 +0000, Nikolay Sivov wrote:
Instead of duplicating checks, it's better to move functional part of GdipIsMatrixInvertible() to some inlined helper.
I have tried such approach, but unfortunately I don't know how to do not calculate `matrix_det(matrix)` twice.
Maybe we could introduce function and provide calculated `det`, like: ``` static inline GpStatus check_det(GDIPCONST REAL det) { if (!(fabs(det) >= 1e-5)) return InvalidParameter; return Ok; } ```