]> git.sesse.net Git - vlc/blobdiff - src/interface/interface.c
* Mandatory step for video output IV and the audio output quality
[vlc] / src / interface / interface.c
index 9fcc08f8622d9092ef27b2bdda6cdb3f6a562fa9..b627e8c2a367bb12d46b25860240a24936e853d5 100644 (file)
@@ -4,8 +4,9 @@
  * interface, such as command line.
  *****************************************************************************
  * Copyright (C) 1998, 1999, 2000 VideoLAN
+ * $Id: interface.c,v 1.75 2001/05/01 04:18:18 sam Exp $
  *
- * Authors:
+ * Authors: Vincent Seguin <seguin@via.ecp.fr>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
 #include <stdlib.h>                                      /* free(), strtol() */
 #include <stdio.h>                                                   /* FILE */
 #include <string.h>                                            /* strerror() */
-#include <sys/types.h>                        /* on BSD, uio.h needs types.h */
-#include <sys/uio.h>                                          /* for input.h */
+#include <sys/types.h>                                              /* off_t */
 
 #include "config.h"
 #include "common.h"
 #include "threads.h"
 #include "mtime.h"
-#include "plugins.h"
-#include "playlist.h"
+#include "modules.h"
+
 #include "stream_control.h"
 #include "input_ext-intf.h"
 
@@ -47,8 +47,8 @@
 
 #include "intf_msg.h"
 #include "interface.h"
-#include "intf_cmd.h"
-#include "intf_console.h"
+#include "intf_playlist.h"
+#include "intf_channels.h"
 #include "keystrokes.h"
 
 #include "video.h"
 
 #include "main.h"
 
-/*****************************************************************************
- * intf_channel_t: channel description
- *****************************************************************************
- * A 'channel' is a descriptor of an input method. It is used to switch easily
- * from source to source without having to specify the whole input thread
- * configuration. The channels array, stored in the interface thread object, is
- * loaded in intf_Create, and unloaded in intf_Destroy.
- *****************************************************************************/
-typedef struct intf_channel_s
-{
-    /* Channel description */
-    int         i_channel;            /* channel number, -1 for end of array */
-    char *      psz_description;              /* channel description (owned) */
-
-    /* Input configuration */
-    int         i_input_method;                   /* input method descriptor */
-    char *      psz_input_source;                   /* source string (owned) */
-    int         i_input_port;                                        /* port */
-    int         i_input_vlan;                                        /* vlan */
-} intf_channel_t;
-
 /*****************************************************************************
  * Local prototypes
  *****************************************************************************/
-static int      LoadChannels    ( intf_thread_t *p_intf, char *psz_filename );
-static void     UnloadChannels  ( intf_thread_t *p_intf );
-static int      ParseChannel    ( intf_channel_t *p_channel, char *psz_str );
+static void intf_Manage( intf_thread_t *p_intf );
 
 /*****************************************************************************
  * intf_Create: prepare interface before main loop
@@ -93,161 +70,105 @@ static int      ParseChannel    ( intf_channel_t *p_channel, char *psz_str );
 intf_thread_t* intf_Create( void )
 {
     intf_thread_t * p_intf;
-    typedef void    ( intf_getplugin_t ) ( intf_thread_t * p_intf );
-    int             i_index;
-    int             i_best_index = 0, i_best_score = 0;
 
     /* Allocate structure */
     p_intf = malloc( sizeof( intf_thread_t ) );
     if( !p_intf )
     {
-        intf_ErrMsg("error: %s\n", strerror( ENOMEM ) );
+        intf_ErrMsg( "intf error: cannot create interface thread (%s)",
+                     strerror( ENOMEM ) );
         return( NULL );
     }
 
-    /* Get a suitable interface plugin */
-    for( i_index = 0 ; i_index < p_main->p_bank->i_plugin_count ; i_index++ )
-    {
-        /* If there's a plugin in p_info ... */
-        if( p_main->p_bank->p_info[ i_index ] != NULL )
-        {
-            /* ... and if this plugin provides the functions we want ... */
-            if( p_main->p_bank->p_info[ i_index ]->intf_GetPlugin != NULL )
-            {
-                /* ... and if this plugin has a good score ... */
-                if( p_main->p_bank->p_info[ i_index ]->i_score > i_best_score )
-                {
-                    /* ... then take it */
-                    i_best_score = p_main->p_bank->p_info[ i_index ]->i_score;
-                    i_best_index = i_index;
-                }
-            }
-        }
-    }
+    /* Choose the best module */
+    p_intf->p_module = module_Need( MODULE_CAPABILITY_INTF, NULL );
 
