vbo: fix another GL_LINE_LOOP bug
authorBrian Paul <brianp@vmware.com>
Sat, 31 Oct 2015 13:02:36 +0000 (07:02 -0600)
committerBrian Paul <brianp@vmware.com>
Wed, 4 Nov 2015 18:51:59 +0000 (11:51 -0700)
commitbdf6cef0333bf7278e2e2347aaae399288e87dcd
tree7cc7adc6254eff9aa5d803a0d5533c22d551115a
parentd31481e70ab0da293d4c3010815f643f161b7168
vbo: fix another GL_LINE_LOOP bug

Very long line loops which spanned 3 or more vertex buffers were not
handled correctly and could result in stray lines.

The piglit lineloop test draws 10000 vertices by default, and is not
long enough to trigger this.  Even 'lineloop -count 100000' doesn't
trigger the bug.

For future reference, the issue can be reproduced by changing Mesa's
VBO_VERT_BUFFER_SIZE to 4096 and changing the piglit lineloop test to
use glVertex2f(), draw 3 loops instead of 1, and specifying -count
1023.

Acked-by: Sinclair Yeh <syeh@vmware.com>
Reviewed-by: Roland Scheidegger <sroland@vmware.com>
src/mesa/vbo/vbo_exec_api.c
src/mesa/vbo/vbo_exec_draw.c