obs-text: Render only when changed

[exeldro]

Description

Render text only when changed

Motivation and Context

Rendering text can take lots of time (in my testing example 86 ms), so in case nothing changed, like for example a obs-websocket request to update the text to the same value, the rendering step can be skipped.

How Has This Been Tested?

On windows 11 by changing a text file without resulting text changes to be rendered and by calling SetInputSettings on obs-websockets

Types of changes

• Performance enhancement (non-breaking change which improves efficiency)

Checklist:

• My code has been run through clang-format.
• I have read the contributing document.
• My code is not on the master branch.
• The code has been tested.
• All commit messages are properly formatted and commits squashed where appropriate.
• I have included updates to all appropriate documentation.

[RytoEX]

Nits. Don't know enough about these code changes to make a quick call, so hopefully someone else familiar with them can give it a look.

[RytoEX]

Probably fine as-is, as we'll be doing a pass with clang-format, but might as well add braces now if time permits.

[RytoEX]

Suggested change

	if (color != new_color || opacity != new_opacity ||
	(gradient && color2 != new_color2) ||
	(gradient && opacity2 != new_opacity2) ||
	gradient_dir != new_grad_dir || vertical != new_vertical ||
	bk_color != new_bk_color || bk_opacity != new_bk_opacity ||
	use_extents != new_extents || wrap != new_extents_wrap ||
	extents_cx != n_extents_cx || extents_cy != n_extents_cy ||
	antialiasing != new_antialiasing)
	needs_render = true;
	if (color != new_color || opacity != new_opacity ||
	(gradient && color2 != new_color2) ||
	(gradient && opacity2 != new_opacity2) ||
	gradient_dir != new_grad_dir || vertical != new_vertical ||
	bk_color != new_bk_color || bk_opacity != new_bk_opacity ||
	use_extents != new_extents || wrap != new_extents_wrap ||
	extents_cx != n_extents_cx || extents_cy != n_extents_cy ||
	antialiasing != new_antialiasing) {
	needs_render = true;
	}

[RytoEX]

Suggested change

	if (old_text != text || use_outline != new_outline ||
	outline_color != new_o_color || outline_opacity != new_o_opacity ||
	outline_size != roundf(float(new_o_size)))
	needs_render = true;
	if (old_text != text || use_outline != new_outline ||
	outline_color != new_o_color || outline_opacity != new_o_opacity ||
	outline_size != roundf(float(new_o_size))) {
	needs_render = true;
	}

[RytoEX]

Suggested change

	if (needs_render)
	RenderText();
	if (needs_render) {
	RenderText();
	}

[RytoEX]

Suggested change

	if (old_text != text)
	RenderText();
	if (old_text != text) {
	RenderText();
	}