[Bf-blender-cvs] [0ee52a5741] clay-engine: Get rid of runtime data struct.
Clément Foucault
noreply at git.blender.org
Fri Feb 3 13:36:53 CET 2017
Commit: 0ee52a5741a1fd831efff0d454bab3c6b00e0d1b
Author: Clément Foucault
Date: Fri Feb 3 13:00:15 2017 +0100
Branches: clay-engine
https://developer.blender.org/rB0ee52a5741a1fd831efff0d454bab3c6b00e0d1b
Get rid of runtime data struct.
===================================================================
M source/blender/blenkernel/intern/material.c
M source/blender/blenkernel/intern/scene.c
M source/blender/blenloader/intern/readfile.c
M source/blender/draw/DRW_engine.h
M source/blender/draw/engines/clay/clay.c
M source/blender/draw/intern/DRW_render.h
M source/blender/draw/intern/draw_manager.c
M source/blender/makesdna/DNA_material_types.h
M source/blender/makesdna/DNA_scene_types.h
M source/blender/makesrna/intern/rna_scene.c
===================================================================
diff --git a/source/blender/blenkernel/intern/material.c b/source/blender/blenkernel/intern/material.c
index 89af048f73..435504859a 100644
--- a/source/blender/blenkernel/intern/material.c
+++ b/source/blender/blenkernel/intern/material.c
@@ -113,8 +113,6 @@ void BKE_material_free(Material *ma)
BKE_previewimg_free(&ma->preview);
for (MaterialEngineSettings *mes = ma->engines_settings.first; mes; mes = mes->next) {
- if (mes->runtime)
- MEM_SAFE_FREE(mes->runtime);
if (mes->data)
MEM_SAFE_FREE(mes->data);
}
diff --git a/source/blender/blenkernel/intern/scene.c b/source/blender/blenkernel/intern/scene.c
index 807496d88e..cc136e8e74 100644
--- a/source/blender/blenkernel/intern/scene.c
+++ b/source/blender/blenkernel/intern/scene.c
@@ -570,8 +570,6 @@ void BKE_scene_free(Scene *sce)
/* Runtime Engine Data */
for (RenderEngineSettings *res = sce->engines_settings.first; res; res = res->next) {
- if (res->runtime)
- MEM_freeN(res->runtime);
if (res->data)
MEM_freeN(res->data);
}
diff --git a/source/blender/blenloader/intern/readfile.c b/source/blender/blenloader/intern/readfile.c
index cb69d54fc4..9206f4b65e 100644
--- a/source/blender/blenloader/intern/readfile.c
+++ b/source/blender/blenloader/intern/readfile.c
@@ -3994,7 +3994,6 @@ static void direct_link_material(FileData *fd, Material *ma)
link_list(fd, &ma->engines_settings);
for (mes = ma->engines_settings.first; mes; mes = mes->next) {
- mes->runtime = NULL;
mes->data = newdataadr(fd, mes->data);
}
}
@@ -6234,7 +6233,6 @@ static void direct_link_scene(FileData *fd, Scene *sce)
link_list(fd, &sce->engines_settings);
for (res = sce->engines_settings.first; res; res = res->next) {
- res->runtime = NULL;
res->data = newdataadr(fd, res->data);
}
}
diff --git a/source/blender/draw/DRW_engine.h b/source/blender/draw/DRW_engine.h
index 05222574ae..4f86e3fcd9 100644
--- a/source/blender/draw/DRW_engine.h
+++ b/source/blender/draw/DRW_engine.h
@@ -39,7 +39,7 @@ void DRW_engines_free(void);
void DRW_pass_free(struct DRWPass *pass);
/* Settings */
-void *DRW_material_settings_get(struct Material *ma, const char *engine_name, void ***runtime);
-void *DRW_render_settings_get(struct Scene *scene, const char *engine_name, void ***runtime);
+void *DRW_material_settings_get(struct Material *ma, const char *engine_name);
+void *DRW_render_settings_get(struct Scene *scene, const char *engine_name);
#endif /* __DRW_ENGINE_H__ */
\ No newline at end of file
diff --git a/source/blender/draw/engines/clay/clay.c b/source/blender/draw/engines/clay/clay.c
index 7be13c079f..3bdd48d319 100644
--- a/source/blender/draw/engines/clay/clay.c
+++ b/source/blender/draw/engines/clay/clay.c
@@ -84,8 +84,10 @@ static struct CLAY_data {
} data = {NULL};
/* CLAY_data.ubo_flag */
-#define CLAY_UBO_CLEAR (1 << 0)
-#define CLAY_UBO_REFRESH (1 << 1)
+enum {
+ CLAY_UBO_CLEAR = (1 << 0),
+ CLAY_UBO_REFRESH = (1 << 1),
+};
/* keep it under MAX_BUFFERS */
typedef struct CLAY_FramebufferList{
@@ -105,9 +107,11 @@ typedef struct CLAY_TextureList{
} CLAY_TextureList;
/* for clarity follow the same layout as CLAY_TextureList */
-#define SCENE_COLOR 0
-#define SCENE_DEPTH 1
-#define SCENE_DEPTH_LOW 2
+enum {
+ SCENE_COLOR,
+ SCENE_DEPTH,
+ SCENE_DEPTH_LOW,
+};
/* keep it under MAX_PASSES */
typedef struct CLAY_PassList{
@@ -268,8 +272,10 @@ MaterialEngineSettings *CLAY_material_settings_create(void)
return (MaterialEngineSettings *)settings;
}
-static void CLAY_engine_init(void)
+static void CLAY_engine_init(const bContext *C)
{
+ Main *bmain = CTX_data_main(C);
+
/* Create Texture Array */
if (!data.matcap_array) {
PreviewImage *prv[24]; /* For now use all of the 24 internal matcaps */
@@ -347,6 +353,28 @@ static void CLAY_engine_init(void)
BLI_dynstr_free(ds);
MEM_freeN(matcap_with_ao);
}
+
+ /* Cleanup all runtime data loaded from file */
+ for (Scene *sce = bmain->scene.first; sce; sce = sce->id.next) {
+ /* Using render settings as material settings */
+ MaterialEngineSettingsClay *res = DRW_render_settings_get(sce, RE_engine_id_BLENDER_CLAY);
+ res->flag = CLAY_OUTDATED;
+ res->ubo_index = -1;
+
+ /* Update Collections Materials */
+ for (SceneLayer *sl = sce->render_layers.first; sl; sl = sl->next) {
+ for (LayerCollection *lc = sl->layer_collections.first; lc; lc = lc->next) {
+ CollectionEngineSettings *ces;
+ ces = BKE_layer_collection_engine_get(lc, RE_engine_id_BLENDER_CLAY);
+ if (ces) { /* May not exists */
+ BKE_collection_engine_property_value_set_int(ces, "flag", CLAY_OUTDATED);
+ BKE_collection_engine_property_value_set_int(ces, "ubo_index", -1);
+ }
+ }
+ }
+ }
+
+ data.ubo_flag |= CLAY_UBO_REFRESH;
}
static void CLAY_ssao_setup(void)
@@ -362,7 +390,7 @@ static void CLAY_ssao_setup(void)
};
int i;
float *size = DRW_viewport_size_get();
- RenderEngineSettingsClay *settings = DRW_render_settings_get(NULL, RE_engine_id_BLENDER_CLAY, NULL);
+ RenderEngineSettingsClay *settings = DRW_render_settings_get(NULL, RE_engine_id_BLENDER_CLAY);
DRW_get_dfdy_factors(dfdyfacs);
@@ -425,100 +453,84 @@ static DRWShadingGroup *CLAY_shgroup_create(DRWPass *pass, int *UNUSED(material_
return grp;
}
-static void CLAY_update_material_runtime(MaterialEngineSettingsClay *settings, MaterialRuntimeClay **runtime)
+static void update_ubo_storage(float matcap_rot, float matcap_hue, float matcap_sat, float matcap_val,
+ float ssao_distance, float ssao_factor_cavity, float ssao_factor_edge,
+ float ssao_attenuation, int matcap_icon, unsigned int current_id)
{
- MaterialRuntimeClay *rt;
-
- if (*runtime == NULL) {
- rt = MEM_mallocN(sizeof(MaterialRuntimeClay), "MaterialRuntimeClay");
- rt->flag = CLAY_OUTDATED;
- data.ubo_flag |= CLAY_UBO_CLEAR;
- *runtime = rt;
- }
-
- rt = *runtime;
-
- if (rt->flag & CLAY_OUTDATED) {
-
- /* Update default material */
- rt->matcap_rot[0] = cosf(settings->matcap_rot * 3.14159f * 2.0f);
- rt->matcap_rot[1] = sinf(settings->matcap_rot * 3.14159f * 2.0f);
-
- rt->matcap_hsv[0] = settings->matcap_hue + 0.5f;
- rt->matcap_hsv[1] = settings->matcap_sat * 2.0f;
- rt->matcap_hsv[2] = settings->matcap_val * 2.0f;
-
- rt->ssao_params_var[0] = settings->ssao_distance;
- rt->ssao_params_var[1] = settings->ssao_factor_cavity;
- rt->ssao_params_var[2] = settings->ssao_factor_edge;
- rt->ssao_params_var[3] = settings->ssao_attenuation;
-
- if (settings->matcap_icon < ICON_MATCAP_01 ||
- settings->matcap_icon > ICON_MATCAP_24)
- {
- settings->matcap_icon = ICON_MATCAP_01;
- }
-
- rt->matcap_id = matcap_to_index(settings->matcap_icon);
+ CLAY_UBO_Material *ubo = &data.mat_storage.materials[current_id];
- if ((rt->type != settings->type)) {
- data.ubo_flag |= CLAY_UBO_CLEAR;
- }
+ ubo->matcap_rot[0] = cosf(matcap_rot * 3.14159f * 2.0f);
+ ubo->matcap_rot[1] = sinf(matcap_rot * 3.14159f * 2.0f);
- rt->type = settings->type;
+ ubo->matcap_hsv[0] = matcap_hue + 0.5f;
+ ubo->matcap_hsv[1] = matcap_sat * 2.0f;
+ ubo->matcap_hsv[2] = matcap_val * 2.0f;
- data.ubo_flag |= CLAY_UBO_REFRESH;
- rt->flag &= ~CLAY_OUTDATED;
- }
-}
+ ubo->ssao_params_var[0] = ssao_distance;
+ ubo->ssao_params_var[1] = ssao_factor_cavity;
+ ubo->ssao_params_var[2] = ssao_factor_edge;
+ ubo->ssao_params_var[3] = ssao_attenuation;
-static void update_ubo_storage(MaterialRuntimeClay *runtime, unsigned int current_id)
-{
- CLAY_UBO_Material *ubo = &data.mat_storage.materials[current_id];
- ubo->matcap_id = runtime->matcap_id;
- copy_v3_v3(ubo->matcap_hsv, runtime->matcap_hsv);
- copy_v2_v2(ubo->matcap_rot, runtime->matcap_rot);
- copy_v4_v4(ubo->ssao_params_var, runtime->ssao_params_var);
-
- runtime->material_id = current_id;
+ ubo->matcap_id = matcap_to_index(matcap_icon);
}
static void CLAY_update_material_ubo(const struct bContext *C)
{
Main *bmain = CTX_data_main(C);
- MaterialRuntimeClay **runtime;
/* Update Default materials */
for (Scene *sce = bmain->scene.first; sce; sce = sce->id.next) {
/* Using render settings as material settings */
- MaterialEngineSettingsClay *res = DRW_render_settings_get(sce, RE_engine_id_BLENDER_CLAY, (void ***)&runtime);
- CLAY_update_material_runtime(res, runtime);
- }
+ MaterialEngineSettingsClay *res = DRW_render_settings_get(sce, RE_engine_id_BLENDER_CLAY);
+
+ if (res->flag & CLAY_OUTDATED)
+ data.ubo_flag |= CLAY_UBO_REFRESH;
- /* Update Scene Materials */
- for (Material *mat = bmain->mat.first; mat; mat = mat->id.next) {
- MaterialEngineSettingsClay *mesc = DRW_material_settings_get(mat, RE_engine_id_BLENDER_CLAY, (void ***)&runtime);
- CLAY_update_material_runtime(mesc, runtime);
+ if (res->matcap_icon < ICON_MATCAP_01 ||
+ res->matcap_icon > ICON_MATCAP_24)
+ {
+ res->matcap_icon = ICON_MATCAP_01;
+ }
+
+ res->flag &= ~CLAY_OUTDATED;
+
+
+ /* Update Collections Materials */
+ for (SceneLayer *sl = sce->render_layers.first; sl; sl = sl->next) {
+ for (LayerCollection *lc = sl->layer_collections.first; lc; lc = lc->next) {
+ CollectionEngineSettings *ces;
+ ces = BKE_layer_collection_engine_get(lc, RE_engine_id_BLENDER_CLAY);
+
+ BKE_collection_engine_property_value_set_int(ces, "flag", 0);
+ BKE_collection_engine_property_value_set_int(ces, "ubo_index", 0);
+ }
+ }
}
if (data.ubo_flag & CLAY_UBO_REFRESH) {
int current_id = 0;
+
/* Default materials */
for (Scene *sce = bmain->scene.first; sce; sce = sce->id.next) {
- DRW_render_settings_get(sce, RE_engine_id_BLENDER_CLAY, (void ***)&runtime);
- update_ubo_storage(*runtime, current_id);
+ MaterialEngineSettingsClay *res = DRW_render_settings_get(sce, RE_engine_id_BLENDER_CLAY);
+
+ update_ubo_storage(res->matcap_rot, res->matcap_hue, res->matcap_sat, res->matcap_val,
+ res->ssao_distance, res->ssao_factor_cavity, res->ssao_factor_edge,
+ res->ssao_attenuation, res->matcap_icon, current_id);
current_id++;
- }
- /* TODO only add materials linked to geometry */
- for (Material *mat = bmain->mat.first; mat; mat = mat->id.next) {
- DRW_material_settings_get(mat, RE_engine_id_BLENDER_CL
@@ Diff output truncated at 10240 characters. @@
More information about the Bf-blender-cvs
mailing list