[quake3-commits] [ioquake/ioq3] c74b9a: Remove unused renderer_buffer variable

Sat Jul 21 00:46:43 EDT 2018

  Branch: refs/heads/master
  Home:   https://github.com/ioquake/ioq3
  Commit: c74b9a44526bae1f1cfca1dcbc9b19e7bebc332a
      https://github.com/ioquake/ioq3/commit/c74b9a44526bae1f1cfca1dcbc9b19e7bebc332a
  Author: Zack Middleton <zack at cloemail.com>
  Date:   2018-07-20 (Fri, 20 Jul 2018)

  Changed paths:
    M code/renderergl1/tr_init.c
    M code/renderergl2/tr_init.c

  Log Message:
  -----------
  Remove unused renderer_buffer variable

  Commit: bead4aa8c7fe80b99b0f6ef2c8b1af0eaedbb9ae
      https://github.com/ioquake/ioq3/commit/bead4aa8c7fe80b99b0f6ef2c8b1af0eaedbb9ae
  Author: Zack Middleton <zack at cloemail.com>
  Date:   2018-07-20 (Fri, 20 Jul 2018)

  Changed paths:
    M code/renderergl1/tr_init.c
    M code/renderergl2/tr_init.c
    M code/sdl/sdl_glimp.c

  Log Message:
  -----------
  Fix renderer not clearing some GL extension variables at vid_restart

If renderer is compiled into client (USE_RENDERER_DLOPEN=0) and after
start up set r_allowExtension to 0 and run vid_restart, some extension
were still used.

  Commit: 476134f5a68dae222526930fe88e3550e15bbf34
      https://github.com/ioquake/ioq3/commit/476134f5a68dae222526930fe88e3550e15bbf34
  Author: Zack Middleton <zack at cloemail.com>
  Date:   2018-07-20 (Fri, 20 Jul 2018)

  Changed paths:
    M code/renderergl1/tr_image.c
    M code/renderergl2/tr_image.c

  Log Message:
  -----------
  Don't upload 8 bit grayscale images as 16 bit luminance

Using more color bits than the source image provides shouldn't improve
the quality.

  Commit: 255c33b36762be72e9bcc73a9d7d3fc508ea0e9a
      https://github.com/ioquake/ioq3/commit/255c33b36762be72e9bcc73a9d7d3fc508ea0e9a
  Author: Zack Middleton <zack at cloemail.com>
  Date:   2018-07-20 (Fri, 20 Jul 2018)

  Changed paths:
    M code/renderergl1/tr_backend.c

  Log Message:
  -----------
  OpenGL1: Use RE_UploadCinematic() instead of duplicate code

Already done in opengl2 renderer.

  Commit: 7391215bd4d1228b9887fab4b181d6f881d12574
      https://github.com/ioquake/ioq3/commit/7391215bd4d1228b9887fab4b181d6f881d12574
  Author: Zack Middleton <zack at cloemail.com>
  Date:   2018-07-20 (Fri, 20 Jul 2018)

  Changed paths:
    M code/renderercommon/qgl.h
    M code/renderergl1/tr_local.h
    M code/renderergl2/tr_local.h
    M code/renderergl2/tr_shadows.c
    M code/sdl/sdl_glimp.c

  Log Message:
  -----------
  Don't load non-core GL functions for OpenGL 3.2 core context

Also declare the GL functions in tr_local.h so there is compile error
for non-core GL functions instead of SEGFAULT from dereferencing a NULL
pointer.

Disable the non-functional stencil shadow code that hasn't been updated
to use OpenGL 3.2 core compatible drawing.

  Commit: d861a4f42753b632e5a6f4d07e69d1ec319e5164
      https://github.com/ioquake/ioq3/commit/d861a4f42753b632e5a6f4d07e69d1ec319e5164
  Author: Zack Middleton <zack at cloemail.com>
  Date:   2018-07-20 (Fri, 20 Jul 2018)

  Changed paths:
    M code/renderercommon/qgl.h
    M code/renderercommon/tr_common.h
    M code/renderergl1/tr_init.c
    M code/renderergl2/tr_extensions.c
    M code/renderergl2/tr_init.c
    M code/renderergl2/tr_local.h
    M code/sdl/sdl_glimp.c

  Log Message:
  -----------
  Load OpenGL ES 2.0 function procs

