Fix read access violation in the bitmapfont demo

[skaarj1989]

The font_interface should be cleared before Rml::Shutdown
A valid render_interface (non nullptr) is required for unloading TextureResource in the font_interface

RmlUi/Samples/basic/bitmapfont/src/FontEngineBitmap.cpp

Line 43 in c54ef6e

fonts.clear();

[mikke89]

Thank you! I've been keeping a note on this bug for too long now.

However, I'm not sure if this is a sufficient fix. Since we keep a reference to the font interface inside RmlUi until shutdown, we easily risk use-after free here. Plus, it breaks the shutdown in reverse initialization order, which is a good idiom to follow.

If you look at the Rml::Shutdown procedure, you can see the appropriate shutdown order for the default interface. Ideally we would have the same order for the bitmap font engine. Any ideas?

[mikke89]

I resolved this issue a bit differently: 64b36dc.

Thanks for reporting and the suggested solution.