From e7b0790a8639643acf46060d68e3a3ea9627e174 Mon Sep 17 00:00:00 2001 From: Pierre d'Herbemont Date: Sun, 6 Jul 2008 18:43:06 +0200 Subject: [PATCH] Revert "messages: Prefer the object name over the object type if type is VLC_GENERIC." This reverts commit 0386618539878fde94110df15b77ba7c79e54e8c. Conflicts: src/misc/messages.c --- include/vlc_messages.h | 2 +- modules/gui/beos/MessagesWindow.cpp | 2 +- src/control/log.c | 2 +- src/misc/messages.c | 13 +++---------- 4 files changed, 6 insertions(+), 13 deletions(-) diff --git a/include/vlc_messages.h b/include/vlc_messages.h index 6035f81d48..1ce7d7b914 100644 --- a/include/vlc_messages.h +++ b/include/vlc_messages.h @@ -44,7 +44,7 @@ typedef struct { int i_type; /**< message type, see below */ int i_object_id; - char * psz_object; + const char *psz_object_type; char * psz_module; char * psz_msg; /**< the message itself */ char * psz_header; /**< Additional header */ diff --git a/modules/gui/beos/MessagesWindow.cpp b/modules/gui/beos/MessagesWindow.cpp index b83107eff0..b4b2029330 100644 --- a/modules/gui/beos/MessagesWindow.cpp +++ b/modules/gui/beos/MessagesWindow.cpp @@ -80,7 +80,7 @@ void MessagesView::Pulse() case VLC_MSG_DBG: color = gray; break; } - psz_module_type = p_sub->p_msg[i_start].psz_object; + psz_module_type = p_sub->p_msg[i_start].psz_object_type; if( LockLooper() ) { diff --git a/src/control/log.c b/src/control/log.c index 11b0fe617b..b8fb181136 100644 --- a/src/control/log.c +++ b/src/control/log.c @@ -178,7 +178,7 @@ libvlc_log_message_t *libvlc_log_iterator_next( libvlc_log_iterator_t *p_iter, vlc_mutex_lock(p_iter->p_messages->p_lock); msg = p_iter->p_messages->p_msg+i_pos; buffer->i_severity = msg->i_type; - buffer->psz_type = msg->psz_object; + buffer->psz_type = msg->psz_object_type; buffer->psz_name = msg->psz_module; buffer->psz_header = msg->psz_header; buffer->psz_message = msg->psz_msg; diff --git a/src/misc/messages.c b/src/misc/messages.c index f00a1de1ed..be832c65c8 100644 --- a/src/misc/messages.c +++ b/src/misc/messages.c @@ -78,11 +78,6 @@ static void QueueMsg ( vlc_object_t *, int, const char *, static void FlushMsg ( msg_queue_t * ); static void PrintMsg ( vlc_object_t *, msg_item_t * ); -static inline char * object_description( vlc_object_t * p_this ) -{ - return strdup( p_this->psz_object_type ); -} - /** * Initialize messages queues * This function initializes all message queues @@ -427,7 +422,7 @@ static void QueueMsg( vlc_object_t *p_this, int i_type, const char *psz_module, p_item->i_type = VLC_MSG_WARN; p_item->i_object_id = p_this->i_object_id; - p_item->psz_object = object_description( p_this ); + p_item->psz_object_type = p_this->psz_object_type; p_item->psz_module = strdup( "message" ); p_item->psz_msg = strdup( "message queue overflowed" ); p_item->psz_header = NULL; @@ -448,7 +443,7 @@ static void QueueMsg( vlc_object_t *p_this, int i_type, const char *psz_module, /* Fill message information fields */ p_item->i_type = i_type; p_item->i_object_id = p_this->i_object_id; - p_item->psz_object = object_description( p_this ); + p_item->psz_object_type = p_this->psz_object_type; p_item->psz_module = strdup( psz_module ); p_item->psz_msg = psz_str; p_item->psz_header = psz_header; @@ -460,7 +455,6 @@ static void QueueMsg( vlc_object_t *p_this, int i_type, const char *psz_module, free( p_item->psz_module ); free( p_item->psz_msg ); free( p_item->psz_header ); - free( p_item->psz_object ); } vlc_mutex_unlock ( &p_queue->lock ); @@ -506,7 +500,6 @@ static void FlushMsg ( msg_queue_t *p_queue ) free( p_queue->msg[i_index].psz_msg ); free( p_queue->msg[i_index].psz_module ); free( p_queue->msg[i_index].psz_header ); - free( p_queue->msg[i_index].psz_object ); } /* Update the new start value */ @@ -552,7 +545,7 @@ static void PrintMsg ( vlc_object_t * p_this, msg_item_t * p_item ) break; } - psz_object = p_item->psz_object; + psz_object = p_item->psz_object_type; #ifdef UNDER_CE # define CE_WRITE(str) WriteFile( QUEUE.logfile, \ -- 2.39.2