Fix: VSE function has inverted logic

`seq_need_scale_to_render_size` returned true, when scaling was not
needed. The code that used this function worked correctly, so the logic
was inverted in both places.
This commit is contained in:
Richard Antalik
2025-06-15 06:49:46 +02:00
parent d9ef240311
commit 1fbd83f72e

View File

@@ -430,15 +430,15 @@ static bool seq_input_have_to_preprocess(const RenderData *context,
static bool seq_need_scale_to_render_size(const Strip *strip, bool is_proxy_image)
{
if (is_proxy_image) {
return true;
return false;
}
if ((strip->type & STRIP_TYPE_EFFECT) != 0 || strip->type == STRIP_TYPE_MASK ||
strip->type == STRIP_TYPE_META ||
(strip->type == STRIP_TYPE_SCENE && ((strip->flag & SEQ_SCENE_STRIPS) != 0)))
{
return true;
return false;
}
return false;
return true;
}
static float3x3 sequencer_image_crop_transform_matrix(const Strip *strip,
@@ -537,7 +537,7 @@ static void sequencer_preprocess_transform_crop(
float(scene->r.size) / 100 :
rendersize_to_scale_factor(context->preview_render_size);
const bool do_scale_to_render_size = seq_need_scale_to_render_size(strip, is_proxy_image);
const float image_scale_factor = do_scale_to_render_size ? 1.0f : preview_scale_factor;
const float image_scale_factor = do_scale_to_render_size ? preview_scale_factor : 1.0f;
float3x3 matrix = sequencer_image_crop_transform_matrix(
strip, in, out, image_scale_factor, preview_scale_factor);