Just removing the zeros will shift the pattern to the left potentially producing<br>an effect that is not expected. I will take a look at what happens on Windows<br>to see what needs to be replicated.<br><br><div class="gmail_quote">

On 19 October 2011 20:05, Alexandre Julliard <span dir="ltr"><<a href="mailto:julliard@winehq.org">julliard@winehq.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div class="im">damian dixon <<a href="mailto:damian.dixon@gmail.com">damian.dixon@gmail.com</a>> writes:<br>
<br>
> diff --git a/dlls/winex11.drv/pen.c b/dlls/winex11.drv/pen.c<br>
> index b677515..71b1bea 100644<br>
> --- a/dlls/winex11.drv/pen.c<br>
> +++ b/dlls/winex11.drv/pen.c<br>
> @@ -108,7 +108,7 @@ HPEN X11DRV_SelectPen( PHYSDEV dev, HPEN hpen )<br>
> � � � �case PS_USERSTYLE:<br>
> � � � � � � �physDev->pen.dash_len = min(elp->elpNumEntries, MAX_DASHLEN);<br>
> � � � � � � �for(i = 0; i < physDev->pen.dash_len ; i++)<br>
> - � � � � � � � �physDev->pen.dashes[i] = min(elp->elpStyleEntry[i], 255);<br>
> + � � � � � � � �physDev->pen.dashes[i] = min(elp->elpStyleEntry[i], 255) ? min(elp->elpStyleEntry[i], 255) : 1;<br>
<br>
</div>Removing 0 entries (and merging the adjacent entries) would probably be<br>
more correct. It could also use some test cases.<br>
<font color="#888888"><br>
--<br>
Alexandre Julliard<br>
<a href="mailto:julliard@winehq.org">julliard@winehq.org</a><br>
</font></blockquote></div><br>