[Piglit] [PATCH 2/4] Add test to draw multisample lines with GL_LINE_SMOOTH enabled

[Paul Berry]

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>
> wrote:
> > 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?
> It's
> > 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:
> http://www.opengl.org/archives/resources/faq/technical/rasterization.htm
>
>
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...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20120529/1cdb5fc7/attachment.htm>