DRW: Codestyle refactor: Use macro to create shader libs.

This clears up the code from many DynStr usage. Easier to read.

5 files changed, 118 insertions, 148 deletions

diff --git a/source/blender/draw/engines/eevee/eevee_lightprobes.c b/source/blender/draw/engines/eevee/eevee_lightprobes.c
index dd49ae0481e..73a513d5794 100644
--- a/source/blender/draw/engines/eevee/eevee_lightprobes.c
+++ b/source/blender/draw/engines/eevee/eevee_lightprobes.c
@@ -204,71 +204,54 @@ static void lightprobe_shaders_init(void)
 
 	char *shader_str = NULL;
 
-	DynStr *ds_frag = BLI_dynstr_new();
-	BLI_dynstr_append(ds_frag, datatoc_bsdf_common_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_bsdf_sampling_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_lightprobe_filter_glossy_frag_glsl);
-	shader_str = BLI_dynstr_get_cstring(ds_frag);
-	BLI_dynstr_free(ds_frag);
-
-	e_data.probe_filter_glossy_sh = DRW_shader_create(
-	        datatoc_lightprobe_vert_glsl, datatoc_lightprobe_geom_glsl, shader_str, filter_defines);
-
-	e_data.probe_default_sh = DRW_shader_create(
-	        datatoc_background_vert_glsl, NULL, datatoc_default_world_frag_glsl, NULL);
-
+	DRW_shader_create_lib(shader_str,
+	                      datatoc_bsdf_common_lib_glsl,
+	                      datatoc_bsdf_sampling_lib_glsl,
+	                      datatoc_lightprobe_filter_glossy_frag_glsl);
+	e_data.probe_filter_glossy_sh = DRW_shader_create(datatoc_lightprobe_vert_glsl,
+	                                                  datatoc_lightprobe_geom_glsl,
+	                                                  shader_str,
+	                                                  filter_defines);
 	MEM_freeN(shader_str);
 
-	ds_frag = BLI_dynstr_new();
-	BLI_dynstr_append(ds_frag, datatoc_bsdf_common_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_bsdf_sampling_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_lightprobe_filter_diffuse_frag_glsl);
-	shader_str = BLI_dynstr_get_cstring(ds_frag);
-	BLI_dynstr_free(ds_frag);
+	e_data.probe_default_sh = DRW_shader_create(datatoc_background_vert_glsl, NULL,
+	                                            datatoc_default_world_frag_glsl, NULL);
 
+	DRW_shader_create_lib(shader_str,
+	                      datatoc_bsdf_common_lib_glsl,
+	                      datatoc_bsdf_sampling_lib_glsl,
+	                      datatoc_lightprobe_filter_diffuse_frag_glsl);
 	e_data.probe_filter_diffuse_sh = DRW_shader_create_fullscreen(shader_str, filter_defines);
-
 	MEM_freeN(shader_str);
 
-	ds_frag = BLI_dynstr_new();
-	BLI_dynstr_append(ds_frag, datatoc_bsdf_common_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_bsdf_sampling_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_lightprobe_filter_visibility_frag_glsl);
-	shader_str = BLI_dynstr_get_cstring(ds_frag);
-	BLI_dynstr_free(ds_frag);
+	DRW_shader_create_lib(shader_str,
+	                      datatoc_bsdf_common_lib_glsl,
+	                      datatoc_bsdf_sampling_lib_glsl,
+	                      datatoc_lightprobe_filter_visibility_frag_glsl);
 
 	e_data.probe_filter_visibility_sh = DRW_shader_create_fullscreen(shader_str, filter_defines);
 
 	MEM_freeN(shader_str);
 
-	ds_frag = BLI_dynstr_new();
-	BLI_dynstr_append(ds_frag, datatoc_octahedron_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_bsdf_common_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_irradiance_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_lightprobe_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_lightprobe_grid_display_frag_glsl);
-	shader_str = BLI_dynstr_get_cstring(ds_frag);
-	BLI_dynstr_free(ds_frag);
-
-	e_data.probe_grid_display_sh = DRW_shader_create(
-	        datatoc_lightprobe_grid_display_vert_glsl, NULL, shader_str, filter_defines);
-
+	DRW_shader_create_lib(shader_str,
+	                      datatoc_octahedron_lib_glsl,
+	                      datatoc_bsdf_common_lib_glsl,
+	                      datatoc_irradiance_lib_glsl,
+	                      datatoc_lightprobe_lib_glsl,
+	                      datatoc_lightprobe_grid_display_frag_glsl);
+	e_data.probe_grid_display_sh = DRW_shader_create(datatoc_lightprobe_grid_display_vert_glsl, NULL,
+	                                                 shader_str, filter_defines);
 	MEM_freeN(shader_str);
 