Load functions procs supported by OpenGL ES 2.0, though there is not a
compatible renderer yet. Change argument for GLimp_Init from coreContext
to fixedFunction.

  Commit: 658165cfbb997a83315780be6913a04f371b42ee
      https://github.com/ioquake/ioq3/commit/658165cfbb997a83315780be6913a04f371b42ee
  Author: Zack Middleton <zack at cloemail.com>
  Date:   2018-07-20 (Fri, 20 Jul 2018)

  Changed paths:
    M code/renderergl2/tr_init.c
    M code/sdl/sdl_glimp.c

  Log Message:
  -----------
  Don't check fixed function GL extensions when using shader pipeline

GL_TEXTURE_UNITS_ARB is not part of OpenGL ES 2 API and sets GL error.
It's not part of OpenGL 3.2 core profile either.

  Commit: 14cc4cc6cb18b78c5cc7ea43a2836a71788a49d3
      https://github.com/ioquake/ioq3/commit/14cc4cc6cb18b78c5cc7ea43a2836a71788a49d3
  Author: Zack Middleton <zack at cloemail.com>
  Date:   2018-07-20 (Fri, 20 Jul 2018)

  Changed paths:
    M code/renderergl2/tr_local.h
    M code/renderergl2/tr_shade.c
    M code/renderergl2/tr_vbo.c

  Log Message:
  -----------
  OpenGL2: Fix world VAO cache drawing when glIndex_t is unsigned short

OpenGL ES is only required to support unsigned short for element buffer
values.

R_DrawElements() firstIndex argument was glIndex_t which caused element
indexes to wrap around to 0 when glIndex_t is an unsigned short.
(glIndex_t is an index into the vertexes buffer, not element buffer.)
Change it to 'int' like tess.firstIndex which is passed to
R_DrawElements().

World VAO cache buffer size allowed storing more vertexes than unsigned
short glIndex_t could reference. This resulted in the vertex indexes in
the element buffer wrapping around to 0.

  Commit: 39e2113c73b8147e97a56168be38091c8f1a3f7f
      https://github.com/ioquake/ioq3/commit/39e2113c73b8147e97a56168be38091c8f1a3f7f
  Author: Zack Middleton <zack at cloemail.com>
  Date:   2018-07-20 (Fri, 20 Jul 2018)

  Changed paths:
    M code/renderergl2/glsl/calclevels4x_fp.glsl
    M code/renderergl2/glsl/ssao_fp.glsl
    M code/renderergl2/tr_backend.c
    M code/renderergl2/tr_cmds.c
    M code/renderergl2/tr_dsa.c
    M code/renderergl2/tr_extensions.c
    M code/renderergl2/tr_flares.c
    M code/renderergl2/tr_glsl.c
    M code/renderergl2/tr_scene.c
    M code/renderergl2/tr_shadows.c
    M code/renderergl2/tr_vbo.c

  Log Message:
  -----------
  OpenGL2: Misc fixes and cleanup

Fix two constants in GLSL shaders. Remove f suffix from float and fix
int to float assignment. They were causing shader compile errors in
OpenGL ES 2 context.

Remove disabling clip plane. Clip plane is unused and never enabled in
the opengl2 renderer. Remove disabling it to avoid causing a GL error
when using OpenGL 3.2 core profile or OpenGL ES.

Make VAO cache vertex stride be size of srfVert_t since that is what
is uploaded to the GPU. No behavior change. There is a disabled debug
id in srfVert_t though which if enabled changes srfVert_t size.

Compare: https://github.com/ioquake/ioq3/compare/809a7765b25a...39e2113c73b8
      **NOTE:** This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/

      Functionality will be removed from GitHub.com on January 31st, 2019.