Code review comment for lp:~widelands-dev/widelands/fonts

Revision history for this message
GunChleoc (gunchleoc) wrote :

Duh, I didn't get it that you were running an external test - I only tested this in-game. I take it the old richtext renderer is used in scenario dialogs and / the building help?

When I run this:

./build/src/graphic/text/test/wl_render_richtext 373 blub.png src/graphic/text/test/data/sub_fixedwidth_floatright/input00.txt

I do indeed get a segfault. So, we need to give g_fh a fontset too, and change it every time we change it for g_fh1.

« Back to merge proposal