-    if( i_best_score == 0 )
+    if( p_intf->p_module == NULL )
     {
+        intf_ErrMsg( "intf error: no suitable intf module" );
         free( p_intf );
-        intf_ErrMsg( "error: no suitable plugin to create interface\n" );
         return( NULL );
     }
 
-    /* Get the plugin functions */
-    ( (intf_getplugin_t *)
-      p_main->p_bank->p_info[ i_best_index ]->intf_GetPlugin )( p_intf );
+#define f p_intf->p_module->p_functions->intf.functions.intf
+    p_intf->pf_open       = f.pf_open;
+    p_intf->pf_close      = f.pf_close;
+    p_intf->pf_run        = f.pf_run;
+#undef f
+
+    /* Initialize callbacks */
+    p_intf->pf_manage     = intf_Manage;
 
     /* Initialize structure */
-    p_intf->b_die =     0;
-    p_intf->p_vout =    NULL;
-    p_intf->p_input =   NULL;
-    p_intf->p_keys =    NULL;
+    p_intf->b_die         = 0;
 
-    /* Warning level initialisation */
-    p_intf->i_warning_level = main_GetIntVariable( INTF_WARNING_VAR, INTF_WARNING_DEFAULT );
-    
-    /* Load channels - the pointer will be set to NULL on failure. The
-     * return value is ignored since the program can work without
-     * channels */
-    LoadChannels( p_intf, main_GetPszVariable( INTF_CHANNELS_VAR, INTF_CHANNELS_DEFAULT ));
+    p_intf->p_input       = NULL;
+    p_intf->p_keys        = NULL;
+    p_intf->b_menu        = 0;
+    p_intf->b_menu_change = 0;
 
-    /* Start interfaces */
-    p_intf->p_console = intf_ConsoleCreate();
-    if( p_intf->p_console == NULL )
-    {
-        intf_ErrMsg("error: can't create control console\n");
-        free( p_intf );
-        return( NULL );
-    }
-    if( p_intf->p_sys_create( p_intf ) )
+    if( p_intf->pf_open( p_intf ) )
     {
-        intf_ErrMsg("error: can't create interface\n");
-        intf_ConsoleDestroy( p_intf->p_console );
+        intf_ErrMsg("intf error: cannot create interface");
+        module_Unneed( p_intf->p_module );
         free( p_intf );
         return( NULL );
     }
 
-    intf_Msg("Interface initialized\n");
+    /* Initialize mutexes */
+    vlc_mutex_init( &p_intf->change_lock );
+
+    /* Load channels - the pointer will be set to NULL on failure. The
+     * return value is ignored since the program can work without
+     * channels */
+    intf_LoadChannels( p_intf, main_GetPszVariable( INTF_CHANNELS_VAR,
+                                                    INTF_CHANNELS_DEFAULT ));
+
+    intf_Msg("intf: interface initialized");
     return( p_intf );
 }
 
 /*****************************************************************************
- * intf_Run
+ * intf_Manage: manage interface
  *****************************************************************************
- * Initialization script and main interface loop.
+ * This function has to be called regularly by the interface plugin. It
+ * checks for playlist end, module expiration, message flushing, and a few
+ * other useful things.
  *****************************************************************************/
