Hi all, Currently, these tests fail:
./svg/svg-affine-test.h:179: Error: Test failed: matrix(1,2,3,4,5,6,) ./svg/svg-affine-test.h:179: Error: Assertion failed: !sp_svg_transform_read(read_fail_tests[i], &cm) ./svg/svg-affine-test.h:179: Error: Test failed: matrix(1,2,3,4,5e6-3) ./svg/svg-affine-test.h:179: Error: Assertion failed: !sp_svg_transform_read(read_fail_tests[i], &cm) ./svg/svg-affine-test.h:179: Error: Test failed: matrix(1,2,3,4,5e6.3) ./svg/svg-affine-test.h:179: Error: Assertion failed: !sp_svg_transform_read(read_fail_tests[i], &cm) ./svg/svg-affine-test.h:179: Error: Test failed: translate(1,) ./svg/svg-affine-test.h:179: Error: Assertion failed: !sp_svg_transform_read(read_fail_tests[i], &cm) ./svg/svg-affine-test.h:179: Error: Test failed: translate(1,6,) ./svg/svg-affine-test.h:179: Error: Assertion failed: !sp_svg_transform_read(read_fail_tests[i], &cm)
The current code is too permissive, and (perhaps unintentionally) tries to do its best in making sense of the matrix string. What do you guys think? Should we allow "matrix(1,2,3,4,5,6,)" to become "matrix(1,2,3,4,5,6)" ? (similar for the other failures)
thanks, Johan