archive/blender-archive
Archived
1
1
Fork 0
Code Pull Requests Packages Activity

Cleanup: bool & const

Browse Source 
Using bool when we're asking yes/no questions such as whether some GPU
feature is supported.

Consolidated these simple functions into gpu_extensions.c and grouped
them in the header.

Const-ified some args where the functions don't modify the pointed-to
data.
This commit is contained in:
Mike Erwin
2015-03-16 22:57:56 -04:00
parent b0bbd34ac2
commit c0fa864e24
3 changed files with 50 additions and 49 deletions
Download Patch File Download Diff File Expand all files Collapse all files

38
source/blender/gpu/GPU_extensions.h
View File

@@ -56,14 +56,16 @@ typedef struct GPUShader GPUShader;
void GPU_extensions_disable(void); void GPU_extensions_disable(void);
int GPU_glsl_support(void); bool GPU_glsl_support(void);
int GPU_non_power_of_two_support(void); bool GPU_non_power_of_two_support(void);
int GPU_vertex_buffer_support(void); bool GPU_vertex_buffer_support(void);
int GPU_display_list_support(void); bool GPU_display_list_support(void);
bool GPU_bicubic_bump_support(void);
int GPU_max_texture_size(void);
int GPU_color_depth(void); int GPU_color_depth(void);
void GPU_code_generate_glsl_lib(void); void GPU_code_generate_glsl_lib(void);
int GPU_bicubic_bump_support(void);
int GPU_max_texture_size (void);
/* GPU Types */ /* GPU Types */
@@ -91,7 +93,7 @@ typedef enum GPUDriverType {
GPU_DRIVER_ANY = (0xff0000) GPU_DRIVER_ANY = (0xff0000)
} GPUDriverType; } GPUDriverType;
int GPU_type_matches(GPUDeviceType device, GPUOSType os, GPUDriverType driver); bool GPU_type_matches(GPUDeviceType device, GPUOSType os, GPUDriverType driver);
/* GPU Texture /* GPU Texture
* - always returns unsigned char RGBA textures * - always returns unsigned char RGBA textures
@@ -113,13 +115,13 @@ typedef enum GPUHDRType {
GPU_HDR_FULL_FLOAT = (1 << 1), GPU_HDR_FULL_FLOAT = (1 << 1),
} GPUHDRType; } GPUHDRType;
GPUTexture *GPU_texture_create_1D(int w, float *pixels, char err_out[256]); GPUTexture *GPU_texture_create_1D(int w, const float *pixels, char err_out[256]);
GPUTexture *GPU_texture_create_2D(int w, int h, float *pixels, GPUHDRType hdr, char err_out[256]); GPUTexture *GPU_texture_create_2D(int w, int h, const float *pixels, GPUHDRType hdr, char err_out[256]);
GPUTexture *GPU_texture_create_3D(int w, int h, int depth, int channels, float *fpixels); GPUTexture *GPU_texture_create_3D(int w, int h, int depth, int channels, const float *fpixels);
GPUTexture *GPU_texture_create_depth(int w, int h, char err_out[256]); GPUTexture *GPU_texture_create_depth(int w, int h, char err_out[256]);
GPUTexture *GPU_texture_create_vsm_shadow_map(int size, char err_out[256]); GPUTexture *GPU_texture_create_vsm_shadow_map(int size, char err_out[256]);
GPUTexture *GPU_texture_create_2D_procedural(int w, int h, float *pixels, char err_out[256]); GPUTexture *GPU_texture_create_2D_procedural(int w, int h, const float *pixels, char err_out[256]);
GPUTexture *GPU_texture_create_1D_procedural(int w, float *pixels, char err_out[256]); GPUTexture *GPU_texture_create_1D_procedural(int w, const float *pixels, char err_out[256]);
GPUTexture *GPU_texture_from_blender(struct Image *ima, GPUTexture *GPU_texture_from_blender(struct Image *ima,
struct ImageUser *iuser, bool is_data, double time, int mipmap); struct ImageUser *iuser, bool is_data, double time, int mipmap);
GPUTexture *GPU_texture_from_preview(struct PreviewImage *prv, int mipmap); GPUTexture *GPU_texture_from_preview(struct PreviewImage *prv, int mipmap);
@@ -138,10 +140,10 @@ void GPU_depth_texture_mode(GPUTexture *tex, bool compare, bool use_filter);
GPUFrameBuffer *GPU_texture_framebuffer(GPUTexture *tex); GPUFrameBuffer *GPU_texture_framebuffer(GPUTexture *tex);
int GPU_texture_target(GPUTexture *tex); int GPU_texture_target(const GPUTexture *tex);
int GPU_texture_opengl_width(GPUTexture *tex); int GPU_texture_opengl_width(const GPUTexture *tex);
int GPU_texture_opengl_height(GPUTexture *tex); int GPU_texture_opengl_height(const GPUTexture *tex);
int GPU_texture_opengl_bindcode(GPUTexture *tex); int GPU_texture_opengl_bindcode(const GPUTexture *tex);
/* GPU Framebuffer /* GPU Framebuffer
* - this is a wrapper for an OpenGL framebuffer object (FBO). in practice * - this is a wrapper for an OpenGL framebuffer object (FBO). in practice
@@ -174,8 +176,8 @@ void GPU_offscreen_free(GPUOffScreen *ofs);
void GPU_offscreen_bind(GPUOffScreen *ofs, bool save); void GPU_offscreen_bind(GPUOffScreen *ofs, bool save);
void GPU_offscreen_unbind(GPUOffScreen *ofs, bool restore); void GPU_offscreen_unbind(GPUOffScreen *ofs, bool restore);
void GPU_offscreen_read_pixels(GPUOffScreen *ofs, int type, void *pixels); void GPU_offscreen_read_pixels(GPUOffScreen *ofs, int type, void *pixels);
int GPU_offscreen_width(GPUOffScreen *ofs); int GPU_offscreen_width(const GPUOffScreen *ofs);
int GPU_offscreen_height(GPUOffScreen *ofs); int GPU_offscreen_height(const GPUOffScreen *ofs);
/* GPU Shader /* GPU Shader
* - only for fragment shaders now * - only for fragment shaders now

5
source/blender/gpu/intern/gpu_codegen.c
View File

@@ -719,11 +719,6 @@ static char *code_generate_vertex(ListBase *nodes, const GPUMatType type)
return code; return code;
} }
int GPU_bicubic_bump_support(void)
{
return GLEW_ARB_texture_query_lod && GLEW_VERSION_3_0;
}
void GPU_code_generate_glsl_lib(void) void GPU_code_generate_glsl_lib(void)
{ {
DynStr *ds; DynStr *ds;

56
source/blender/gpu/intern/gpu_extensions.c
View File

@@ -123,7 +123,7 @@ struct GPUFrameBuffer {
/* GPU Types */ /* GPU Types */
int GPU_type_matches(GPUDeviceType device, GPUOSType os, GPUDriverType driver) bool GPU_type_matches(GPUDeviceType device, GPUOSType os, GPUDriverType driver)
{ {
return (GG.device & device) && (GG.os & os) && (GG.driver & driver); return (GG.device & device) && (GG.os & os) && (GG.driver & driver);
} }
@@ -250,29 +250,34 @@ void gpu_extensions_exit(void)
GPU_invalid_tex_free(); GPU_invalid_tex_free();
} }
int GPU_glsl_support(void) bool GPU_glsl_support(void)
{ {
return !GG.extdisabled && GG.glslsupport; return !GG.extdisabled && GG.glslsupport;
} }
int GPU_non_power_of_two_support(void) bool GPU_non_power_of_two_support(void)
{ {
if (GG.npotdisabled) if (GG.npotdisabled)
return 0; return false;
return GLEW_ARB_texture_non_power_of_two; return GLEW_ARB_texture_non_power_of_two;
} }
int GPU_vertex_buffer_support(void) bool GPU_vertex_buffer_support(void)
{ {
return GLEW_ARB_vertex_buffer_object || GLEW_VERSION_1_5; return GLEW_ARB_vertex_buffer_object || GLEW_VERSION_1_5;
} }
int GPU_display_list_support(void) bool GPU_display_list_support(void)
{ {
return !GG.dlistsdisabled; return !GG.dlistsdisabled;
} }
bool GPU_bicubic_bump_support(void)
{
return GLEW_ARB_texture_query_lod && GLEW_VERSION_3_0;
}
int GPU_color_depth(void) int GPU_color_depth(void)
{ {
return GG.colordepth; return GG.colordepth;
@@ -333,17 +338,16 @@ struct GPUTexture {
GPUFrameBuffer *fb; /* GPUFramebuffer this texture is attached to */ GPUFrameBuffer *fb; /* GPUFramebuffer this texture is attached to */
int fb_attachment; /* slot the texture is attached to */ int fb_attachment; /* slot the texture is attached to */
int depth; /* is a depth texture? */ int depth; /* is a depth texture? if 3D how deep? */
}; };
static unsigned char *GPU_texture_convert_pixels(int length, float *fpixels) static unsigned char *GPU_texture_convert_pixels(int length, const float *fpixels)
{ {
unsigned char *pixels, *p; unsigned char *pixels, *p;
const float *fp; const float *fp = fpixels;
int a, len; const int len = 4*length;
int a;
len = 4*length;
fp = fpixels;
p = pixels = MEM_callocN(sizeof(unsigned char)*len, "GPUTexturePixels"); p = pixels = MEM_callocN(sizeof(unsigned char)*len, "GPUTexturePixels");
for (a=0; a<len; a++, p++, fp++) for (a=0; a<len; a++, p++, fp++)
@@ -365,7 +369,7 @@ static void GPU_glTexSubImageEmpty(GLenum target, GLenum format, int x, int y, i
} }
static GPUTexture *GPU_texture_create_nD( static GPUTexture *GPU_texture_create_nD(
int w, int h, int n, float *fpixels, int depth, GPUHDRType hdr_type, int components, int w, int h, int n, const float *fpixels, int depth, GPUHDRType hdr_type, int components,
char err_out[256]) char err_out[256])
{ {
GPUTexture *tex; GPUTexture *tex;
@@ -507,12 +511,12 @@ static GPUTexture *GPU_texture_create_nD(
} }
GPUTexture *GPU_texture_create_3D(int w, int h, int depth, int channels, float *fpixels) GPUTexture *GPU_texture_create_3D(int w, int h, int depth, int channels, const float *fpixels)
{ {
GPUTexture *tex; GPUTexture *tex;
GLenum type, format, internalformat; GLenum type, format, internalformat;
void *pixels = NULL; void *pixels = NULL;
float vfBorderColor[4] = {0.0f, 0.0f, 0.0f, 0.0f}; const float vfBorderColor[4] = {0.0f, 0.0f, 0.0f, 0.0f};
if (!GLEW_VERSION_1_2) if (!GLEW_VERSION_1_2)
return NULL; return NULL;
@@ -684,7 +688,7 @@ GPUTexture *GPU_texture_from_preview(PreviewImage *prv, int mipmap)
} }
GPUTexture *GPU_texture_create_1D(int w, float *fpixels, char err_out[256]) GPUTexture *GPU_texture_create_1D(int w, const float *fpixels, char err_out[256])
{ {
GPUTexture *tex = GPU_texture_create_nD(w, 1, 1, fpixels, 0, GPU_HDR_NONE, 4, err_out); GPUTexture *tex = GPU_texture_create_nD(w, 1, 1, fpixels, 0, GPU_HDR_NONE, 4, err_out);
@@ -694,7 +698,7 @@ GPUTexture *GPU_texture_create_1D(int w, float *fpixels, char err_out[256])
return tex; return tex;
} }
GPUTexture *GPU_texture_create_2D(int w, int h, float *fpixels, GPUHDRType hdr, char err_out[256]) GPUTexture *GPU_texture_create_2D(int w, int h, const float *fpixels, GPUHDRType hdr, char err_out[256])
{ {
GPUTexture *tex = GPU_texture_create_nD(w, h, 2, fpixels, 0, hdr, 4, err_out); GPUTexture *tex = GPU_texture_create_nD(w, h, 2, fpixels, 0, hdr, 4, err_out);
@@ -732,7 +736,7 @@ GPUTexture *GPU_texture_create_vsm_shadow_map(int size, char err_out[256])
return tex; return tex;
} }
GPUTexture *GPU_texture_create_2D_procedural(int w, int h, float *pixels, char err_out[256]) GPUTexture *GPU_texture_create_2D_procedural(int w, int h, const float *pixels, char err_out[256])
{ {
GPUTexture *tex = GPU_texture_create_nD(w, h, 2, pixels, 0, GPU_HDR_HALF_FLOAT, 2, err_out); GPUTexture *tex = GPU_texture_create_nD(w, h, 2, pixels, 0, GPU_HDR_HALF_FLOAT, 2, err_out);
@@ -749,7 +753,7 @@ GPUTexture *GPU_texture_create_2D_procedural(int w, int h, float *pixels, char e
return tex; return tex;
} }
GPUTexture *GPU_texture_create_1D_procedural(int w, float *pixels, char err_out[256]) GPUTexture *GPU_texture_create_1D_procedural(int w, const float *pixels, char err_out[256])
{ {
GPUTexture *tex = GPU_texture_create_nD(w, 0, 1, pixels, 0, GPU_HDR_HALF_FLOAT, 2, err_out); GPUTexture *tex = GPU_texture_create_nD(w, 0, 1, pixels, 0, GPU_HDR_HALF_FLOAT, 2, err_out);
@@ -767,7 +771,7 @@ GPUTexture *GPU_texture_create_1D_procedural(int w, float *pixels, char err_out[
void GPU_invalid_tex_init(void) void GPU_invalid_tex_init(void)
{ {
float color[4] = {1.0f, 0.0f, 1.0f, 1.0}; const float color[4] = {1.0f, 0.0f, 1.0f, 1.0f};
GG.invalid_tex_1D = GPU_texture_create_1D(1, color, NULL); GG.invalid_tex_1D = GPU_texture_create_1D(1, color, NULL);
GG.invalid_tex_2D = GPU_texture_create_2D(1, 1, color, GPU_HDR_NONE, NULL); GG.invalid_tex_2D = GPU_texture_create_2D(1, 1, color, GPU_HDR_NONE, NULL);
GG.invalid_tex_3D = GPU_texture_create_3D(1, 1, 1, 4, color); GG.invalid_tex_3D = GPU_texture_create_3D(1, 1, 1, 4, color);
@@ -920,22 +924,22 @@ void GPU_texture_ref(GPUTexture *tex)
tex->refcount++; tex->refcount++;
} }
int GPU_texture_target(GPUTexture *tex) int GPU_texture_target(const GPUTexture *tex)
{ {
return tex->target; return tex->target;
} }
int GPU_texture_opengl_width(GPUTexture *tex) int GPU_texture_opengl_width(const GPUTexture *tex)
{ {
return tex->w; return tex->w;
} }
int GPU_texture_opengl_height(GPUTexture *tex) int GPU_texture_opengl_height(const GPUTexture *tex)
{ {
return tex->h; return tex->h;
} }
int GPU_texture_opengl_bindcode(GPUTexture *tex) int GPU_texture_opengl_bindcode(const GPUTexture *tex)
{ {
return tex->bindcode; return tex->bindcode;
} }
@@ -1364,12 +1368,12 @@ void GPU_offscreen_read_pixels(GPUOffScreen *ofs, int type, void *pixels)
glReadPixels(0, 0, ofs->color->w, ofs->color->h, GL_RGBA, type, pixels); glReadPixels(0, 0, ofs->color->w, ofs->color->h, GL_RGBA, type, pixels);
} }
int GPU_offscreen_width(GPUOffScreen *ofs) int GPU_offscreen_width(const GPUOffScreen *ofs)
{ {
return ofs->color->w; return ofs->color->w;
} }
int GPU_offscreen_height(GPUOffScreen *ofs) int GPU_offscreen_height(const GPUOffScreen *ofs)
{ {
return ofs->color->h; return ofs->color->h;
} }