<div dir="ltr">On 7 August 2013 12:34, Ian Romanick <span dir="ltr"><<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Reviewed-by: Ian Romanick <<a href="mailto:ian.d.romanick@intel.com" target="_blank">ian.d.romanick@intel.com</a>><br>
<br>
I believe we have someone writing a piglit test for this?</blockquote><div><br></div><div>Yes, I believe Nick is writing a test for it.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<br>
On 08/07/2013 10:28 AM, Paul Berry wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
>From section E.1 (Profiles and Deprecated Features of OpenGL 3.0)<br>
</blockquote>
of the OpenGL 3.0 spec:<br>
<br>
     "LineWidth is not deprecated, but values greater than 1.0<br>
     will generate an INVALID VALUE error"<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
>From context it is clear that values greater than 1.0 should only<br>
</blockquote>
generate an INVALID VALUE error in a forward-compatible context.<br>
<br>
The code was correctly quoting this spec text, but it was disallowing<br>
all line widths in forward-compatible contexts, instead of just widths<br>
greater than 1.0.<br>
<br>
This patch introduces the correct check, so that setting a line width<br>
of 1.0 or less is permitted.<br>
---<br>
  src/mesa/main/lines.c | 3 ++-<br>
  1 file changed, 2 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/mesa/main/lines.c b/src/mesa/main/lines.c<br>
index 0df9d66..3c08ed2 100644<br>
--- a/src/mesa/main/lines.c<br>
+++ b/src/mesa/main/lines.c<br>
@@ -56,19 +56,20 @@ _mesa_LineWidth( GLfloat width )<br>
      *     "Wide lines and line stipple - LineWidth is not deprecated, but<br>
      *     values greater than 1.0 will generate an INVALID_VALUE error;"<br>
      *<br>
      * This is one of the very few cases where functionality was deprecated but<br>
      * *NOT* removed in a later spec.  Therefore, we only disallow this in a<br>
      * forward compatible context.<br>
      */<br>
     if (ctx->API == API_OPENGL_CORE<br>
         && ((ctx->Const.ContextFlags & GL_CONTEXT_FLAG_FORWARD_<u></u>COMPATIBLE_BIT)<br>
-           != 0)) {<br>
+           != 0)<br>
+       && width > 1.0) {<br>
        _mesa_error( ctx, GL_INVALID_VALUE, "glLineWidth" );<br>
        return;<br>
     }<br>
<br>
     if (ctx->Line.Width == width)<br>
        return;<br>
<br>
     FLUSH_VERTICES(ctx, _NEW_LINE);<br>
     ctx->Line.Width = width;<br>
<br>
</blockquote>
<br>
</div></div></blockquote></div><br></div></div>