-void intf_Run( intf_thread_t *p_intf )
+static void intf_Manage( intf_thread_t *p_intf )
 {
-    char * psz_server = main_GetPszVariable( INPUT_SERVER_VAR, NULL );
-    input_config_t *    p_input_config;
+    /* Flush waiting messages */
+    intf_FlushMsg();
 
-    /* If a server was specified */
-    if( psz_server )
-    {
-        if( (p_input_config =
-              (input_config_t *)malloc( sizeof(input_config_t) )) == NULL )
-        {
-            intf_ErrMsg("Out of memory");
-        }
-        else
-        {
-            p_input_config->i_method = INPUT_METHOD_UCAST;
-            p_input_config->p_source = psz_server;
-            p_input_config->p_default_aout = p_main->p_aout;
-            p_input_config->p_default_vout = p_intf->p_vout;
+    /* Manage module bank */
+    module_ManageBank( );
 
-            p_intf->p_input = input_CreateThread( p_input_config, NULL );
-        }
-    }
-    /* Or if a file was specified */
-    else if( p_main->p_playlist->p_list != NULL )
+    if( ( p_intf->p_input != NULL ) &&
+            ( p_intf->p_input->b_error || p_intf->p_input->b_eof ) )
     {
-        if( (p_input_config =
-              (input_config_t *)malloc( sizeof(input_config_t) )) == NULL )
-        {
-            intf_ErrMsg("Out of memory");
-        }
-        else
-        {
-            p_input_config->i_method = INPUT_METHOD_FILE;
-            p_input_config->p_source = p_main->p_playlist->p_list[0]; /* FIXME ??? */
-            p_input_config->p_default_aout = p_main->p_aout;
-            p_input_config->p_default_vout = p_intf->p_vout;
-
-            p_intf->p_input = input_CreateThread( p_input_config, NULL );
-        }
-    }
-    /* Execute the initialization script - if a positive number is returned,
-     * the script could be executed but failed */
-    else if( intf_ExecScript( main_GetPszVariable( INTF_INIT_SCRIPT_VAR, INTF_INIT_SCRIPT_DEFAULT ) ) > 0 )
-    {
-        intf_ErrMsg("warning: error(s) during startup script\n");
+        input_DestroyThread( p_intf->p_input, NULL );
+        p_intf->p_input = NULL;
+        intf_DbgMsg("Input thread destroyed");
     }
 
-    /* Main loop */
-    while(!p_intf->b_die)
+    /* If no stream is being played, try to find one */
+    if( p_intf->p_input == NULL && !p_intf->b_die )
     {
-        /* Flush waiting messages */
-        intf_FlushMsg();
-
-        /* Manage specific interface */
-        p_intf->p_sys_manage( p_intf );
+        /* Select the next playlist item */
+        intf_PlaylistNext( p_main->p_playlist );
 
-        /* Check attached threads status */
-        if( (p_intf->p_vout != NULL) && p_intf->p_vout->b_error )
+        if( p_main->p_playlist->i_index == -1 )
         {
-            /* FIXME: add aout error detection ?? */
+            /*    FIXME: wait for user to add stuff to playlist ? */
+#if 0
             p_intf->b_die = 1;
+#endif
         }
-        if( (p_intf->p_input != NULL) && p_intf->p_input->b_error )
+        else
         {
-            input_DestroyThread( p_intf->p_input, NULL );
-            p_intf->p_input = NULL;
-            intf_DbgMsg("Input thread destroyed\n");
+            p_intf->p_input =
+                input_CreateThread( &p_main->p_playlist->current, NULL );
         }
-
-        /* Sleep to avoid using all CPU - since some interfaces needs to access
-         * keyboard events, a 100ms delay is a good compromise */
-        msleep( INTF_IDLE_SLEEP );
     }
 }
 
@@ -260,12 +181,18 @@ void intf_Destroy( intf_thread_t *p_intf )
 {
     p_intf_key  p_cur;
     p_intf_key  p_next;
-    /* Destroy interfaces */
-    p_intf->p_sys_destroy( p_intf );
-    intf_ConsoleDestroy( p_intf->p_console );
 
     /* Unload channels */
-    UnloadChannels( p_intf );
+    intf_UnloadChannels( p_intf );
+
+    /* Destroy interfaces */
+    p_intf->pf_close( p_intf );
+
+    /* Close input thread, if any (blocking) */
+    if( p_intf->p_input )
+    {   
+        input_DestroyThread( p_intf->p_input, NULL );
+    }
 
     /* Destroy keymap */
     p_cur = p_intf->p_keys;
@@ -276,55 +203,13 @@ void intf_Destroy( intf_thread_t *p_intf )
         p_cur = p_next;
     }
          
