RESOLVED FIXED 6042
"S" and "T" path types control points errors 
https://bugs.webkit.org/show_bug.cgi?id=6042
Summary "S" and "T" path types control points errors
Julien Palmas
Reported 2005-12-11 07:25:02 PST
The spec is not respected. In the attached test case, the "S" command should set its first control point to be the current point and the "T" command should set its only control point to be the current point as well. FF renders the test case correctly
Attachments
Test case (362 bytes, image/svg+xml)
2005-12-11 07:26 PST, Julien Palmas 		no flags
Details
Expected result (6.49 KB, image/png)
2005-12-11 07:31 PST, Julien Palmas 		no flags
Details
Actuall result (7.25 KB, image/png)
2005-12-11 07:32 PST, Julien Palmas 		no flags
Details
Should fix this path problem (1.27 KB, patch)
2005-12-11 08:53 PST, Rob Buis 		no flags
DetailsFormatted DiffDiff
Another attempt atfixing this bug (1.79 KB, patch)
2005-12-11 09:33 PST, Rob Buis 		eric: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Julien Palmas
Comment 1 2005-12-11 07:26:00 PST
Created attachment 5031 [details] Test case
Julien Palmas
Comment 2 2005-12-11 07:31:55 PST
Created attachment 5032 [details] Expected result
Julien Palmas
Comment 3 2005-12-11 07:32:15 PST
Created attachment 5033 [details] Actuall result
Rob Buis
Comment 4 2005-12-11 08:53:29 PST
Created attachment 5035 [details] Should fix this path problem This should fix it, again pleasetest well :) It doesnt seem to regress on my system. Cheers, Rob.
Rob Buis
Comment 5 2005-12-11 09:33:41 PST
Created attachment 5036 [details] Another attempt atfixing this bug Previous one was wrong, since the test was done afterwards. This one should work. Cheers, Rob.
Julien Palmas
Comment 6 2005-12-11 09:39:09 PST
This patch looks good to me. Nice Rob!
Eric Seidel (no email)
Comment 7 2005-12-12 09:26:22 PST
Comment on attachment 5036 [details] Another attempt atfixing this bug I dislike the existing variable names, but the patch itself looks fine.
Note You need to log in before you can comment on or make changes to this bug.