Rémi Bernon (@rbernon) commented about dlls/winegstreamer/video_encoder.c:
}
+static HRESULT video_encoder_create_wg_transform(struct video_encoder *encoder) +{
- return wg_transform_create_mf(encoder->input_type, encoder->output_type,
&encoder->wg_transform_attrs, &encoder->wg_transform);
+}
+static void video_encoder_destroy_wg_transform(struct video_encoder *encoder) +{
- if (encoder->wg_transform)
- {
wg_transform_destroy(encoder->wg_transform);
encoder->wg_transform = 0;
- }
+}
Similarly here, I think it would be better for a potential future refactoring to keep the same code pattern and use a similar `try_create_wg_transform` helper as the other transform implementations.