-        
-        /* Free structure */
-    free( p_intf );
-}
-
-/*****************************************************************************
- * intf_SelectChannel: change channel
- *****************************************************************************
- * Kill existing input, if any, and try to open a new one, using an input
- * configuration table.
- *****************************************************************************/
-int intf_SelectChannel( intf_thread_t * p_intf, int i_channel )
-{
-    /* FIXME */
-#if 0
-    intf_channel_t *    p_channel;                                /* channel */
+    /* Unlock module */
+    module_Unneed( p_intf->p_module );
 
-    /* Look for channel in array */
-    if( p_intf->p_channel != NULL )
-    {
-        for( p_channel = p_intf->p_channel; p_channel->i_channel != -1; p_channel++ )
-        {
-            if( p_channel->i_channel == i_channel )
-            {
-            /*
-             * Change channel
-             */
-
-            /* Kill existing input, if any */
-            if( p_intf->p_input != NULL )
-            {
-                input_DestroyThread( p_intf->p_input, NULL );
-            }
-
-            intf_Msg("Channel %d: %s\n", i_channel, p_channel->psz_description );
-
-            /* Open a new input */
-            p_intf->p_input = input_CreateThread( p_channel->i_input_method, p_channel->psz_input_source,
-                                  p_channel->i_input_port, p_channel->i_input_vlan,
-                                  p_intf->p_vout, p_main->p_aout, NULL );
-            return( p_intf->p_input == NULL );
-            }
-        }
-    }
+    vlc_mutex_destroy( &p_intf->change_lock );
 
-    /* Channel does not exist */
-    intf_Msg("Channel %d does not exist\n", i_channel );
-#endif
-    return( 1 );
+    /* Free structure */
+    free( p_intf );
 }
 
 /*****************************************************************************
@@ -421,6 +306,7 @@ void intf_AssignNormalKeys( intf_thread_t *p_intf)
     intf_AssignKey( p_intf , ' ', INTF_KEY_TOGGLE_INTERFACE, 0);
     intf_AssignKey( p_intf , 'i', INTF_KEY_TOGGLE_INFO, 0);
     intf_AssignKey( p_intf , 's', INTF_KEY_TOGGLE_SCALING, 0);
+    intf_AssignKey( p_intf , 'd', INTF_KEY_DUMP_STREAM, 0);
 }   
 
 /*****************************************************************************
@@ -435,86 +321,58 @@ int intf_ProcessKey( intf_thread_t *p_intf, int g_key )
     keyparm k_reply;
     
     k_reply = intf_GetKey( p_intf, g_key); 
-    
     switch( k_reply.key )
     {
-    case INTF_KEY_QUIT:                                                  /* quit order */
+    case INTF_KEY_QUIT:                                        /* quit order */
         p_intf->b_die = 1;
         break;
     case INTF_KEY_SET_CHANNEL:
         /* Change channel - return code is ignored since SelectChannel displays
          * its own error messages */
-        intf_SelectChannel( p_intf, k_reply.param );
-        break;
-    case INTF_KEY_INC_VOLUME:                                                    /* volume + */
-        if( (p_main->p_aout != NULL) && (p_main->p_aout->vol < VOLMAX) )
-            p_main->p_aout->vol += VOLSTEP;
-        break;
-    case INTF_KEY_DEC_VOLUME:                                                    /* volume - */
-        if( (p_main->p_aout != NULL) && (p_main->p_aout->vol > VOLSTEP) )
-            p_main->p_aout->vol -= VOLSTEP;
+/*        intf_SelectChannel( p_intf, k_reply.param ); */
+/*        network_ChannelJoin() */
+/* FIXME : keyboard event is for the time being half handled by the interface
+ * half handled directly by the plugins. We should decide what to do. */        
         break;
-    case INTF_KEY_TOGGLE_VOLUME:                                                 /* toggle mute */
-        if( (p_main->p_aout != NULL) && (p_main->p_aout->vol))
-        {
-            i_volbackup = p_main->p_aout->vol;
-            p_main->p_aout->vol = 0;
-        }
-        else if( (p_main->p_aout != NULL) && (!p_main->p_aout->vol))
-            p_main->p_aout->vol = i_volbackup;
+    case INTF_KEY_INC_VOLUME:                                    /* volume + */
+        if( (p_main->p_aout != NULL) && (p_main->p_aout->i_vol < VOLUME_MAX) )
+            p_main->p_aout->i_vol += VOLUME_STEP;
         break;
