]> git.sesse.net Git - vlc/commit
Run FontBuilder thread after Freetype has been initialized and no error paths are...
authorJean-Paul Saman <jpsaman@videolan.org>
Mon, 8 Oct 2007 20:46:23 +0000 (20:46 +0000)
committerJean-Paul Saman <jpsaman@videolan.org>
Mon, 8 Oct 2007 20:46:23 +0000 (20:46 +0000)
commite3206fdb39a95bb8ce3022119ef5ae9c6c2e706c
tree379734479f9e2be8f44d3e70d4280be79607a6a0
parent8e914de13f635dd9c05c63284c22adbaa2ba3800
Run FontBuilder thread after Freetype has been initialized and no error paths are can be taken. This should solve the crash reported by several people with the video title (or meta-name) appearing at start of the movie.

The freetype module starts another thread called FontBuilder and depending on timing this creates a crash or works allright. The crash is triggered by the font not being found. This patch tries to load the font first before the FontBuilder thread is started.

I was unable to reproduce the crash with the patch applied. Without the patch the crash is reproducable.

As discussed on the mailinglist the FontBuilder thread doesn't depend on the freetype initialisation and rearanging the order should do no harm.
modules/misc/freetype.c