[quake3-bugzilla] [Bug 5160] Yet another reworked renderer

[bugzilla-daemon at icculus.org]

https://bugzilla.icculus.org/show_bug.cgi?id=5160

--- Comment #17 from q3urt.undead at gmail.com 2012-04-08 14:10:30 EDT ---
(In reply to comment #16)
> yes, the randerer is still evolving slowly, but as I develop on Tremulous, I
> have to port the changes to ioq3 manually. My latest patch is vbo4.diff, the
> main improvement is that dynamic geometry is rendered with a streaming VBO and
> the old q3 dynamic lights are replaced with GLSL rendered dlights on DX10
> capable hardware.

Ok!  I added a branch for it in openarena_engine.  I made your patch into a
modular renderer with two exceptions: I didn't modify tr_public.h or
tr_types.h.  How do you want to handle that?  I noticed the tr_public.h change
was only enabled if RETURN_GLTIME.  Since those two files are shared (always?)
between modular renderers, is there another way to handle that?

For the tr_types.h change, I put the #define in tr_local.h instead.  It looks
like that change should be in tr_types.h but that file is included by
tr_public.h.  I'm trying to keep it so I can switch between opengl1 (ioquake3),
openarena1 (0A 0.8.8), opengl2 (Canete's r29) and vbo1 (your vbo4.diff) on the
fly with +set cl_renderer.

I see you made a lot of changes to sdl_glimp.  It appears to be incompatible
changes with the other renderers so I put your sdl_glimp.c in code/renderer_vbo
and changed the Makefile so it would pull in your version.  With the others, I
added tr_extension.c but your changes look to be more invasive so creating a
separate file seems justified.

Your renderer doesn't work with RAVENMD4 which OpenArena 0.8.8 enables.  For
this to build, I had to disable RAVENMD4.

https://github.com/undeadzy/openarena_engine/tree/vbo_renderer

-- 
Configure bugmail: https://bugzilla.icculus.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.