-    case INTF_KEY_DEC_GAMMA:                                                     /* gamma - */
-        if( (p_intf->p_vout != NULL) && (p_intf->p_vout->f_gamma > -INTF_GAMMA_LIMIT) )
-        {
-            vlc_mutex_lock( &p_intf->p_vout->change_lock );
-            p_intf->p_vout->f_gamma   -= INTF_GAMMA_STEP;
-            p_intf->p_vout->i_changes |= VOUT_GAMMA_CHANGE;
-            vlc_mutex_unlock( &p_intf->p_vout->change_lock );
-        }
+    case INTF_KEY_DEC_VOLUME:                                    /* volume - */
+        if( (p_main->p_aout != NULL) && (p_main->p_aout->i_vol > VOLUME_STEP) )
+            p_main->p_aout->i_vol -= VOLUME_STEP;
         break;
-    case INTF_KEY_INC_GAMMA:                                                     /* gamma + */
-        if( (p_intf->p_vout != NULL) && (p_intf->p_vout->f_gamma < INTF_GAMMA_LIMIT) )
+    case INTF_KEY_TOGGLE_VOLUME:                              /* toggle mute */
+        if( (p_main->p_aout != NULL) && (p_main->p_aout->i_vol))
         {
-            vlc_mutex_lock( &p_intf->p_vout->change_lock );
-            p_intf->p_vout->f_gamma   += INTF_GAMMA_STEP;
-            p_intf->p_vout->i_changes |= VOUT_GAMMA_CHANGE;
-            vlc_mutex_unlock( &p_intf->p_vout->change_lock );
+            i_volbackup = p_main->p_aout->i_vol;
+            p_main->p_aout->i_vol = 0;
         }
+        else if( (p_main->p_aout != NULL) && (!p_main->p_aout->i_vol))
+            p_main->p_aout->i_vol = i_volbackup;
         break;
-    case INTF_KEY_TOGGLE_GRAYSCALE:                                            /* toggle grayscale */
-        if( p_intf->p_vout != NULL )
+    case INTF_KEY_DEC_GAMMA:                                      /* gamma - */
+        if( (p_main->p_vout != NULL) && (p_main->p_vout->f_gamma > -INTF_GAMMA_LIMIT) )
         {
-            vlc_mutex_lock( &p_intf->p_vout->change_lock );
-            p_intf->p_vout->b_grayscale = !p_intf->p_vout->b_grayscale;
-            p_intf->p_vout->i_changes  |= VOUT_GRAYSCALE_CHANGE;
-            vlc_mutex_unlock( &p_intf->p_vout->change_lock );
+            /* FIXME: we should lock if called from the interface */
+            p_main->p_vout->f_gamma   -= INTF_GAMMA_STEP;
+            p_main->p_vout->i_changes |= VOUT_GAMMA_CHANGE;
         }
         break;
-    case INTF_KEY_TOGGLE_INTERFACE:                                            /* toggle interface */
-        if( p_intf->p_vout != NULL )
+    case INTF_KEY_INC_GAMMA:                                      /* gamma + */
+        if( (p_main->p_vout != NULL) && (p_main->p_vout->f_gamma < INTF_GAMMA_LIMIT) )
         {
-            vlc_mutex_lock( &p_intf->p_vout->change_lock );
-            p_intf->p_vout->b_interface     = !p_intf->p_vout->b_interface;
-            p_intf->p_vout->i_changes |= VOUT_INTF_CHANGE;
-            vlc_mutex_unlock( &p_intf->p_vout->change_lock );
+            /* FIXME: we should lock if called from the interface */
+            p_main->p_vout->f_gamma   += INTF_GAMMA_STEP;
+            p_main->p_vout->i_changes |= VOUT_GAMMA_CHANGE;
         }
         break;
-    case INTF_KEY_TOGGLE_INFO:                                                 /* toggle info */
-        if( p_intf->p_vout != NULL )
+   case INTF_KEY_DUMP_STREAM:
+        if( p_intf->p_input != NULL )
         {
-            vlc_mutex_lock( &p_intf->p_vout->change_lock );
-            p_intf->p_vout->b_info     = !p_intf->p_vout->b_info;
-            p_intf->p_vout->i_changes |= VOUT_INFO_CHANGE;
-            vlc_mutex_unlock( &p_intf->p_vout->change_lock );
-        }
-        break;
-    case INTF_KEY_TOGGLE_SCALING:                                              /* toggle scaling */
-        if( p_intf->p_vout != NULL )
-        {
-            vlc_mutex_lock( &p_intf->p_vout->change_lock );
-            p_intf->p_vout->b_scale    = !p_intf->p_vout->b_scale;
-            p_intf->p_vout->i_changes |= VOUT_SCALE_CHANGE;
-            vlc_mutex_unlock( &p_intf->p_vout->change_lock );
+            vlc_mutex_lock( &p_intf->p_input->stream.stream_lock );
+            input_DumpStream( p_intf->p_input );
+            vlc_mutex_unlock( &p_intf->p_input->stream.stream_lock );
         }
         break;
    default:                                                   /* unknown key */
