summaryrefslogtreecommitdiff
path: root/src/mesa/main/lines.c
diff options
context:
space:
mode:
authorRoland Scheidegger <sroland@vmware.com>2010-11-08 21:49:51 +0100
committerRoland Scheidegger <sroland@vmware.com>2010-11-10 17:24:41 +0100
commitc7192ab11f7e34fdfe17d36d089260c6703ddfa8 (patch)
tree364876e2f4206afa6d1e2ee4b35f15a8183ae32b /src/mesa/main/lines.c
parentaa139a14ba7279f908da523522cb9ec0578b1ea0 (diff)
r200: fix r200 large points
DD_POINT_SIZE got never set for some time now (as it was set only in ifdefed out code), which caused the r200 driver to use the point primitive mistakenly in some cases which can only do size 1 instead of point sprite. Since the logic to use point instead of point sprite prim is flaky at best anyway (can't work correctly for per-vertex point size), just drop this and always emit point sprites (except for AA points) - reasons why the driver tried to use points for size 1.0 are unknown though it is possible they are faster or more conformant. Note that we can't emit point sprites without point sprite cntl as that might result in undefined point sizes, hence need drm version check (which was unnecessary before as it should always have selected points). An alternative would be to rely on the RE point size clamp controls which could clamp the size to 1.0 min/max even if the SE point size is undefined, but currently always use 0 for min clamp. (As a side note, this also means the driver does not honor the gl spec which mandates points, but not point sprites, with zero size to be rendered as size 1.) This should fix recent reports of https://bugs.freedesktop.org/show_bug.cgi?id=702. This is a candidate for the mesa 7.9 branch.
Diffstat (limited to 'src/mesa/main/lines.c')
0 files changed, 0 insertions, 0 deletions