comparison gui/interface.c @ 33307:552f1f7731c8

Set GUI initialization flag earlier. So far, the flag is set after the call of guiInit(), but there are calls to GUI functions (like reading the GUI config) before that. As MPlayer (when exiting) only calls GUI's cleanup function guiDone() if the flag is set, it neglects calling guiDone() on any error between the first GUI function call and execution of guiInit(). Now the flag is set before the first GUI function call. As a result, GUI's own exit function (also used for fatal GUI errors, and thus also for errors occurring before guiInit() properly finishes) doesn't have to handle cleanup any longer. The code can be moved to guiDone().

33306:37e215798390

#ifdef CONFIG_DVDREAD
#include "stream/stream_dvd.h"
#endif

#define DONE -1

guiInterface_t guiIntfStruct;

int guiWinID = -1;

char *skinName;

    initialized = 1;
}

void guiDone(void)
{
    mplMainRender = 0;

    mp_msg(MSGT_GPLAYER, MSGL_V, "[GUI] done.\n");

    if (gui_save_pos) {

    ass_bottom_margin = gtkASS.bottom_margin;
#endif

    cfg_write();
    wsXDone();

    guiExit(DONE);
}

// NOTE TO MYSELF: Before calling guiInit(), MPlayer calls GUI functions
//                 cfg_read() and import_initial_playtree_into_gui(). Only
//                 after guiInit() has been called successfully, guiDone()
//                 (and thus guiExit()) will be executed by MPlayer on exit.
//                 In other words, any MPlayer's exit between cfg_read() and
//                 guiInit() will not execute guiDone().
//                 With this function it is at least possible to handle
//                 GUI's own abortions during (and before) guiInit().
void guiExit(int how)
{
    if (!initialized || (how == DONE)) {
        if (gui_conf) {
            m_config_free(gui_conf);
            gui_conf = NULL;
        }

        appFreeStruct();
    }

    if (how != DONE)
        exit_player_with_rc(how, how >= EXIT_ERROR);
}

void guiLoadFont(void)
{
#ifdef CONFIG_FREETYPE

33307:552f1f7731c8

#ifdef CONFIG_DVDREAD
#include "stream/stream_dvd.h"
#endif

guiInterface_t guiIntfStruct;

int guiWinID = -1;

char *skinName;

    initialized = 1;
}

void guiDone(void)
{
    if (initialized) {
    mplMainRender = 0;

    mp_msg(MSGT_GPLAYER, MSGL_V, "[GUI] done.\n");

    if (gui_save_pos) {

    ass_bottom_margin = gtkASS.bottom_margin;
#endif

    cfg_write();
    wsXDone();
    }

    appFreeStruct();

    if (gui_conf) {
        m_config_free(gui_conf);
        gui_conf = NULL;
    }
}

void guiExit(int how)
{
    exit_player_with_rc(how, how >= EXIT_ERROR);
}

void guiLoadFont(void)
{
#ifdef CONFIG_FREETYPE