diff options
author | Campbell Barton <ideasman42@gmail.com> | 2020-09-19 07:32:41 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2020-09-19 07:34:32 +0300 |
commit | 17a2820da8ad8ea23d336129f32e060e5746b047 (patch) | |
tree | 21f965c6f2d15a53fc67cd6dcad1a3a2a09df833 /source/blender/gpu/intern | |
parent | a78130c6101517c09eed14215087ab3dfe36bc5c (diff) |
Cleanup: consistent TODO/FIXME formatting for names
Following the most widely used convention for including todo's in
the code, that is: `TODO(name):`, `FIXME(name)` ... etc.
Diffstat (limited to 'source/blender/gpu/intern')
-rw-r--r-- | source/blender/gpu/intern/gpu_batch.cc | 2 | ||||
-rw-r--r-- | source/blender/gpu/intern/gpu_codegen.c | 4 | ||||
-rw-r--r-- | source/blender/gpu/intern/gpu_framebuffer.cc | 6 | ||||
-rw-r--r-- | source/blender/gpu/intern/gpu_immediate.cc | 2 | ||||
-rw-r--r-- | source/blender/gpu/intern/gpu_query.cc | 2 | ||||
-rw-r--r-- | source/blender/gpu/intern/gpu_shader.cc | 2 | ||||
-rw-r--r-- | source/blender/gpu/intern/gpu_shader_interface.cc | 2 | ||||
-rw-r--r-- | source/blender/gpu/intern/gpu_shader_interface.hh | 2 | ||||
-rw-r--r-- | source/blender/gpu/intern/gpu_state.cc | 2 | ||||
-rw-r--r-- | source/blender/gpu/intern/gpu_state_private.hh | 6 | ||||
-rw-r--r-- | source/blender/gpu/intern/gpu_texture.cc | 2 | ||||
-rw-r--r-- | source/blender/gpu/intern/gpu_texture_private.hh | 4 | ||||
-rw-r--r-- | source/blender/gpu/intern/gpu_viewport.c | 2 |
13 files changed, 19 insertions, 19 deletions
diff --git a/source/blender/gpu/intern/gpu_batch.cc b/source/blender/gpu/intern/gpu_batch.cc index 1f873570f4a..511ddd210af 100644 --- a/source/blender/gpu/intern/gpu_batch.cc +++ b/source/blender/gpu/intern/gpu_batch.cc @@ -216,7 +216,7 @@ int GPU_batch_vertbuf_add_ex(GPUBatch *batch, GPUVertBuf *verts, bool own_vbo) /* -------------------------------------------------------------------- */ /** \name Uniform setters * - * TODO(fclem) port this to GPUShader. + * TODO(fclem): port this to GPUShader. * \{ */ void GPU_batch_set_shader(GPUBatch *batch, GPUShader *shader) diff --git a/source/blender/gpu/intern/gpu_codegen.c b/source/blender/gpu/intern/gpu_codegen.c index f10fd8cd137..2d76e793fc0 100644 --- a/source/blender/gpu/intern/gpu_codegen.c +++ b/source/blender/gpu/intern/gpu_codegen.c @@ -421,7 +421,7 @@ static void codegen_call_functions(DynStr *ds, GPUNodeGraph *graph, GPUOutput *f ds, input->link->output->type, input->type, "tmp", input->link->output->id); } else if (input->source == GPU_SOURCE_BUILTIN) { - /* TODO(fclem) get rid of that. */ + /* TODO(fclem): get rid of that. */ if (input->builtin == GPU_INVERSE_VIEW_MATRIX) { BLI_dynstr_append(ds, "viewinv"); } @@ -527,7 +527,7 @@ static char *code_generate_fragment(GPUMaterial *material, if (builtins & GPU_BARYCENTRIC_TEXCO) { BLI_dynstr_append(ds, " vec2 barytexco = barycentric_resolve(barycentricTexCo);\n"); } - /* TODO(fclem) get rid of that. */ + /* TODO(fclem): get rid of that. */ if (builtins & GPU_VIEW_MATRIX) { BLI_dynstr_append(ds, " #define viewmat ViewMatrix\n"); } diff --git a/source/blender/gpu/intern/gpu_framebuffer.cc b/source/blender/gpu/intern/gpu_framebuffer.cc index 88779dead28..8d9a1301be0 100644 --- a/source/blender/gpu/intern/gpu_framebuffer.cc +++ b/source/blender/gpu/intern/gpu_framebuffer.cc @@ -112,7 +112,7 @@ void FrameBuffer::attachment_set(GPUAttachmentType type, const GPUAttachment &ne return; /* Exact same texture already bound here. */ } /* Unbind previous and bind new. */ - /* TODO(fclem) cleanup the casts. */ + /* TODO(fclem): cleanup the casts. */ if (attachment.tex) { reinterpret_cast<Texture *>(attachment.tex)->detach_from(this); } @@ -411,7 +411,7 @@ void GPU_framebuffer_read_color(GPUFrameBuffer *gpu_fb, unwrap(gpu_fb)->read(GPU_COLOR_BIT, format, rect, channels, slot, data); } -/* TODO(fclem) rename to read_color. */ +/* TODO(fclem): rename to read_color. */ void GPU_frontbuffer_read_pixels( int x, int y, int w, int h, int channels, eGPUDataFormat format, void *data) { @@ -420,7 +420,7 @@ void GPU_frontbuffer_read_pixels( } /* read_slot and write_slot are only used for color buffers. */ -/* TODO(fclem) port as texture operation. */ +/* TODO(fclem): port as texture operation. */ void GPU_framebuffer_blit(GPUFrameBuffer *gpufb_read, int read_slot, GPUFrameBuffer *gpufb_write, diff --git a/source/blender/gpu/intern/gpu_immediate.cc b/source/blender/gpu/intern/gpu_immediate.cc index 38d206ec3ad..44c6cac02ca 100644 --- a/source/blender/gpu/intern/gpu_immediate.cc +++ b/source/blender/gpu/intern/gpu_immediate.cc @@ -170,7 +170,7 @@ static void wide_line_workaround_start(GPUPrimType prim_type) immUnbindProgram(); - /* TODO(fclem) Don't use geometry shader and use quad instancing with double load. */ + /* TODO(fclem): Don't use geometry shader and use quad instancing with double load. */ // GPU_vertformat_multiload_enable(imm->vertex_format, 2); immBindBuiltinProgram(polyline_sh); diff --git a/source/blender/gpu/intern/gpu_query.cc b/source/blender/gpu/intern/gpu_query.cc index ad9b6d21420..610215fbe91 100644 --- a/source/blender/gpu/intern/gpu_query.cc +++ b/source/blender/gpu/intern/gpu_query.cc @@ -25,4 +25,4 @@ using namespace blender::gpu; -/* TODO(fclem) Make the associated C-API to use inside DRW profiler. */ +/* TODO(fclem): Make the associated C-API to use inside DRW profiler. */ diff --git a/source/blender/gpu/intern/gpu_shader.cc b/source/blender/gpu/intern/gpu_shader.cc index 1bd076f96f8..05c81c2efeb 100644 --- a/source/blender/gpu/intern/gpu_shader.cc +++ b/source/blender/gpu/intern/gpu_shader.cc @@ -487,7 +487,7 @@ void GPU_shader_unbind(void) /* -------------------------------------------------------------------- */ /** \name Transform feedback * - * TODO(fclem) Should be replaced by compute shaders. + * TODO(fclem): Should be replaced by compute shaders. * \{ */ bool GPU_shader_transform_feedback_enable(GPUShader *shader, GPUVertBuf *vertbuf) diff --git a/source/blender/gpu/intern/gpu_shader_interface.cc b/source/blender/gpu/intern/gpu_shader_interface.cc index dc59dca9f78..4a7c7bc15a3 100644 --- a/source/blender/gpu/intern/gpu_shader_interface.cc +++ b/source/blender/gpu/intern/gpu_shader_interface.cc @@ -34,7 +34,7 @@ namespace blender::gpu { ShaderInterface::ShaderInterface(void) { - /* TODO(fclem) add unique ID for debugging. */ + /* TODO(fclem): add unique ID for debugging. */ } ShaderInterface::~ShaderInterface(void) diff --git a/source/blender/gpu/intern/gpu_shader_interface.hh b/source/blender/gpu/intern/gpu_shader_interface.hh index fce6fda5f14..622f1ebb782 100644 --- a/source/blender/gpu/intern/gpu_shader_interface.hh +++ b/source/blender/gpu/intern/gpu_shader_interface.hh @@ -50,7 +50,7 @@ typedef struct ShaderInput { * Base class which is then specialized for each implementation (GL, VK, ...). **/ class ShaderInterface { - /* TODO(fclem) should be protected. */ + /* TODO(fclem): should be protected. */ public: /** Flat array. In this order: Attributes, Ubos, Uniforms. */ ShaderInput *inputs_ = NULL; diff --git a/source/blender/gpu/intern/gpu_state.cc b/source/blender/gpu/intern/gpu_state.cc index 9621aeeda7c..44ad9cc9a84 100644 --- a/source/blender/gpu/intern/gpu_state.cc +++ b/source/blender/gpu/intern/gpu_state.cc @@ -292,7 +292,7 @@ bool GPU_depth_mask_get(void) bool GPU_mipmap_enabled(void) { - /* TODO(fclem) this used to be a userdef option. */ + /* TODO(fclem): this used to be a userdef option. */ return true; } diff --git a/source/blender/gpu/intern/gpu_state_private.hh b/source/blender/gpu/intern/gpu_state_private.hh index 149a1ece186..d93556a1058 100644 --- a/source/blender/gpu/intern/gpu_state_private.hh +++ b/source/blender/gpu/intern/gpu_state_private.hh @@ -56,9 +56,9 @@ union GPUState { uint32_t invert_facing : 1; uint32_t shadow_bias : 1; /** Number of clip distances enabled. */ - /* TODO(fclem) This should be a shader property. */ + /* TODO(fclem): This should be a shader property. */ uint32_t clip_distances : 3; - /* TODO(fclem) remove, old opengl features. */ + /* TODO(fclem): remove, old opengl features. */ uint32_t polygon_smooth : 1; uint32_t line_smooth : 1; }; @@ -99,7 +99,7 @@ union GPUStateMutable { /** TODO remove */ float depth_range[2]; /** Positive if using program point size. */ - /* TODO(fclem) should be passed as uniform to all shaders. */ + /* TODO(fclem): should be passed as uniform to all shaders. */ float point_size; /** Not supported on every platform. Prefer using wideline shader. */ float line_width; diff --git a/source/blender/gpu/intern/gpu_texture.cc b/source/blender/gpu/intern/gpu_texture.cc index f7f29c7cece..99d286c3abd 100644 --- a/source/blender/gpu/intern/gpu_texture.cc +++ b/source/blender/gpu/intern/gpu_texture.cc @@ -186,7 +186,7 @@ using namespace blender::gpu; uint GPU_texture_memory_usage_get(void) { - /* TODO(fclem) Do that inside the new Texture class. */ + /* TODO(fclem): Do that inside the new Texture class. */ return 0; } diff --git a/source/blender/gpu/intern/gpu_texture_private.hh b/source/blender/gpu/intern/gpu_texture_private.hh index 04156632c5e..d4efebbaf95 100644 --- a/source/blender/gpu/intern/gpu_texture_private.hh +++ b/source/blender/gpu/intern/gpu_texture_private.hh @@ -93,7 +93,7 @@ class Texture { eGPUTextureType type_; /** Number of mipmaps this texture has (Max miplvl). */ - /* TODO(fclem) Should become immutable and the need for mipmaps should be specified upfront. */ + /* TODO(fclem): Should become immutable and the need for mipmaps should be specified upfront. */ int mipmaps_ = -1; /** For error checking */ int mip_min_ = 0, mip_max_ = 0; @@ -130,7 +130,7 @@ class Texture { virtual void update_sub( int mip, int offset[3], int extent[3], eGPUDataFormat format, const void *data) = 0; - /* TODO(fclem) Legacy. Should be removed at some point. */ + /* TODO(fclem): Legacy. Should be removed at some point. */ virtual uint gl_bindcode_get(void) const = 0; int width_get(void) const diff --git a/source/blender/gpu/intern/gpu_viewport.c b/source/blender/gpu/intern/gpu_viewport.c index 4a2ab30d74f..9063c8bdbce 100644 --- a/source/blender/gpu/intern/gpu_viewport.c +++ b/source/blender/gpu/intern/gpu_viewport.c @@ -112,7 +112,7 @@ struct GPUViewport { ColorManagedDisplaySettings display_settings; CurveMapping *orig_curve_mapping; float dither; - /* TODO(fclem) the uvimage display use the viewport but do not set any view transform for the + /* TODO(fclem): the uvimage display use the viewport but do not set any view transform for the * moment. The end goal would be to let the GPUViewport do the color management. */ bool do_color_management; struct GPUViewportBatch batch; |