[3.x] BakedLightmap add null bake_end_function check
This commit is contained in:
parent
4769aa4499
commit
85811c2b99
1 changed files with 30 additions and 10 deletions
|
@ -616,14 +616,18 @@ BakedLightmap::BakeError BakedLightmap::bake(Node *p_from_node, String p_data_sa
|
|||
if (bake_step_function) {
|
||||
bool cancelled = bake_step_function(0.0, TTR("Finding meshes and lights"), nullptr, true);
|
||||
if (cancelled) {
|
||||
bake_end_function(time_started);
|
||||
if (bake_end_function) {
|
||||
bake_end_function(time_started);
|
||||
}
|
||||
return BAKE_ERROR_USER_ABORTED;
|
||||
}
|
||||
}
|
||||
|
||||
Ref<Lightmapper> lightmapper = Lightmapper::create();
|
||||
if (lightmapper.is_null()) {
|
||||
bake_end_function(time_started);
|
||||
if (bake_end_function) {
|
||||
bake_end_function(time_started);
|
||||
}
|
||||
return BAKE_ERROR_NO_LIGHTMAPPER;
|
||||
}
|
||||
|
||||
|
@ -633,7 +637,9 @@ BakedLightmap::BakeError BakedLightmap::bake(Node *p_from_node, String p_data_sa
|
|||
_find_meshes_and_lights(p_from_node ? p_from_node : get_parent(), meshes_found, lights_found);
|
||||
|
||||
if (meshes_found.size() == 0) {
|
||||
bake_end_function(time_started);
|
||||
if (bake_end_function) {
|
||||
bake_end_function(time_started);
|
||||
}
|
||||
return BAKE_ERROR_NO_MESHES;
|
||||
}
|
||||
|
||||
|
@ -642,7 +648,9 @@ BakedLightmap::BakeError BakedLightmap::bake(Node *p_from_node, String p_data_sa
|
|||
float p = (float)(m_i) / meshes_found.size();
|
||||
bool cancelled = bake_step_function(p * 0.05, vformat(TTR("Preparing geometry (%d/%d)"), m_i + 1, meshes_found.size()), nullptr, false);
|
||||
if (cancelled) {
|
||||
bake_end_function(time_started);
|
||||
if (bake_end_function) {
|
||||
bake_end_function(time_started);
|
||||
}
|
||||
return BAKE_ERROR_USER_ABORTED;
|
||||
}
|
||||
}
|
||||
|
@ -845,7 +853,9 @@ BakedLightmap::BakeError BakedLightmap::bake(Node *p_from_node, String p_data_sa
|
|||
Lightmapper::BakeError bake_err = lightmapper->bake(Lightmapper::BakeQuality(bake_quality), use_denoiser, bounces, bounce_indirect_energy, bias, gen_atlas, max_atlas_size, environment_image, environment_xform, _lightmap_bake_step_function, &bsud, bake_substep_function);
|
||||
|
||||
if (bake_err != Lightmapper::BAKE_OK) {
|
||||
bake_end_function(time_started);
|
||||
if (bake_end_function) {
|
||||
bake_end_function(time_started);
|
||||
}
|
||||
switch (bake_err) {
|
||||
case Lightmapper::BAKE_ERROR_USER_ABORTED: {
|
||||
return BAKE_ERROR_USER_ABORTED;
|
||||
|
@ -875,7 +885,9 @@ BakedLightmap::BakeError BakedLightmap::bake(Node *p_from_node, String p_data_sa
|
|||
if (bake_step_function) {
|
||||
bool cancelled = bake_step_function(0.85, TTR("Generating capture"), nullptr, true);
|
||||
if (cancelled) {
|
||||
bake_end_function(time_started);
|
||||
if (bake_end_function) {
|
||||
bake_end_function(time_started);
|
||||
}
|
||||
return BAKE_ERROR_USER_ABORTED;
|
||||
}
|
||||
}
|
||||
|
@ -955,7 +967,9 @@ BakedLightmap::BakeError BakedLightmap::bake(Node *p_from_node, String p_data_sa
|
|||
if (bake_step_function) {
|
||||
bool cancelled = bake_step_function(0.9, TTR("Saving lightmaps"), nullptr, true);
|
||||
if (cancelled) {
|
||||
bake_end_function(time_started);
|
||||
if (bake_end_function) {
|
||||
bake_end_function(time_started);
|
||||
}
|
||||
return BAKE_ERROR_USER_ABORTED;
|
||||
}
|
||||
}
|
||||
|
@ -1102,7 +1116,9 @@ BakedLightmap::BakeError BakedLightmap::bake(Node *p_from_node, String p_data_sa
|
|||
if (bake_step_function) {
|
||||
bool cancelled = bake_step_function(1.0, TTR("Done"), nullptr, true);
|
||||
if (cancelled) {
|
||||
bake_end_function(time_started);
|
||||
if (bake_end_function) {
|
||||
bake_end_function(time_started);
|
||||
}
|
||||
return BAKE_ERROR_USER_ABORTED;
|
||||
}
|
||||
}
|
||||
|
@ -1111,12 +1127,16 @@ BakedLightmap::BakeError BakedLightmap::bake(Node *p_from_node, String p_data_sa
|
|||
data->set_path(p_data_save_path);
|
||||
|
||||
if (err != OK) {
|
||||
bake_end_function(time_started);
|
||||
if (bake_end_function) {
|
||||
bake_end_function(time_started);
|
||||
}
|
||||
return BAKE_ERROR_CANT_CREATE_IMAGE;
|
||||
}
|
||||
|
||||
set_light_data(data);
|
||||
bake_end_function(time_started);
|
||||
if (bake_end_function) {
|
||||
bake_end_function(time_started);
|
||||
}
|
||||
|
||||
return BAKE_ERROR_OK;
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue