Bug 1648

I discovered that projected textures work well
on radeon / r200 only if you send the w-coord to the card.
-> tiny vertex format isn't allowed for projtex!

The old template based swtcl code (which is still
used by radeon driver) worked that way:
tiny vertex format isn't used when texturing.

The new "t_vertex" code emits the w-coord only if
its really necessary: it prefers a "tiny" vertex format.
You could try to just always disable that tiny vertex
format in r200ChooseVertexState().

Created attachment 1609 [details] [review]
fixes this projtex problem for me; needs cleanup and maybe optimization

please test.
feel free to cleanup/optimize/commit

Question:
Do we need w component for specular color? (the old code used the NOTEX_VERTEX
format for that) If not, cut that piece out of the patch.


unfortunately the glut menues are broken on r200 atm., but thats another
problem...

Created attachment 1625 [details] [review]
r200_projtex_fix2.diff.txt

+ cleanup
+ possible fixes for points and transition swtnl<->hwtcl

(when testing make sure to test the sw-tcl case...)

Created attachment 1759 [details] [review]
slightly different projtex fix

new version of the fix, which eliminates the additional statechange.
I'm not really sure this is sufficient & correct, but it works for projtex.
Some of the vte / vap combinations look a little odd, though maybe it wouldn't
if I'd know what those registers actually do...

your patch seems to work well at least for projtex.
First I thought
if (tnl->render_inputs & _TNL_BITS_TEX_ANY)
might not be up to date, but at least in projtex it seems so.

Someone here to try this patch with other programs that make use of projtex ?

(In reply to comment #5)
> First I thought
> if (tnl->render_inputs & _TNL_BITS_TEX_ANY)
> might not be up to date, but at least in projtex it seems so.
Thought that too first, but then I guessed if _TNL_BITS_TEX_ANY wouldn't be up
to date, it would not work correctly at all, since the code would incorrectly
select the tiny vertex format in cases where it shouldn't.
Applied to cvs.

Mass version move, cvs -> git