Fix how curveTo2 (v operator) is translated to SVG

[bhasto]

Based on PDF spec, page 133, with v operator, current point should be used as the first control point of the curve.

Do not overwrite current point before an SVG curve is built, so it can be actually used as first control point.

[timvandermeij]

Interesting. Do you have an actual PDF file that is broken because of this? I think the fix is fine because we do that already in the canvas back-end; see:

pdf.js/src/display/canvas.js

Lines 1229 to 1241 in 794744c

	case OPS.curveTo2:
	ctx.bezierCurveTo(
	x,
	y,
	args[j],
	args[j + 1],
	args[j + 2],
	args[j + 3]
	);
	x = args[j + 2];
	y = args[j + 3];
	j += 4;
	break;

[bhasto]

Unfortunately I don’t have an image readily available that I could share in a public forum.

I can try to come up with one, but I don’t have experience creating PDFs, so it’ll take some time.

[timvandermeij]

/botio-linux preview

[pdfjsbot]

From: Bot.io (Linux m4)

---

Received

Command cmd_preview from @timvandermeij received. Current queue size: 0

Live output at: http://54.67.70.0:8877/d8d1fadeabd0e65/output.txt

[pdfjsbot]

From: Bot.io (Linux m4)

---

Success

Full output at http://54.67.70.0:8877/d8d1fadeabd0e65/output.txt

Total script time: 1.74 mins

Published

• Viewer: http://54.67.70.0:8877/d8d1fadeabd0e65/web/viewer.html

[timvandermeij]

Thank you!