Monday, 4 March 2019

Re: [PATCH] SVG content handler: Fix plot style stroke_width

On 2018-12-23, Michael Forney <mforney@mforney.org> wrote:
> In 8332bf6b2a, when the stroke width was moved from a parameter to
> the plot style field, it accidentally used the `stroke` field of
> the svgtiny shape (the color) instead of `stroke_width`.
> ---
> content/handlers/image/svg.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/content/handlers/image/svg.c b/content/handlers/image/svg.c
> index 51260733d..99722495f 100644
> --- a/content/handlers/image/svg.c
> +++ b/content/handlers/image/svg.c
> @@ -189,7 +189,7 @@ svg_redraw_internal(struct content *c,
> for (i = 0; i != diagram->shape_count; i++) {
> if (diagram->shape[i].path) {
> pstyle.stroke_width = plot_style_int_to_fixed(
> - diagram->shape[i].stroke);
> + diagram->shape[i].stroke_width);
> pstyle.stroke_colour = BGR(diagram->shape[i].stroke);
> pstyle.fill_colour = BGR(diagram->shape[i].fill);
> res = ctx->plot->path(ctx,
> --
> 2.20.1
>

Ping. Anyone able to take a look at this patch?

No comments:

Post a Comment