<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 12 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:SimSun;
panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:Tahoma;
panose-1:2 11 6 4 3 5 4 4 2 4;}
@font-face
{font-family:SimSun;
panose-1:2 1 6 0 3 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal;
font-family:"Calibri","sans-serif";
color:windowtext;}
span.EmailStyle18
{mso-style-type:personal-reply;
font-family:"Calibri","sans-serif";
color:#1F497D;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.25in 1.0in 1.25in;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="color:#1F497D">Hi, <o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D">IMHO, It is valid to report error after glEndList with offend API removed from the list, and it is also reasonable to still keep the APIs and report the same error again when glCallLists.
<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D">I have filed an BUG#12012 in khronos and regarding to the feedback I have collected, it seems we need to relax this test case.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D">If no objection, Crystal could you prepare a patch to address this issue?<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D">Thanks. <o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D">Best Regards,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D">David<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
<div>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> Piglit [mailto:piglit-bounces@lists.freedesktop.org]
<b>On Behalf Of </b>Jin, Crystal<br>
<b>Sent:</b> Friday, March 21, 2014 6:29 PM<br>
<b>To:</b> piglit@lists.freedesktop.org<br>
<b>Subject:</b> [Piglit] whether require glBegin/glEnd check during display list compiling<o:p></o:p></span></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Hi, <o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">When I tested piglit test case ‘spec/!OpenGL 1.0/gl-1.0-beginend-coverage’, I found that mostly errors with our amd driver is expected error not match getError after ‘glCallList(dlist)’. In another words, display list executed got error.
And I found the root cause is that there is glBegin/End check functionality in our driver which leads to display list compiling directly return rather than add the GL command into display list. So the display list only includes glBegin and glEnd function.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">And my question is whether require glBegin/glEnd check during display list compiling, and need add wrong GL commands sequence into display list or not (our driver does not add it). For example, glBitmap cannot be in glBegin/glEnd, but it
is in glBegin/glEnd during glNewList/glEndList, if it is required to add it into display list.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Spec 4 page 411 “<i>After calling NewList all subsequent GL commands are placed in the display list (in the order the commands are issued) until a call to Endlist(void)</i>”. It is not obviously mentioned whether it need to be added into
display list if subsequent GL commands are wrong.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Can you please help me to confirm the problem, thanks?<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Best wishes!<o:p></o:p></p>
<p class="MsoNormal">Crystal Jin<o:p></o:p></p>
</div>
</body>
</html>