@@ -523,240 +381,3 @@ int intf_ProcessKey( intf_thread_t *p_intf, int g_key )
 
     return( 0 );
 }
-
-/* following functions are local */
-
-/*****************************************************************************
- * LoadChannels: load channels description from a file
- *****************************************************************************
- * This structe describes all interface-specific data of the main (interface)
- * thread.
- * Each line of the file is a semicolon separated list of the following
- * fields :
- *      integer         channel number
- *      string          channel description
- *      integer         input method (see input.h)
- *      string          input source
- *      integer         input port
- *      integer         input vlan
- * The last field must end with a semicolon.
- * Comments and empty lines are not explicitely allowed, but lines with parsing
- * errors are ignored without warning.
- *****************************************************************************/
-static int LoadChannels( intf_thread_t *p_intf, char *psz_filename )
-{
-    FILE *              p_file;                                      /* file */
-    intf_channel_t *    p_channel;                        /* current channel */
-    char                psz_line[INTF_MAX_CMD_SIZE];          /* line buffer */
-    int                 i_index;                   /* channel or field index */
-
-    /* Set default value */
-    p_intf->p_channel = NULL;
-
-    /* FIXME: channels are disabled */
-    //return( 0 );
-
-    /* Open file */
-    p_file = fopen( psz_filename, "r" );
-    if( p_file == NULL )
-    {
-        intf_ErrMsg("error: can't open %s (%s)\n", psz_filename, strerror(errno));
-        return( 1 );
-    }
-
-    /* First pass: count number of lines */
-    for( i_index = 0; fgets( psz_line, INTF_MAX_CMD_SIZE, p_file ) != NULL; i_index++ )
-    {
-        ;
-    }
-
-    if( i_index != 0 )
-    {
-        /* Allocate array and rewind - some of the lines may be invalid, and the
-         * array will probably be larger than the actual number of channels, but
-         * it has no consequence. */
-        p_intf->p_channel = malloc( sizeof( intf_channel_t ) * i_index );
-        if( p_intf->p_channel == NULL )
-        {
-            intf_ErrMsg("error: %s\n", strerror(ENOMEM));
-            fclose( p_file );
-            return( 1 );
-        }
-        p_channel = p_intf->p_channel;
-        rewind( p_file );
-
-        /* Second pass: read channels descriptions */
-        while( fgets( psz_line, INTF_MAX_CMD_SIZE, p_file ) != NULL )
-        {
-            if( !ParseChannel( p_channel, psz_line ) )
-            {
-                intf_DbgMsg( "channel [%d] %s : method %d (%s:%d vlan %d)\n",
-                         p_channel->i_channel, p_channel->psz_description,
-                         p_channel->i_input_method,
-                         p_channel->psz_input_source,
-                         p_channel->i_input_port, p_channel->i_input_vlan );
-                p_channel++;
-            }
-        }
-
-        /* Add marker at the end of the array */
-        p_channel->i_channel = -1;
-    }
-
-    /* Close file */
-    fclose( p_file );
-    return( 0 );
-}
-
-/*****************************************************************************
- * UnloadChannels: unload channels description
- *****************************************************************************
- * This function free all resources allocated by LoadChannels, if any.
- *****************************************************************************/
-static void UnloadChannels( intf_thread_t *p_intf )
-{
-    int i_channel;                                          /* channel index */
-
-    if( p_intf->p_channel != NULL )
-    {
-        /* Free allocated strings */
-        for( i_channel = 0;
-             p_intf->p_channel[ i_channel ].i_channel != -1;
-             i_channel++ )
-        {
-            if( p_intf->p_channel[ i_channel ].psz_description != NULL )
-            {
-                free( p_intf->p_channel[ i_channel ].psz_description );
-            }
-            if( p_intf->p_channel[ i_channel ].psz_input_source != NULL )
-            {
-                free( p_intf->p_channel[ i_channel ].psz_input_source );
-            }
-        }
-
-        /* Free array */
-        free( p_intf->p_channel );
-        p_intf->p_channel = NULL;
-    }
-}
-
-
-/*****************************************************************************
- * ParseChannel: parse a channel description line
- *****************************************************************************
- * See LoadChannels. This function return non 0 on parsing error.
- *****************************************************************************/
-static int ParseChannel( intf_channel_t *p_channel, char *psz_str )
-{
-    char *      psz_index;                              /* current character */
-    char *      psz_end;                           /* end pointer for strtol */
-    int         i_field;                        /* field number, -1 on error */
-    int         i_field_length;             /* field length, for text fields */
-
-    /* Set some default fields */
-    p_channel->i_channel =              0;
-    p_channel->psz_description =        NULL;
-    p_channel->i_input_method =         0;
-    p_channel->psz_input_source =       NULL;
-    p_channel->i_input_port =           0;
-    p_channel->i_input_vlan =           0;
-
-    /* Parse string */
-    i_field = 0;
-    for( psz_index = psz_str; (i_field != -1) && (*psz_index != '\0'); psz_index++ )
-    {
-        if( *psz_index == ';' )
-        {
-            /* Mark end of field */
-            *psz_index = '\0';
-
-            /* Parse field */
-            switch( i_field++ )
-            {
-            case 0:                                        /* channel number */
-                p_channel->i_channel = strtol( psz_str, &psz_end, 0);
-                if( (*psz_str == '\0') || (*psz_end != '\0') )
-                {
-                    i_field = -1;
-                }
-                break;
-            case 1:                                   /* channel description */
-                i_field_length = strlen( psz_str );
-                if( i_field_length != 0 )
-                {
-                    p_channel->psz_description = malloc( i_field_length + 1 );
-                    if( p_channel->psz_description == NULL )
-                    {
-                        intf_ErrMsg("error: %s\n", strerror( ENOMEM ));
-                        i_field = -1;
-                    }
-                    else
-                    {
-                        strcpy( p_channel->psz_description, psz_str );
-                    }
-                }
-                break;
-            case 2:                                          /* input method */
-                p_channel->i_input_method = strtol( psz_str, &psz_end, 0);
-                if( (*psz_str == '\0') || (*psz_end != '\0') )
-                {
-                    i_field = -1;
-                }
-                break;
-            case 3:                                          /* input source */
-                i_field_length = strlen( psz_str );
-                if( i_field_length != 0 )
-                {
-                    p_channel->psz_input_source = malloc( i_field_length + 1 );
-                    if( p_channel->psz_input_source == NULL )
-                    {
-                        intf_ErrMsg("error: %s\n", strerror( ENOMEM ));
-                        i_field = -1;
-                    }
-                    else
-                    {
-                        strcpy( p_channel->psz_input_source, psz_str );
-                    }
-                }
-                break;
-            case 4:                                            /* input port */
-                p_channel->i_input_port = strtol( psz_str, &psz_end, 0);
-                if( (*psz_str == '\0') || (*psz_end != '\0') )
-                {
-                    i_field = -1;
-                }
-                break;
-            case 5:                                            /* input vlan */
-                p_channel->i_channel = strtol( psz_str, &psz_end, 0);
-                if( (*psz_str == '\0') || (*psz_end != '\0') )
-                {
-                    i_field = -1;
-                }
-                break;
-                /* ... following fields are ignored */
-            }
-
-            /* Set new beginning of field */
-            psz_str = psz_index + 1;
-        }
-    }
-
-    /* At least the first three fields must be parsed sucessfully for function
-     * success. Other parsing errors are returned using i_field = -1. */
-    if( i_field < 3 )
-    {
-        /* Function fails. Free allocated strings */
-        if( p_channel->psz_description != NULL )
-        {
-            free( p_channel->psz_description );
-        }
-        if( p_channel->psz_input_source != NULL )
-        {
-            free( p_channel->psz_input_source );
-        }
-        return( 1 );
-    }
-
-    /* Return success */
-    return( 0 );
-}