[Piglit] [PATCH 2/4] Add test to draw multisample lines with GL_LINE_SMOOTH enabled
stereotype441 at gmail.com
Tue May 29 15:27:52 PDT 2012
On 22 May 2012 14:42, Anuj Phogat <anuj.phogat at gmail.com> wrote:
> On Tue, May 22, 2012 at 8:37 AM, Paul Berry <stereotype441 at gmail.com>
> > On 18 May 2012 16:56, Anuj Phogat <anuj.phogat at gmail.com> wrote:
> >> +
> >> + /* Draw test pattern in mulisample test_fbo with GL_LINE_SMOOTH
> >> + * enabled
> >> + */
> >> + glEnable(GL_LINE_SMOOTH);
> >> + glHint(GL_LINE_SMOOTH_HINT, GL_NICEST);
> >> + glDisable (GL_DEPTH_TEST);
> > Can you comment on why it's necessary to disable depth testing here?
> > not immediately obvious to me.
> It's required to correctly render overlapping smooth primitives.
> Otherwise we have to render the primitives
> in back to front order.
> For more explanation read OpenGL FAQ 14.150:
Ok, thanks. I'm not 100% sure this is necessary for this test (since it's
not drawing any overlapping lines) but I'm ok with leaving it in anyhow to
be on the safe side. Would you mind including a comment at this point in
the file to clarify?
With that change, this patch is:
Reviewed-by: Paul Berry <stereotype441 at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Piglit