-	e_data.probe_grid_fill_sh = DRW_shader_create_fullscreen(
-	        datatoc_lightprobe_grid_fill_frag_glsl, filter_defines);
-
-	ds_frag = BLI_dynstr_new();
-	BLI_dynstr_append(ds_frag, datatoc_octahedron_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_bsdf_common_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_lightprobe_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_lightprobe_cube_display_frag_glsl);
-	shader_str = BLI_dynstr_get_cstring(ds_frag);
-	BLI_dynstr_free(ds_frag);
+	e_data.probe_grid_fill_sh = DRW_shader_create_fullscreen(datatoc_lightprobe_grid_fill_frag_glsl, filter_defines);
 
+	DRW_shader_create_lib(shader_str,
+	                      datatoc_octahedron_lib_glsl,
+	                      datatoc_bsdf_common_lib_glsl,
+	                      datatoc_lightprobe_lib_glsl,
+	                      datatoc_lightprobe_cube_display_frag_glsl);
 	e_data.probe_cube_display_sh = DRW_shader_create(
 	        datatoc_lightprobe_cube_display_vert_glsl, NULL, shader_str, NULL);
-
 	MEM_freeN(shader_str);
 
 	e_data.probe_planar_display_sh = DRW_shader_create(
diff --git a/source/blender/draw/engines/eevee/eevee_materials.c b/source/blender/draw/engines/eevee/eevee_materials.c
index 8149c105a81..7628ff05756 100644
--- a/source/blender/draw/engines/eevee/eevee_materials.c
+++ b/source/blender/draw/engines/eevee/eevee_materials.c
@@ -47,7 +47,9 @@
 /* *********** STATIC *********** */
 static struct {
 	char *frag_shader_lib;
+	char *prepass_shader_lib;
 	char *volume_shader_lib;
+	char *default_frag_glsl;
 
 	struct GPUShader *default_prepass_sh;
 	struct GPUShader *default_prepass_clip_sh;
@@ -116,10 +118,11 @@ static struct GPUTexture *create_ggx_lut_texture(int UNUSED(w), int UNUSED(h))
 	BLI_dynstr_free(ds_vert);
 
 	struct GPUShader *sh = DRW_shader_create_with_lib(
-	        datatoc_lightprobe_vert_glsl, datatoc_lightprobe_geom_glsl, datatoc_bsdf_lut_frag_glsl, lib_str,
+	        datatoc_lightprobe_vert_glsl, datatoc_lightprobe_geom_glsl, datatoc_bsdf_lut_frag_glsl,
 	        "#define HAMMERSLEY_SIZE 8192\n"
 	        "#define BRDF_LUT_SIZE 64\n"
-	        "#define NOISE_SIZE 64\n");
+	        "#define NOISE_SIZE 64\n",
+	        lib_str);
 
 	DRWPass *pass = DRW_pass_create("LightProbe Filtering", DRW_STATE_WRITE_COLOR);
 	DRWShadingGroup *grp = DRW_shgroup_create(sh, pass);
@@ -418,18 +421,12 @@ static void add_standard_uniforms(
 
 static void create_default_shader(int options)
 {
-	DynStr *ds_frag = BLI_dynstr_new();
-	BLI_dynstr_append(ds_frag, e_data.frag_shader_lib);
-	BLI_dynstr_append(ds_frag, datatoc_default_frag_glsl);
-	char *frag_str = BLI_dynstr_get_cstring(ds_frag);
-	BLI_dynstr_free(ds_frag);
-
 	char *defines = eevee_get_defines(options);
 
-	e_data.default_lit[options] = DRW_shader_create(datatoc_lit_surface_vert_glsl, NULL, frag_str, defines);
+	e_data.default_lit[options] = DRW_shader_create(datatoc_lit_surface_vert_glsl, NULL,
+	                                                e_data.default_frag_glsl, defines);
 
 	MEM_freeN(defines);
-	MEM_freeN(frag_str);
 }
 
 void EEVEE_update_util_texture(double offsets[3])
@@ -489,48 +486,47 @@ void EEVEE_update_util_texture(double offsets[3])
 void EEVEE_materials_init(EEVEE_StorageList *stl)
 {
 	if (!e_data.frag_shader_lib) {
-		char *frag_str = NULL;
-
-		/* Shaders */
-		DynStr *ds_frag = BLI_dynstr_new();
-		BLI_dynstr_append(ds_frag, datatoc_bsdf_common_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_bsdf_sampling_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_ambient_occlusion_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_raytrace_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_ssr_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_octahedron_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_irradiance_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_lightprobe_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_ltc_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_bsdf_direct_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_lamps_lib_glsl);
-		for (int i = 0; i < 7; ++i) {
-			/* Add one for each Closure */
-			BLI_dynstr_append(ds_frag, datatoc_lit_surface_frag_glsl);
-		}
-		BLI_dynstr_append(ds_frag, datatoc_volumetric_lib_glsl);
-		e_data.frag_shader_lib = BLI_dynstr_get_cstring(ds_frag);
-		BLI_dynstr_free(ds_frag);
-
-		ds_frag = BLI_dynstr_new();
-		BLI_dynstr_append(ds_frag, datatoc_bsdf_common_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_ambient_occlusion_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_octahedron_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_irradiance_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_lightprobe_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_ltc_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_bsdf_direct_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_lamps_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_volumetric_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_volumetric_frag_glsl);
-		e_data.volume_shader_lib = BLI_dynstr_get_cstring(ds_frag);
-		BLI_dynstr_free(ds_frag);
-
-		ds_frag = BLI_dynstr_new();
-		BLI_dynstr_append(ds_frag, e_data.frag_shader_lib);
-		BLI_dynstr_append(ds_frag, datatoc_default_frag_glsl);
-		frag_str = BLI_dynstr_get_cstring(ds_frag);
-		BLI_dynstr_free(ds_frag);
+		DRW_shader_create_lib(e_data.frag_shader_lib,
+		                      datatoc_bsdf_common_lib_glsl,
+		                      datatoc_bsdf_sampling_lib_glsl,
+		                      datatoc_ambient_occlusion_lib_glsl,
+		                      datatoc_raytrace_lib_glsl,
+		                      datatoc_ssr_lib_glsl,
+		                      datatoc_octahedron_lib_glsl,
+		                      datatoc_irradiance_lib_glsl,
+		                      datatoc_lightprobe_lib_glsl,
+		                      datatoc_ltc_lib_glsl,
+		                      datatoc_bsdf_direct_lib_glsl,
+		                      datatoc_lamps_lib_glsl,
+		                      /* Add one for each Closure */
+		                      datatoc_lit_surface_frag_glsl,
+		                      datatoc_lit_surface_frag_glsl,
+		                      datatoc_lit_surface_frag_glsl,
+		                      datatoc_lit_surface_frag_glsl,
+		                      datatoc_lit_surface_frag_glsl,
+		                      datatoc_lit_surface_frag_glsl,
+		                      datatoc_lit_surface_frag_glsl,
+		                      datatoc_volumetric_lib_glsl);
+
+		DRW_shader_create_lib(e_data.volume_shader_lib,
+		                      datatoc_bsdf_common_lib_glsl,
+		                      datatoc_ambient_occlusion_lib_glsl,
+		                      datatoc_octahedron_lib_glsl,
+		                      datatoc_irradiance_lib_glsl,
+		                      datatoc_lightprobe_lib_glsl,
+		                      datatoc_ltc_lib_glsl,
+		                      datatoc_bsdf_direct_lib_glsl,
+		                      datatoc_lamps_lib_glsl,
+		                      datatoc_volumetric_lib_glsl,
+		                      datatoc_volumetric_frag_glsl);
+
+		DRW_shader_create_lib(e_data.default_frag_glsl,
+		                      e_data.frag_shader_lib,
+		                      datatoc_default_frag_glsl);
+
+		DRW_shader_create_lib(e_data.prepass_shader_lib,
+		                      e_data.frag_shader_lib,
+		                      datatoc_prepass_frag_glsl);
 
 		e_data.default_background = DRW_shader_create(
 		        datatoc_background_vert_glsl, NULL, datatoc_default_world_frag_glsl,
@@ -544,8 +540,6 @@ void EEVEE_materials_init(EEVEE_StorageList *stl)
 		        datatoc_prepass_vert_glsl, NULL, datatoc_prepass_frag_glsl,
 		        "#define CLIP_PLANES\n");
 
-		MEM_freeN(frag_str);
-
 		double offsets[3] = {0.0, 0.0, 0.0};
 		EEVEE_update_util_texture(offsets);
 	}
@@ -739,20 +733,13 @@ struct GPUMaterial *EEVEE_material_mesh_depth_get(
 
 	char *defines = eevee_get_defines(options);
 
-	DynStr *ds_frag = BLI_dynstr_new();
-	BLI_dynstr_append(ds_frag, e_data.frag_shader_lib);
-	BLI_dynstr_append(ds_frag, datatoc_prepass_frag_glsl);
-	char *frag_str = BLI_dynstr_get_cstring(ds_frag);
-	BLI_dynstr_free(ds_frag);
-
 	mat = GPU_material_from_nodetree(
 	        scene, ma->nodetree, &ma->gpumaterial, engine, options,
 	        (is_shadow) ? datatoc_shadow_vert_glsl : datatoc_lit_surface_vert_glsl,
 	        (is_shadow) ? datatoc_shadow_geom_glsl : NULL,
-	        frag_str,
+	        e_data.prepass_shader_lib,
 	        defines);
 
-	MEM_freeN(frag_str);
 	MEM_freeN(defines);
 
 	return mat;
@@ -1449,7 +1436,9 @@ void EEVEE_materials_free(void)
 		DRW_SHADER_FREE_SAFE(e_data.default_lit[i]);
 	}
 	MEM_SAFE_FREE(e_data.frag_shader_lib);
+	MEM_SAFE_FREE(e_data.prepass_shader_lib);
 	MEM_SAFE_FREE(e_data.volume_shader_lib);
+	MEM_SAFE_FREE(e_data.default_frag_glsl);
 	DRW_SHADER_FREE_SAFE(e_data.default_prepass_sh);
 	DRW_SHADER_FREE_SAFE(e_data.default_prepass_clip_sh);
 	DRW_SHADER_FREE_SAFE(e_data.default_background);
diff --git a/source/blender/draw/engines/eevee/eevee_occlusion.c b/source/blender/draw/engines/eevee/eevee_occlusion.c
index 0ae5fb91db3..bf76e9350a1 100644
--- a/source/blender/draw/engines/eevee/eevee_occlusion.c
+++ b/source/blender/draw/engines/eevee/eevee_occlusion.c
@@ -49,12 +49,12 @@ extern char datatoc_effect_gtao_frag_glsl[];
 
 static void eevee_create_shader_occlusion(void)
 {
-	DynStr *ds_frag = BLI_dynstr_new();
-	BLI_dynstr_append(ds_frag, datatoc_bsdf_common_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_ambient_occlusion_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_effect_gtao_frag_glsl);
-	char *frag_str = BLI_dynstr_get_cstring(ds_frag);
-	BLI_dynstr_free(ds_frag);
+	char *frag_str;
+
+	DRW_shader_create_lib(frag_str,
+	                      datatoc_bsdf_common_lib_glsl,
+	                      datatoc_ambient_occlusion_lib_glsl,
+	                      datatoc_effect_gtao_frag_glsl);
 
 	e_data.gtao_sh = DRW_shader_create_fullscreen(frag_str, NULL);
 	e_data.gtao_layer_sh = DRW_shader_create_fullscreen(frag_str, "#define LAYERED_DEPTH\n");
diff --git a/source/blender/draw/engines/eevee/eevee_screen_raytrace.c b/source/blender/draw/engines/eevee/eevee_screen_raytrace.c
index f4b8ab807f6..f61d94c17eb 100644
--- a/source/blender/draw/engines/eevee/eevee_screen_raytrace.c
+++ b/source/blender/draw/engines/eevee/eevee_screen_raytrace.c
@@ -59,16 +59,16 @@ extern char datatoc_raytrace_lib_glsl[];
 static struct GPUShader *eevee_effects_screen_raytrace_shader_get(int options)
 {
 	if (e_data.ssr_sh[options] == NULL) {
-		DynStr *ds_frag = BLI_dynstr_new();
-		BLI_dynstr_append(ds_frag, datatoc_bsdf_common_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_bsdf_sampling_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_octahedron_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_lightprobe_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_ambient_occlusion_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_raytrace_lib_glsl);
-		BLI_dynstr_append(ds_frag, datatoc_effect_ssr_frag_glsl);
-		char *ssr_shader_str = BLI_dynstr_get_cstring(ds_frag);
-		BLI_dynstr_free(ds_frag);
+		char *ssr_shader_str;
+
+		DRW_shader_create_lib(ssr_shader_str,
+		                      datatoc_bsdf_common_lib_glsl,
+		                      datatoc_bsdf_sampling_lib_glsl,
+		                      datatoc_octahedron_lib_glsl,
+		                      datatoc_lightprobe_lib_glsl,
+		                      datatoc_ambient_occlusion_lib_glsl,
+		                      datatoc_raytrace_lib_glsl,
+		                      datatoc_effect_ssr_frag_glsl);
 
 		DynStr *ds_defines = BLI_dynstr_new();
 		BLI_dynstr_appendf(ds_defines, SHADER_DEFINES);
diff --git a/source/blender/draw/engines/eevee/eevee_volumes.c b/source/blender/draw/engines/eevee/eevee_volumes.c
index dae4503dc32..db5ee19172e 100644
--- a/source/blender/draw/engines/eevee/eevee_volumes.c
+++ b/source/blender/draw/engines/eevee/eevee_volumes.c
@@ -77,46 +77,44 @@ extern char datatoc_gpu_shader_fullscreen_vert_glsl[];
 
 static void eevee_create_shader_volumes(void)
 {
-	DynStr *ds_frag = BLI_dynstr_new();
-	BLI_dynstr_append(ds_frag, datatoc_bsdf_common_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_volumetric_lib_glsl);
-	e_data.volumetric_common_lib = BLI_dynstr_get_cstring(ds_frag);
-	BLI_dynstr_free(ds_frag);
-
-	ds_frag = BLI_dynstr_new();
-	BLI_dynstr_append(ds_frag, datatoc_bsdf_common_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_bsdf_direct_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_octahedron_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_irradiance_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_lamps_lib_glsl);
-	BLI_dynstr_append(ds_frag, datatoc_volumetric_lib_glsl);
-	e_data.volumetric_common_lamps_lib = BLI_dynstr_get_cstring(ds_frag);
-	BLI_dynstr_free(ds_frag);
+	DRW_shader_create_lib(e_data.volumetric_common_lib,
+	                      datatoc_bsdf_common_lib_glsl,
+	                      datatoc_volumetric_lib_glsl);
+
+	DRW_shader_create_lib(e_data.volumetric_common_lamps_lib,
+	                      datatoc_bsdf_common_lib_glsl,
+	                      datatoc_bsdf_direct_lib_glsl,
+	                      datatoc_octahedron_lib_glsl,
+	                      datatoc_irradiance_lib_glsl,
+	                      datatoc_lamps_lib_glsl,
+	                      datatoc_volumetric_lib_glsl);
 
 	e_data.volumetric_clear_sh = DRW_shader_create_with_lib(
 	        datatoc_volumetric_vert_glsl,
 	        datatoc_volumetric_geom_glsl,
 	        datatoc_volumetric_frag_glsl,
-	        e_data.volumetric_common_lib,
 	        "#define VOLUMETRICS\n"
-	        "#define CLEAR\n");
+	        "#define CLEAR\n",
+	        e_data.volumetric_common_lib);
 	e_data.volumetric_scatter_sh = DRW_shader_create_with_lib(
 	        datatoc_volumetric_vert_glsl,
 	        datatoc_volumetric_geom_glsl,
 	        datatoc_volumetric_scatter_frag_glsl,
-	        e_data.volumetric_common_lamps_lib,
 	        SHADER_DEFINES
 	        "#define VOLUMETRICS\n"
-	        "#define VOLUME_SHADOW\n");
+	        "#define VOLUME_SHADOW\n",
+	        e_data.volumetric_common_lamps_lib);
 	e_data.volumetric_integration_sh = DRW_shader_create_with_lib(
 	        datatoc_volumetric_vert_glsl,
 	        datatoc_volumetric_geom_glsl,
 	        datatoc_volumetric_integration_frag_glsl,
-	        e_data.volumetric_common_lib, NULL);
+	        NULL,
+	        e_data.volumetric_common_lib);
 	e_data.volumetric_resolve_sh = DRW_shader_create_with_lib(
 	        datatoc_gpu_shader_fullscreen_vert_glsl, NULL,
 	        datatoc_volumetric_resolve_frag_glsl,
-	        e_data.volumetric_common_lib, NULL);
+	        NULL,
+	        e_data.volumetric_common_lib);
 }
 
 int EEVEE_volumes_init(EEVEE_ViewLayerData *sldata, EEVEE_Data *vedata)